743217 - Do not allow iframes to lock the screen if a parent window's element is fullscreened

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Mounir Lamouri (:mounir)]

Attachment: Patch

This could be abused *and* that was causing a bug because the 'mozfullscreenchange' event is only sent to parent windows so leaving fullscreen wasn't unlocking the screen.

Comment 1

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 612864 [details] [diff] [review]
Patch

># HG changeset patch
># Parent da0d07b5ca1e79037eb66bad22bd2b3aeb106dc8
># User Mounir Lamouri <[e-mail]>
># Date 1333718342 -7200
>
>diff --git a/dom/base/nsScreen.cpp b/dom/base/nsScreen.cpp
>--- a/dom/base/nsScreen.cpp
>+++ b/dom/base/nsScreen.cpp
>@@ -367,18 +367,25 @@ nsScreen::MozLockOrientation(const nsASt
>   }
> 
>   if (!GetOwner()) {
>     *aReturn = false;
>     return NS_OK;
>   }
> 
>   if (!IsChromeType(GetOwner()->GetDocShell())) {
>+    nsCOMPtr<nsIDOMDocument> doc;
>+    GetOwner()->GetDocument(getter_AddRefs(doc));
>+    if (!doc) {
>+      *aReturn = false;
>+      return NS_OK;
>+    }
>+
>     bool fullscreen;
>-    GetOwner()->GetFullScreen(&fullscreen);
>+    doc->GetMozFullScreen(&fullscreen);
>     if (!fullscreen) {
>       *aReturn = false;
>       return NS_OK;
>     }
> 
>     nsCOMPtr<nsIDOMEventTarget> target = do_QueryInterface(GetOwner());
>     if (!target) {
>       *aReturn = false;

Comment 2

[(no longer active)]

https://hg.mozilla.org/mozilla-central/rev/ab8239e81ae4