Refactor ImageDecoder factories

third_party/WebKit/Source/platform/graphics/BitmapImage.cpp

 /*
  * Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2004, 2005, 2006, 2008 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 180 matching lines @@
 
 Image::SizeAvailability BitmapImage::setData(PassRefPtr<SharedBuffer> data, bool allDataReceived)
 {
     if (!data.get())
         return SizeAvailable;
 
     int length = data->size();
     if (!length)
         return SizeAvailable;
 
-    // If ImageSource::setData() returns Invalid, we know that this is a decode error.
+    // If ImageSource::setData() fails, we know that this is a decode error.
     // Report size available so that it gets registered as such in ImageResource.
-    if (m_source.setData(*data, allDataReceived) == ImageDecoder::SniffResult::Invalid)
+    if (!m_source.setData(data, allDataReceived))
         return SizeAvailable;
 
     return dataChanged(allDataReceived);
 }
 
 Image::SizeAvailability BitmapImage::dataChanged(bool allDataReceived)
 {
     TRACE_EVENT0("blink", "BitmapImage::dataChanged");
 
     // Clear all partially-decoded frames. For most image formats, there is only
@@ skipping 403 matching lines @@
 
     // We need to draw this frame if we advanced to it while not skipping, or if
     // while trying to skip frames we hit the last frame and thus had to stop.
     if (skippingFrames != advancedAnimation)
         getImageObserver()->animationAdvanced(this);
 
     return advancedAnimation;
 }
 
 } // namespace blink