Closed
Bug 783355
Opened 12 years ago
Closed 12 years ago
Gaia Settings app doesn't fully repaint when it loads
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
DUPLICATE
of bug 783379
blocking-basecamp | + |
People
(Reporter: cjones, Assigned: khuey)
References
Details
Attachments
(1 file)
27.16 KB,
image/png
|
Details |
STR (1) Get otoro build from inbound ~early this morning (11/16) (2) Launch settings app See attached screenshot --- we don't fully repaint the screen. Just got an otoro again so I'm not sure if this is new or not. Vivien, do you know?
Reporter | ||
Comment 1•12 years ago
|
||
Probably goes without saying that scrolling up and down a little makes us repaint properly.
Reporter | ||
Comment 2•12 years ago
|
||
Trying from before bug 539356, candidate 1.
Reporter | ||
Comment 3•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ed614ea130c0 doesn't exhibit the bug.
Reporter | ||
Comment 4•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/22f6adf8685b is clean. Not bug 539356, yay.
Reporter | ||
Comment 5•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/4db58ba7848f exhibits the bug So initial regression range is http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=22f6adf8685b&tochange=4db58ba7848f
Reporter | ||
Comment 6•12 years ago
|
||
Sorry, not everything is merged http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=22f6adf8685b&tochange=4db58ba7848f
Reporter | ||
Comment 7•12 years ago
|
||
Awesome! $ hg bisect -g Due to skipped revisions, the first bad revision could be any of: changeset: 102444:a0bddf5fcb91 parent: 102443:b9b3b3373c08 parent: 102428:3b46b03dff5c user: Kyle Huey <khuey@kylehuey.com> date: Wed Aug 15 06:25:20 2012 -0700 summary: Merge m-c to b-s. changeset: 102445:d67c02074ced user: Kyle Huey <khuey@kylehuey.com> date: Wed Aug 15 06:52:06 2012 -0700 summary: Bug 780618: Follow up to switch to nsError.h in merged patches. r=me
Reporter | ||
Comment 8•12 years ago
|
||
khuey, (if the bisect isn't bogus,) was this a nontrivial merge by any chance?
Assignee | ||
Comment 9•12 years ago
|
||
The merge was straightforward ... I think the bisect took a wrong turn somewhere.
Reporter | ||
Comment 10•12 years ago
|
||
Narrower regression range http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=dad98b444955&tochange=62bb2e74fe67 Suspect bug 685516.
Assignee | ||
Comment 11•12 years ago
|
||
That's what I would suspect.
Reporter | ||
Comment 12•12 years ago
|
||
The first bad revision is: changeset: 102435:5c730c1f2274 user: Kyle Huey <khuey@kylehuey.com> date: Mon Aug 13 15:04:19 2012 -0700 summary: Bug 697230: Part 1 - Centralize style image observers. r=bz khuey, can you take a look?
Blocks: 697230
Reporter | ||
Comment 13•12 years ago
|
||
Note: testing with gaia rev b3153b347c7b9357f26c0584e676fee3ce5d8aa0
Assignee | ||
Comment 14•12 years ago
|
||
If your bisection is accurate, I suspect this is a dupe of bug 783379 (and unrelated to the tests I disabled).
Reporter | ||
Comment 15•12 years ago
|
||
Possibly. The sequence of events on opening the "settings app" is 1. create iframe 2. set iframe.src = 'app://settings' (approximately) 3. add to enclosing document 4. [do some unrelated animation] 5. call iframe.setVisible(), which ends up doing docShell.active = true 6. call iframe.focus() So in between (3) and (5) the iframe looks like an inactive "tab".
Assignee | ||
Comment 16•12 years ago
|
||
Ok. I'll take this and look into it after I fix bug 783162.
Assignee: nobody → khuey
Reporter | ||
Comment 17•12 years ago
|
||
Fixed by bug 783379.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•