Bug 1223747 - Don't assume all frames are visible in XUL documents. r=tn
authorSeth Fowler <mark.seth.fowler@gmail.com>
Mon, 04 Apr 2016 15:11:43 -0700
changeset 291560 5209d6b9fe6e0e39f1b4c46b0642aae3ece65973
parent 291559 df94a439a147929e467ac492c1ce44b387c48333
child 291561 f1ee7dcaa1cb0002a4074ccab583a8e458257fe4
push id74617
push usermfowler@mozilla.com
push dateMon, 04 Apr 2016 22:48:03 +0000
treeherdermozilla-inbound@f1ee7dcaa1cb [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstn
bugs1223747
milestone48.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1223747 - Don't assume all frames are visible in XUL documents. r=tn
layout/base/nsPresShell.cpp
--- a/layout/base/nsPresShell.cpp
+++ b/layout/base/nsPresShell.cpp
@@ -5998,23 +5998,22 @@ PresShell::AssumeAllFramesVisible()
       "layout.framevisibility.enabled", true);
     sFrameVisibilityPrefCached = true;
   }
 
   if (!sFrameVisibilityEnabled || !mPresContext || !mDocument) {
     return true;
   }
 
-  // We assume all frames are visible in print, print preview, chrome, xul, and
+  // We assume all frames are visible in print, print preview, chrome, and
   // resource docs and don't keep track of them.
   if (mPresContext->Type() == nsPresContext::eContext_PrintPreview ||
       mPresContext->Type() == nsPresContext::eContext_Print ||
       mPresContext->IsChrome() ||
-      mDocument->IsResourceDoc() ||
-      mDocument->IsXULDocument()) {
+      mDocument->IsResourceDoc()) {
     return true;
   }
 
   return false;
 }
 
 void
 PresShell::ScheduleApproximateFrameVisibilityUpdateSoon()