Closed
Bug 714230
Opened 9 years ago
Closed 9 years ago
Opening new tabs in the background suppresses rendering and it never unsuppresses
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox11 unaffected, firefox12 fixed, firefox13 verified)
VERIFIED
FIXED
Firefox 12
Tracking | Status | |
---|---|---|
firefox11 | --- | unaffected |
firefox12 | --- | fixed |
firefox13 | --- | verified |
People
(Reporter: pcwalton, Assigned: pcwalton)
References
Details
(Whiteboard: [not-fennec-11])
Attachments
(1 file)
1.21 KB,
patch
|
kats
:
review+
|
Details | Diff | Splinter Review |
My patches for bug 709492 made tabs opened in the background no longer appear. Sorry. Here's a fix.
Attachment #584918 -
Flags: review?(bugmail.mozilla)
Comment 1•9 years ago
|
||
Comment on attachment 584918 [details] [diff] [review] Proposed patch. Review of attachment 584918 [details] [diff] [review]: ----------------------------------------------------------------- Good thing you caught it early :)
Attachment #584918 -
Flags: review?(bugmail.mozilla) → review+
Assignee | ||
Comment 2•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/f2610136fb54
Comment 3•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d4c526ba5343
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12
Comment 4•9 years ago
|
||
Patrick, please request aurora approval
Comment 5•9 years ago
|
||
Comment on attachment 584918 [details] [diff] [review] Proposed patch. [Triage Comment] Needed on beta since it fixes a regression caused by bug 709492 which is on beta
Attachment #584918 -
Flags: approval-mozilla-beta+
Comment 6•9 years ago
|
||
(In reply to Phil Ringnalda (:philor) from comment #3) > https://hg.mozilla.org/mozilla-central/rev/d4c526ba5343 This should really say https://hg.mozilla.org/mozilla-central/rev/f2610136fb54
tracking-fennec: --- → ?
status-firefox11:
--- → affected
status-firefox12:
--- → fixed
status-firefox13:
--- → fixed
Comment 7•9 years ago
|
||
So this patch was actually backed out as part of the backout of bug 709492 (see comment 61 on that bug), and the modified re-landed version didn't need this patch. So it doesn't need to be uplifted anywhere.
Updated•9 years ago
|
Whiteboard: [not-fennec-11]
Comment 8•9 years ago
|
||
Comment on attachment 584918 [details] [diff] [review] Proposed patch. Removing beta approval since this doesn't need to be uplifted.
Attachment #584918 -
Flags: approval-mozilla-beta+
Comment 9•9 years ago
|
||
This is only about the tabs opened in background that were not appearing or I should look at some other bad behavior also in order to verify it?
Comment 10•9 years ago
|
||
Yeah, this is only about tabs opened in background. STR would be something like: 1) Open some page 2) Long tap on a link, open new tab 3) Wait a while for the background tab to finish loading 4) Switch to the new tab If the new tab shows up normally then that's good. You can also try without step 3, so switching tabs while it's still loading. That should work as well.
Comment 11•9 years ago
|
||
Verified on: Nightly Fennec 13.0a1 (2012-03-04) Device: HTC Desire Z OS: Android 2.3.3 The new tab shows up normally.
Status: RESOLVED → VERIFIED
Updated•9 years ago
|
Updated•8 years ago
|
tracking-fennec: ? → ---
Updated•4 months 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
•