Bug 1317551. Make the main image drawing path ask for async notifications. r=aosmond
authorTimothy Nikkel <tnikkel@gmail.com>
Tue, 15 Nov 2016 11:28:11 -0600
changeset 322734 123e6164b536b9c4c873792dc1c5ed53a5951219
parent 322733 3ee3be133a5e9552eb9e4ba1baec86de0015330f
child 322735 63120fccd9a26e9118c6a9d1c99777f6f39fa0a8
push id30961
push userkwierso@gmail.com
push dateThu, 17 Nov 2016 01:08:03 +0000
treeherdermozilla-central@c27117f67fa3 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersaosmond
bugs1317551
milestone53.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 1317551. Make the main image drawing path ask for async notifications. r=aosmond We don't want sync notifications during painting.
layout/base/nsLayoutUtils.cpp
--- a/layout/base/nsLayoutUtils.cpp
+++ b/layout/base/nsLayoutUtils.cpp
@@ -6424,16 +6424,18 @@ DrawImageInternal(gfxContext&           
                   const nsPoint&         aAnchor,
                   const nsRect&          aDirty,
                   const SVGImageContext* aSVGContext,
                   uint32_t               aImageFlags,
                   ExtendMode             aExtendMode = ExtendMode::CLAMP)
 {
   DrawResult result = DrawResult::SUCCESS;
 
+  aImageFlags |= imgIContainer::FLAG_ASYNC_NOTIFY;
+
   if (aPresContext->Type() == nsPresContext::eContext_Print) {
     // We want vector images to be passed on as vector commands, not a raster
     // image.
     aImageFlags |= imgIContainer::FLAG_BYPASS_SURFACE_CACHE;
   }
   if (aDest.Contains(aFill)) {
     aImageFlags |= imgIContainer::FLAG_CLAMP;
   }