Closed
Bug 739671
Opened 12 years ago
Closed 12 years ago
Order of ColorLayers is reversed every paint
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla17
People
(Reporter: mattwoodrow, Assigned: mattwoodrow)
References
Details
Attachments
(1 file, 1 obsolete file)
|
9.36 KB,
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
When we have multiple ColorLayers within a single container, their order is reversed every paint. I believe this is because we process ThebesLayers in a stack, so we should be retrieving the ColorLayers (and ImageLayers) in reverse order.
Attachment #609753 -
Flags: review?(roc)
Attachment #609753 -
Flags: review?(roc) → review+
|
Assignee | |
Comment 1•12 years ago
|
||
Turns out that the original approach didn't actually work. The ordering isn't consistent at all, and reversing the order broke as many tests as it fixed.
Attachment #609753 -
Attachment is obsolete: true
Attachment #618538 -
Flags: review?(roc)
|
|
Assignee | |
Comment 2•12 years ago
|
||
CC'ing nrc since this may affect his MaskLayer patches
Comment on attachment 618538 [details] [diff] [review] Attach ImageLayers to the ThebesLayer it replaces Review of attachment 618538 [details] [diff] [review]: ----------------------------------------------------------------- Good plan
Attachment #618538 -
Flags: review?(roc) → review+
|
|
Assignee | |
Comment 4•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/d80219c8842c
Assignee: nobody → matt.woodrow
Target Milestone: --- → mozilla15
Version: unspecified → Trunk
|
||
Comment 5•12 years ago
|
||
Ok, so this is pretty sad faces, but I've had to back this out for causing mochitest-4 permaorange (as well as the talos regressions in comment 237): https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=61fd66629c4f eg https://tbpl.mozilla.org/php/getParsedLog.php?id=12544443&tree=Mozilla-Inbound https://hg.mozilla.org/integration/mozilla-inbound/rev/f08886a8cf22 It's for times like these that a relanding script would be pretty useful, for automating the qimport of a lange range of changesets (or at least it would make me feel less bad about having to back things like this out). Sorry! :-(
|
|
||
Updated•12 years ago
|
Target Milestone: mozilla15 → ---
I presume comment #5 was for bug 539356.
|
|
Assignee | |
Comment 7•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/90ab708bab8b
|
||
Comment 8•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/90ab708bab8b
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
|
|
Assignee | |
Comment 9•12 years ago
|
||
This got backed out with the rest of 539356 initially. https://hg.mozilla.org/integration/mozilla-inbound/rev/bc76f965c041
Assignee: matt.woodrow → nobody
Status: RESOLVED → REOPENED
Component: Layout → Keyboard: Navigation
Resolution: FIXED → ---
Target Milestone: mozilla16 → mozilla15
Version: Trunk → Other Branch
|
||
Updated•12 years ago
|
Assignee: nobody → matt.woodrow
Component: Keyboard: Navigation → Layout
OS: Mac OS X → All
Hardware: x86 → All
Target Milestone: mozilla15 → ---
Version: Other Branch → Trunk
|
|
||
Comment 10•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/bc76f965c041
Status: REOPENED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in
before you can comment on or make changes to this bug.
Description
•