Take NativeImageSkia out behind the woodshed.

Source/platform/graphics/BitmapImage.cpp

Index: Source/platform/graphics/BitmapImage.cpp
diff --git a/Source/platform/graphics/BitmapImage.cpp b/Source/platform/graphics/BitmapImage.cpp
index 6bbae0ebc0c863009937184ebce0b4b693fd9da7..bf124793b97e13ece7de1ec2f11fee820b1b4d00 100644
--- a/Source/platform/graphics/BitmapImage.cpp
+++ b/Source/platform/graphics/BitmapImage.cpp
@@ -32,25 +32,24 @@
 #include "platform/geometry/FloatRect.h"
 #include "platform/graphics/GraphicsContextStateSaver.h"
 #include "platform/graphics/ImageObserver.h"
-#include "platform/graphics/skia/NativeImageSkia.h"
 #include "platform/graphics/skia/SkiaUtils.h"
 #include "wtf/PassRefPtr.h"
 #include "wtf/text/WTFString.h"
 
 namespace blink {
 
-PassRefPtr<BitmapImage> BitmapImage::create(PassRefPtr<NativeImageSkia> nativeImage, ImageObserver* observer)
+PassRefPtr<BitmapImage> BitmapImage::create(const SkBitmap& bitmap, ImageObserver* observer)
 {
-    if (!nativeImage) {
+    if (bitmap.isNull()) {
         return BitmapImage::create(observer);
     }
 
-    return adoptRef(new BitmapImage(nativeImage, observer));
+    return adoptRef(new BitmapImage(bitmap, observer));
 }
 
-PassRefPtr<BitmapImage> BitmapImage::createWithOrientationForTesting(PassRefPtr<NativeImageSkia> nativeImage, ImageOrientation orientation)
+PassRefPtr<BitmapImage> BitmapImage::createWithOrientationForTesting(const SkBitmap& bitmap, ImageOrientation orientation)
 {
-    RefPtr<BitmapImage> result = create(nativeImage);
+    RefPtr<BitmapImage> result = create(bitmap);
     result->m_frames[0].m_orientation = orientation;
     if (orientation.usesWidthAsHeight())
         result->m_sizeRespectingOrientation = IntSize(result->m_size.height(), result->m_size.width());
@@ -79,9 +78,9 @@ BitmapImage::BitmapImage(ImageObserver* observer)
 {
 }
 
-BitmapImage::BitmapImage(PassRefPtr<NativeImageSkia> nativeImage, ImageObserver* observer)
+BitmapImage::BitmapImage(const SkBitmap& bitmap, ImageObserver* observer)
     : Image(observer)
-    , m_size(nativeImage->bitmap().width(), nativeImage->bitmap().height())
+    , m_size(bitmap.width(), bitmap.height())
     , m_currentFrame(0)
     , m_frames(0)
     , m_frameTimer(0)
@@ -103,8 +102,8 @@ BitmapImage::BitmapImage(PassRefPtr<NativeImageSkia> nativeImage, ImageObserver*
     m_sizeRespectingOrientation = m_size;
 
     m_frames.grow(1);
-    m_frames[0].m_hasAlpha = !nativeImage->bitmap().isOpaque();
-    m_frames[0].m_frame = nativeImage;
+    m_frames[0].m_hasAlpha = !bitmap.isOpaque();
+    m_frames[0].m_frame = bitmap;
     m_frames[0].m_haveMetadata = true;
 
     checkForSolidColor();
@@ -167,7 +166,7 @@ void BitmapImage::cacheFrame(size_t index)
         m_frames.grow(numFrames);
 
     m_frames[index].m_frame = m_source.createFrameAtIndex(index);
-    if (numFrames == 1 && m_frames[index].m_frame)
+    if (numFrames == 1 && !m_frames[index].m_frame.isNull())
         checkForSolidColor();
 
     m_frames[index].m_orientation = m_source.orientationAtIndex(index);
@@ -182,7 +181,7 @@ void BitmapImage::cacheFrame(size_t index)
     if (frameSize != m_size)
         m_hasUniformFrameSize = false;
 
-    if (m_frames[index].m_frame) {
+    if (!m_frames[index].m_frame.isNull()) {
         int deltaBytes = safeCast<int>(m_frames[index].m_frameBytes);
         // The fully-decoded frame will subsume the partially decoded data used
         // to determine image properties.
@@ -272,13 +271,13 @@ String BitmapImage::filenameExtension() const
 
 void BitmapImage::draw(GraphicsContext* ctxt, const FloatRect& dstRect, const FloatRect& srcRect, SkXfermode::Mode compositeOp, RespectImageOrientationEnum shouldRespectImageOrientation)
 {
-    RefPtr<NativeImageSkia> image = nativeImageForCurrentFrame();
-    if (!image)
+    SkBitmap bitmap = bitmapForCurrentFrame();
+    if (bitmap.isNull())
         return; // It's too early and we don't have an image yet.
 
     FloatRect normDstRect = adjustForNegativeSize(dstRect);
     FloatRect normSrcRect = adjustForNegativeSize(srcRect);
-    normSrcRect.intersect(FloatRect(0, 0, image->bitmap().width(), image->bitmap().height()));
+    normSrcRect.intersect(FloatRect(0, 0, bitmap.width(), bitmap.height()));
 
     if (normSrcRect.isEmpty() || normDstRect.isEmpty())
         return; // Nothing to draw.
@@ -304,7 +303,7 @@ void BitmapImage::draw(GraphicsContext* ctxt, const FloatRect& dstRect, const Fl
         }
     }
 
-    image->draw(ctxt, normSrcRect, normDstRect, compositeOp);
+    drawBitmap(ctxt, bitmap, normSrcRect, normDstRect, compositeOp);
 
     if (ImageObserver* observer = imageObserver())
         observer->didDraw(this);
@@ -339,16 +338,16 @@ bool BitmapImage::ensureFrameIsCached(size_t index)
     if (index >= frameCount())
         return false;
 
-    if (index >= m_frames.size() || !m_frames[index].m_frame)
+    if (index >= m_frames.size() || m_frames[index].m_frame.isNull())
         cacheFrame(index);
 
     return true;
 }
 
-PassRefPtr<NativeImageSkia> BitmapImage::frameAtIndex(size_t index)
+SkBitmap BitmapImage::frameAtIndex(size_t index)
 {
     if (!ensureFrameIsCached(index))
-        return nullptr;
+        return SkBitmap();
 
     return m_frames[index].m_frame;
 }
@@ -369,14 +368,14 @@ float BitmapImage::frameDurationAtIndex(size_t index)
     return m_source.frameDurationAtIndex(index);
 }
 
-PassRefPtr<NativeImageSkia> BitmapImage::nativeImageForCurrentFrame()
+SkBitmap BitmapImage::bitmapForCurrentFrame()
 {
     return frameAtIndex(currentFrame());
 }
 
 PassRefPtr<Image> BitmapImage::imageForDefaultFrame()
 {
-    if (frameCount() > 1 && frameAtIndex(0))
+    if (frameCount() > 1 && !frameAtIndex(0).isNull())
         return BitmapImage::create(frameAtIndex(0));
 
     return Image::imageForDefaultFrame();
@@ -624,15 +623,15 @@ void BitmapImage::checkForSolidColor()
     if (frameCount() > 1)
         return;
 
-    RefPtr<NativeImageSkia> frame = frameAtIndex(0);
+    SkBitmap bitmap = frameAtIndex(0);
 
-    if (frame && size().width() == 1 && size().height() == 1) {
-        SkAutoLockPixels lock(frame->bitmap());
-        if (!frame->bitmap().getPixels())
+    if (!bitmap.isNull() && size().width() == 1 && size().height() == 1) {
+        SkAutoLockPixels lock(bitmap);
+        if (!bitmap.getPixels())
             return;
 
         m_isSolidColor = true;
-        m_solidColor = Color(frame->bitmap().getColor(0, 0));
+        m_solidColor = Color(bitmap.getColor(0, 0));
     }
 }