Closed
Bug 1191148
Opened 9 years ago
Closed 9 years ago
[e10s] Page may display things on fullscreen without triggering the warning prompt
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla42
People
(Reporter: xidorn, Assigned: xidorn)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
|
(deleted),
text/html
|
Details | |
|
(deleted),
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
Steps to reproduce:
1. Open the testcase
2. Click the black box
Expected result:
The browser doesn't enter fullscreen because the black box is removed immediately
Actual result:
The browser enters fullscreen but does not show the warning prompt
|
Assignee | |
Comment 1•9 years ago
|
||
Assignee: nobody → quanxunzhen
Attachment #8643406 -
Flags: review?(bugs)
|
||
Comment 2•9 years ago
|
||
Comment on attachment 8643406 [details] [diff] [review]
patch
Could we have a testcase for this to catch future regressions?
Do we need this on branches?
Attachment #8643406 -
Flags: review?(bugs) → review+
|
|
Assignee | |
Comment 3•9 years ago
|
||
I reasonably believe we have had such testcase... we just haven't enabled them on e10s...
|
|
Assignee | |
Comment 4•9 years ago
|
||
url: https://hg.mozilla.org/integration/mozilla-inbound/rev/dde1881bc8513bc5c81d3d5cdf09c76563b8b743
changeset: dde1881bc8513bc5c81d3d5cdf09c76563b8b743
user: Xidorn Quan <quanxunzhen@gmail.com>
date: Thu Aug 06 15:37:48 2015 +1000
description:
Bug 1191148 - Don't count fullscreen request handled if we don't change the document state. r=smaug
|
||
Updated•9 years ago
|
tracking-e10s:
--- → +
|
||
Comment 5•9 years ago
|
||
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
|
||
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•