Merge lp://staging/~ted/url-dispatcher/intent-pkg-domain-trunk into lp://staging/url-dispatcher/15.04
Proposed by
Ted Gould
Status: | Merged |
---|---|
Approved by: | Charles Kerr |
Approved revision: | 91 |
Merged at revision: | 82 |
Proposed branch: | lp://staging/~ted/url-dispatcher/intent-pkg-domain-trunk |
Merge into: | lp://staging/url-dispatcher/15.04 |
Diff against target: |
209 lines (+112/-1) 8 files modified
service/dispatcher.c (+27/-1) service/update-directory.c (+10/-0) tests/directory-update-test.cc (+22/-0) tests/dispatcher-test.cc (+25/-0) tests/test-config.h.in (+1/-0) tests/test-urls-intent/intent-mixed.url-dispatcher (+13/-0) tests/test-urls-intent/intent-no-good.url-dispatcher (+8/-0) tests/test-urls-intent/intent-single.url-dispatcher (+6/-0) |
To merge this branch: | bzr merge lp://staging/~ted/url-dispatcher/intent-pkg-domain-trunk |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Charles Kerr (community) | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Review via email: mp+245978@code.staging.launchpad.net |
Commit message
Special handling for intent URLs
Description of the change
Makes it so that intent URLs are handled differently in that we use our current domain suffix for the package name instead of the domain. We don't want to choose packages based on domain there, as it isn't as important as the package that is supposed to be used. Many webapps think we're Android, so they send us intent URLs.
To post a comment you must log in.
FAILED: Continuous integration, rev:90 jenkins. qa.ubuntu. com/job/ url-dispatcher- ci/158/ jenkins. qa.ubuntu. com/job/ url-dispatcher- vivid-amd64- ci/1 jenkins. qa.ubuntu. com/job/ url-dispatcher- vivid-armhf- ci/1 jenkins. qa.ubuntu. com/job/ url-dispatcher- vivid-i386- ci/1/console
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/url- dispatcher- ci/158/ rebuild
http://