Bug 1002992. Part 3. Don't build the caret when building a display list for image visibility. r=mats
authorTimothy Nikkel <tnikkel@gmail.com>
Fri, 06 Jun 2014 21:23:21 -0500
changeset 206585 a95c4223dba7aea9d6fb432c24a70dbcc4bce8af
parent 206584 1fabb6ff29cd1480a79b17c636a6642063d5a601
child 206586 3254fa7b1c512724674c1b428b69189b8c4289af
push id3741
push userasasaki@mozilla.com
push dateMon, 21 Jul 2014 20:25:18 +0000
treeherdermozilla-beta@4d6f46f5af68 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmats
bugs1002992
milestone32.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 1002992. Part 3. Don't build the caret when building a display list for image visibility. r=mats It's not needed and complicates things.
layout/base/nsPresShell.cpp
--- a/layout/base/nsPresShell.cpp
+++ b/layout/base/nsPresShell.cpp
@@ -5733,17 +5733,17 @@ PresShell::UpdateImageVisibility()
   nsIFrame* rootFrame = GetRootFrame();
   if (!rootFrame) {
     ClearVisibleImagesList();
     return;
   }
 
   // We could walk the frame tree directly and skip creating a display list for
   // better perf.
-  nsDisplayListBuilder builder(rootFrame, nsDisplayListBuilder::IMAGE_VISIBILITY, true);
+  nsDisplayListBuilder builder(rootFrame, nsDisplayListBuilder::IMAGE_VISIBILITY, false);
   nsRect updateRect(nsPoint(0, 0), rootFrame->GetSize());
   nsIFrame* rootScroll = GetRootScrollFrame();
   if (rootScroll) {
     nsIContent* content = rootScroll->GetContent();
     if (content) {
       nsLayoutUtils::GetDisplayPort(content, &updateRect);
     }