Bug 1318342 - Remove the special casing for apps going fullscreen; r=baku
authorEhsan Akhgari <ehsan@mozilla.com>
Thu, 17 Nov 2016 10:18:36 -0500
changeset 323203 7cfb8d89771b49122fd69614dab0e1c063e9d200
parent 323202 eee080a43674dbd60de1347a4852721177d59192
child 323204 5a62d96f419d1298419ecc41768ddb6437914588
push id21
push usermaklebus@msu.edu
push dateThu, 01 Dec 2016 06:22:08 +0000
reviewersbaku
bugs1318342
milestone53.0a1
Bug 1318342 - Remove the special casing for apps going fullscreen; r=baku
dom/base/Element.cpp
--- a/dom/base/Element.cpp
+++ b/dom/base/Element.cpp
@@ -3308,24 +3308,16 @@ Element::AttrValueToCORSMode(const nsAtt
   }
 
   return CORSMode(aValue->GetEnumValue());
 }
 
 static const char*
 GetFullScreenError(nsIDocument* aDoc)
 {
-  if (aDoc->NodePrincipal()->GetAppStatus() >= nsIPrincipal::APP_STATUS_INSTALLED) {
-    // Request is in a web app and in the same origin as the web app.
-    // Don't enforce as strict security checks for web apps, the user
-    // is supposed to have trust in them. However documents cross-origin
-    // to the web app must still confirm to the normal security checks.
-    return nullptr;
-  }
-
   if (!nsContentUtils::IsRequestFullScreenAllowed()) {
     return "FullscreenDeniedNotInputDriven";
   }
 
   return nullptr;
 }
 
 void