Avoid GPU readback in tex(Sub)Image2D(ImageBitmap)

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

Index: third_party/WebKit/Source/platform/graphics/BitmapImage.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/BitmapImage.cpp b/third_party/WebKit/Source/platform/graphics/BitmapImage.cpp
index 2f3b925fa0913a5cab7cee801239036ab77c9431..59724bdecbfdfcc23385d175f3466b7d8da9c86a 100644
--- a/third_party/WebKit/Source/platform/graphics/BitmapImage.cpp
+++ b/third_party/WebKit/Source/platform/graphics/BitmapImage.cpp
@@ -342,7 +342,7 @@ float BitmapImage::frameDurationAtIndex(size_t index)
     return m_source.frameDurationAtIndex(index);
 }
 
-PassRefPtr<SkImage> BitmapImage::imageForCurrentFrame()
+PassRefPtr<SkImage> BitmapImage::imageForCurrentFrame(WebGraphicsContext3DProvider* contextProvider)
 {
     return frameAtIndex(currentFrame());

[Justin Novosad, 2016/06/07 15:11:07]

If this method is not going to look at contextProvider, it should at least check
that the SkImage is never texture backed.

[xidachen, 2016/06/07 17:49:53]

In my opinion, if this method is called in a test that is running under
virtual/gpu, then the SkImage can be texture-backed.

 }