Bug 1315585 - Don't assume mLastCompositionEndTime has been initialized in NotifyNotUsedAfterComposition. r=dvander
authorMatt Woodrow <mwoodrow@mozilla.com>
Tue, 08 Nov 2016 15:38:30 +1300
changeset 321461 e577673efe22f98618789a3ed2e0f0b83817fca4
parent 321460 72529f7a90dbd9f5ebf2b0d4ac2c02e31fb4f61a
child 321462 b2721e0ef0ef0fc3398e02d80293321bcd3ebe8f
push id83611
push usermwoodrow@mozilla.com
push dateTue, 08 Nov 2016 04:11:53 +0000
treeherdermozilla-inbound@e577673efe22 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdvander
bugs1315585
milestone52.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 1315585 - Don't assume mLastCompositionEndTime has been initialized in NotifyNotUsedAfterComposition. r=dvander
gfx/layers/Compositor.cpp
--- a/gfx/layers/Compositor.cpp
+++ b/gfx/layers/Compositor.cpp
@@ -82,17 +82,17 @@ Compositor::NotifyNotUsedAfterCompositio
 
   mNotifyNotUsedAfterComposition.AppendElement(aTextureHost);
 
   // If Compositor holds many TextureHosts without compositing,
   // the TextureHosts should be flushed to reduce memory consumption.
   const int thresholdCount = 5;
   const double thresholdSec = 2.0f;
   if (mNotifyNotUsedAfterComposition.Length() > thresholdCount) {
-    TimeDuration duration = TimeStamp::Now() - mLastCompositionEndTime;
+    TimeDuration duration = mLastCompositionEndTime ? TimeStamp::Now() - mLastCompositionEndTime : TimeDuration();
     // Check if we could flush
     if (duration.ToSeconds() > thresholdSec) {
       FlushPendingNotifyNotUsed();
     }
   }
 }
 
 void