Last Comment Bug 701260 - Deny requests for DOM full-screen when windowed plugins are present in the entire doc tree
: Deny requests for DOM full-screen when windowed plugins are present in the en...
Product: Core
Classification: Components
Component: DOM: Core & HTML (show other bugs)
: 10 Branch
: All All
-- normal (vote)
: mozilla10
Assigned To: Chris Pearce (:cpearce)
: Andrew Overholt [:overholt]
Depends on:
Blocks: 545812 684618
  Show dependency treegraph
Reported: 2011-11-09 18:13 PST by Chris Pearce (:cpearce)
Modified: 2012-02-01 13:00 PST (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Patch (2.36 KB, patch)
2011-11-15 17:15 PST, Chris Pearce (:cpearce)
roc: review+
christian: approval‑mozilla‑aurora+
Details | Diff | Splinter Review

Description User image Chris Pearce (:cpearce) 2011-11-09 18:13:17 PST
Currently the check which prevents us going full-screen when a doctree contains a windowed plugin only checks the document and its subdocuments, whereas it should check every document in the target document's branch (but not in other branches/tabs).
Comment 1 User image Chris Pearce (:cpearce) 2011-11-15 17:15:59 PST
Created attachment 574747 [details] [diff] [review]

Change to deny requests for full-screen if any doc in the tab has a windowed plugin, rather than if the requesting doc or its subdocs have windowed plugins.

I have tests, but the tests leak on tryserver (even without this patch applied!), so I'll not put the tests up for review.
Comment 3 User image Ed Morley [:emorley] 2011-11-16 03:09:40 PST
Comment 4 User image Chris Pearce (:cpearce) 2011-11-16 16:53:19 PST
Comment on attachment 574747 [details] [diff] [review]

Requesting approval for Aurora, because this fix prevents a work-around which enables keyboard input in full-screen mode which doesn't cause the "Press ESC to leave full-screen mode" warning to pop up. We need this to help protect against phishing attacks.
Comment 5 User image christian 2011-11-17 14:47:34 PST
Comment on attachment 574747 [details] [diff] [review]

[triage comment]
Approved for aurora. Please land as soon as possible
Comment 6 User image Chris Pearce (:cpearce) 2011-11-17 15:05:18 PST
Comment 7 User image Anthony Hughes (:ashughes) [GFX][QA][Mentor] 2011-12-28 14:10:37 PST
Is this something QA can verify?
Comment 8 User image Anthony Hughes (:ashughes) [GFX][QA][Mentor] 2012-02-01 12:58:17 PST
(In reply to Anthony Hughes, Mozilla QA (irc: ashughes) from comment #7)
> Is this something QA can verify?


Note You need to log in before you can comment on or make changes to this bug.