Closed
Bug 1337459
Opened 7 years ago
Closed 7 years ago
Tab URL is no longer displayed in URL bar during session restore before Gecko has loaded
Categories
(Firefox for Android Graveyard :: General, defect, P1)
Tracking
(fennec53+, firefox51 unaffected, firefox52 unaffected, firefox53 verified, firefox54 verified)
VERIFIED
FIXED
Firefox 54
Tracking | Status | |
---|---|---|
fennec | 53+ | --- |
firefox51 | --- | unaffected |
firefox52 | --- | unaffected |
firefox53 | --- | verified |
firefox54 | --- | verified |
People
(Reporter: JanH, Assigned: jchen)
References
Details
(Keywords: regression)
Attachments
(1 file)
2.27 KB,
patch
|
sebastian
:
review+
lizzard
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
STR: 1. Have at least on tab with some content open. 2. Kill Firefox and start it again. Expected: The URL of the restored foreground tab is displayed as soon as the UI has started up. Actual: The awesomebar is displaying its default "about:home" text ("Search or enter adress") and is only updated once Gecko has started and is reloading the page. Regressing push is this one: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=e8d6755b7a392f5cc2b3aa58df5b185ef620747f&filter-searchStr=android%2015%20opt
Flags: needinfo?(nchen)
Updated•7 years ago
|
tracking-fennec: ? → 53+
Updated•7 years ago
|
Priority: -- → P1
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → nchen
Status: NEW → ASSIGNED
Flags: needinfo?(nchen)
Assignee | ||
Comment 1•7 years ago
|
||
We don't update ToolbarDisplayLayout when it's not ready (i.e. when it's not attached to a window yet), but when it does become ready, we should update it to the selected tab, if any.
Attachment #8837903 -
Flags: review?(s.kaspari)
Updated•7 years ago
|
Attachment #8837903 -
Flags: review?(s.kaspari) → review+
Pushed by nchen@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/c94cfb623248 Update to selected tab when ToolbarDisplayLayout is ready; r=sebastian
Comment 3•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c94cfb623248
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54
Assignee | ||
Comment 4•7 years ago
|
||
Comment on attachment 8837903 [details] [diff] [review] Update to selected tab when ToolbarDisplayLayout is ready (v1) Approval Request Comment [Feature/Bug causing the regression]: Bug 1329268 [User impact if declined]: Not as good user experience; URL not shown when restoring tabs [Is this code covered by automated tests?]: No [Has the fix been verified in Nightly?]: Yes [Needs manual test from QE? If yes, steps to reproduce]: No [List of other uplifts needed for the feature/fix]: None [Is the change risky?]: Not very [Why is the change risky/not risky?]: Patch is small and only affects certain scenarios when starting up [String changes made/needed]: None
Attachment #8837903 -
Flags: approval-mozilla-aurora?
Comment 5•7 years ago
|
||
Comment on attachment 8837903 [details] [diff] [review] Update to selected tab when ToolbarDisplayLayout is ready (v1) Sounds best to keep things consistent for users - let's uplift to aurora.
Attachment #8837903 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 6•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/e49433bc9ada
Comment 7•7 years ago
|
||
Verified that the issue was fixed on Aurora 54 (3/16/2017) and Beta 53.0b2. Devices: -Lenovo Yoga Tablet 2 (Android 4.4.2) -Huawei Honor 5X (Android 5.1.1)
Updated•3 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
•