Properly handle google.co.* (and more generally google based urls where the SLD is not google)
Bug #1517527 reported by
Alexandre Abreu
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical System Image |
Fix Released
|
High
|
David Barth | ||
| The Webapps-core project |
Fix Released
|
High
|
Alexandre Abreu | ||
| webapps-sprint |
Fix Released
|
High
|
Alexandre Abreu | ||
| webbrowser-app (Ubuntu) |
Fix Released
|
High
|
Alexandre Abreu | ||
Bug Description
See https:/
URL patterns such as google.co.* that cover google.co.uk, google.co.in, etc. are not valid, so might trigger a redirect to the browser in some instances,
Related branches
lp:~abreu-alexandre/webbrowser-app/enhance-google-url-patterns
- Olivier Tilloy: Approve
- PS Jenkins bot: Approve (continuous-integration)
-
Diff: 59 lines (+17/-2)2 files modifiedsrc/app/webcontainer/url-pattern-utils.cpp (+8/-2)
tests/unittests/container-url-patterns/tst_ContainerUrlPatternsTests.cpp (+9/-0)
| Changed in webapps-core: | |
| assignee: | nobody → Alexandre Abreu (abreu-alexandre) |
| importance: | Undecided → High |
| status: | New → Triaged |
| Changed in webapps-sprint: | |
| assignee: | nobody → Alexandre Abreu (abreu-alexandre) |
| Changed in webbrowser-app (Ubuntu): | |
| assignee: | nobody → Alexandre Abreu (abreu-alexandre) |
| Changed in webapps-sprint: | |
| milestone: | none → sprint-16 |
| importance: | Undecided → High |
| Changed in webbrowser-app (Ubuntu): | |
| importance: | Undecided → High |
| Changed in webapps-sprint: | |
| status: | New → Triaged |
| Changed in webbrowser-app (Ubuntu): | |
| status: | New → Triaged |
| Changed in webapps-core: | |
| status: | Triaged → In Progress |
| Changed in webapps-sprint: | |
| status: | Triaged → In Progress |
| Changed in webbrowser-app (Ubuntu): | |
| status: | Triaged → In Progress |
| Changed in webapps-sprint: | |
| milestone: | sprint-16 → sprint-17 |
| Changed in webapps-sprint: | |
| milestone: | sprint-17 → sprint-18 |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #1 |
| Changed in webbrowser-app (Ubuntu): | |
| status: | In Progress → Fix Released |
| Changed in canonical-devices-system-image: | |
| status: | New → Fix Committed |
| importance: | Undecided → High |
| assignee: | nobody → Bill Filler (bfiller) |
| milestone: | none → ww08-2016 |
| Changed in webapps-sprint: | |
| status: | In Progress → Fix Released |
| Changed in webapps-core: | |
| status: | In Progress → Fix Released |
| Changed in canonical-devices-system-image: | |
| assignee: | Bill Filler (bfiller) → David Barth (dbarth) |
| Changed in canonical-devices-system-image: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
This bug was fixed in the package webbrowser-app - 0.23+16.
---------------
webbrowser-app (0.23+16.
[ Alexandre Abreu ]
* Enhance google.com specific patterns to enable co.* & com.* patterns
(LP: #1517527)
* Make sure that the overlay views closed by the user are not part of
the list of recovered overlays opened after the oom closed the app.
(LP: #1535154)
[ CI Train Bot ]
* Resync trunk. added: po/sco.po
-- Alexandre Abreu <email address hidden> Tue, 02 Feb 2016 15:33:33 +0000