Backed out changeset 2ac62de7365d (bug 1271002) for crash in GTest: [@ mozilla::image::RasterImage::UpdateImageContainer]. r=backout

image/DecodePool.cpp

@@ -469,7 +469,6 @@ void
 DecodePool::NotifyProgress(Decoder* aDecoder)
 {
   MOZ_ASSERT(aDecoder);
-  MOZ_ASSERT(aDecoder->HasProgress() && !aDecoder->IsMetadataDecode());
 
   if (!NS_IsMainThread() ||
       (aDecoder->GetDecoderFlags() & DecoderFlags::ASYNC_NOTIFY)) {

image/DecodePool.h

@@ -79,11 +79,6 @@ public:
    */
   already_AddRefed<nsIEventTarget> GetIOEventTarget();
 
-  /**
-   * Notify about progress on aDecoder.
-   */
-  void NotifyProgress(Decoder* aDecoder);
-
 private:
   friend class DecodePoolWorker;
 
@@ -92,6 +87,7 @@ private:
 
   void Decode(Decoder* aDecoder);
   void NotifyDecodeComplete(Decoder* aDecoder);
+  void NotifyProgress(Decoder* aDecoder);
 
   static StaticRefPtr<DecodePool> sSingleton;
   static uint32_t sNumCores;

image/Decoder.cpp

@@ -449,12 +449,6 @@ Decoder::PostFrameStop(Opacity aFrameOpacity
     mInvalidRect.UnionRect(mInvalidRect,
                            gfx::IntRect(gfx::IntPoint(0, 0), GetSize()));
   }
-
-  // If we are going to keep decoding we should notify now about the first frame being done.
-  if (mFrameCount == 1 && HasAnimation()) {
-    MOZ_ASSERT(HasProgress());
-    DecodePool::Singleton()->NotifyProgress(this);
-  }
 }
 
 void