The default bug view has changed. See this FAQ.

Remove content-order check from SortByZOrder

RESOLVED FIXED in mozilla17

Status

()

Core
Layout
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: dholbert, Unassigned)

Tracking

Trunk
mozilla17
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Filing this bug on finishing off / landing patch 8 in bug 735898, to remove the secondary content-order sort when we're sorting display-lists by z-index.

For background, see bug 735898 comment 50 & beyond.
(Reporter)

Comment 1

5 years ago
Actually, it looks like bug 748646 is already filed as a followup on this.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 748646
(Reporter)

Comment 2

5 years ago
Actually, I shouldn't have duped this -- this bug isn't really a duplicate of bug 772690 -- it just depends on it.

That bugs' patches are now landed (on m-i), so the content-order-sort in SortByZOrder can now land without causing orange.

So, I'll land the SortByZOrder tweak in this bug. (simply using the changeset for part 8 from bug 735898, but with commit-message updated to point to this bug instead of that one)
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
(Reporter)

Comment 3

5 years ago
Created attachment 644938 [details] [diff] [review]
fix [r=dbaron granted in bug 735898 comment 47]

Here's the patch (review already granted in bug 735898 comment 47)
Attachment #644938 - Flags: review+
(Reporter)

Comment 4

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/a9053e89bab9
OS: Linux → All
Hardware: x86_64 → All
Target Milestone: --- → mozilla17
Blocks: 741682
https://hg.mozilla.org/mozilla-central/rev/a9053e89bab9
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED
(Reporter)

Updated

5 years ago
Blocks: 783474

Updated

4 years ago
Depends on: 867569
You need to log in before you can comment on or make changes to this bug.