Removing getBackingTextureImage from ImageBufferSurface

Source/platform/graphics/ImageBufferSurface.h

Index: Source/platform/graphics/ImageBufferSurface.h
diff --git a/Source/platform/graphics/ImageBufferSurface.h b/Source/platform/graphics/ImageBufferSurface.h
index b6bbc276ceb00a6629efc6fbe70b0c7878e5ce66..e4ea84bc122d120d82b665649f4b8516167d69ea 100644
--- a/Source/platform/graphics/ImageBufferSurface.h
+++ b/Source/platform/graphics/ImageBufferSurface.h
@@ -82,11 +82,7 @@ public:
     virtual void draw(GraphicsContext*, const FloatRect& destRect, const FloatRect& srcRect, SkXfermode::Mode);
     virtual void setHasExpensiveOp() { }
 
-    // These methods return an SkImage representing the backing texture. We support both variants
-    // to avoid the snapshot overhead (flushing deferred draws, COW semantics, fallback in
-    // RecordingImageBufferSurface) when not strictly required.
     virtual PassRefPtr<SkImage> newImageSnapshot() const { return nullptr; }

[reed1, 2015/06/23 15:57:55]

Lets add some dox here, describing when/why it might return NULL.

are all clients of it ok to not ever force an answer (from the recording
variant)?

-    virtual PassRefPtr<SkImage> getBackingTextureImage() const { return nullptr; }
 
     OpacityMode opacityMode() const { return m_opacityMode; }
     const IntSize& size() const { return m_size; }