Closed
Bug 791612
Opened 13 years ago
Closed 13 years ago
"other origin" bar persists after pressing Back button to return to same-origin page
Categories
(Firefox for Android Graveyard :: Web Apps (PWAs), defect, P1)
Tracking
(firefox17 fixed, firefox18 verified)
VERIFIED
FIXED
Firefox 18
People
(Reporter: myk, Assigned: wesj)
Details
(Whiteboard: [blocking-webrtandroid1+])
Attachments
(2 files)
248.50 KB,
image/png
|
Details | |
932 bytes,
patch
|
mfinkle
:
review+
akeybl
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
If you go from a page in a webapp's origin to a page in another origin, then press the Android Back button to return to the page in the webapp's origin, the "other origin" bar doesn't go away.
Steps to Reproduce:
1. install and open the Firefox Marketplace app;
2. tap the "Log in / Register" button to go to a page at the origin https://login.persona.org;
3. tap the Android Back button.
Expected Results: the origin bar disappears.
Actual Results: the origin bar persists (and continues to show the origin https://login.persona.org).
Updated•13 years ago
|
Priority: -- → P1
Whiteboard: [blocking-webrtandroid1+]
Assignee | ||
Comment 1•13 years ago
|
||
In this case we're actually closing a tab and selection an old tab. No "location change" event fires. I think we probably want to update this any time the selected tab changes though.
Attachment #663171 -
Flags: review?(mark.finkle)
Comment 2•13 years ago
|
||
Comment on attachment 663171 [details] [diff] [review]
Patch
Agreed
Attachment #663171 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 3•13 years ago
|
||
Assignee | ||
Comment 4•13 years ago
|
||
Comment on attachment 663171 [details] [diff] [review]
Patch
[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 771915
User impact if declined: Other origin bar persists some times when it shouldn't
Testing completed (on m-c, etc.): Landed on mc today
Risk to taking this patch (and alternatives if risky): Very low risk. Webapp only fix for a situation that's fairly rare.
String or UUID changes made by this patch: None.
Attachment #663171 -
Flags: approval-mozilla-aurora?
Comment 5•13 years ago
|
||
Comment on attachment 663171 [details] [diff] [review]
Patch
[Triage Comment]
Bug fix for new feature - approving for Aurora 17.
Attachment #663171 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 6•13 years ago
|
||
Assignee: nobody → wjohnston
Status: NEW → RESOLVED
Closed: 13 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → Firefox 18
Updated•13 years ago
|
Status: RESOLVED → VERIFIED
status-firefox18:
--- → verified
Assignee | ||
Comment 7•13 years ago
|
||
Updated•13 years ago
|
status-firefox17:
--- → fixed
Updated•5 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•