Closed Bug 1215365 Opened 9 years ago Closed 8 years ago

Sync the ua style sheets of fullscreen element to the latest spec

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla46
Tracking Status
firefox44 --- affected
firefox46 --- fixed

People

(Reporter: xidorn, Assigned: xidorn)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

There has been several update on the default style sheet of fullscreen elements in the Fullscreen API spec, we should update our ua.css accordingly.

Note that we currently have "background: black" in our style sheet while the spec does not have. This line can be removed only when we implement bug 1064843.
Attached patch patchSplinter Review
Assignee: nobody → quanxunzhen
Attachment #8701399 - Flags: review?(cam)
Attachment #8701399 - Flags: review?(cam) → review+
It doesn't appear to be a permafail, but it did fail 8 out of 9 times so far.
I'm going to reland this patch without removing the z-index line. We would probably need more work to remove that line, especially on gaia side.

That line is useless for web content because all fullscreen elements from content other than :root are now rendered in the top layer, while it is not true for <iframe mozbrowser>.
Depends on: 1235470
The "background: black" line will be removed in bug 1064843 directly. So this bug no longer depends on that one.
No longer depends on: ::backdrop
As m-i has merged my patch for bug 1235470, reland this patch.
Flags: needinfo?(quanxunzhen)
https://hg.mozilla.org/mozilla-central/rev/bf2d8f46f02c
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
Blocks: 1240434
Blocks: 761099
You need to log in before you can comment on or make changes to this bug.