Implement WebGL's commit on the main thread

third_party/WebKit/Source/platform/graphics/StaticBitmapImage.h

Index: third_party/WebKit/Source/platform/graphics/StaticBitmapImage.h
diff --git a/third_party/WebKit/Source/platform/graphics/StaticBitmapImage.h b/third_party/WebKit/Source/platform/graphics/StaticBitmapImage.h
index 2c2a6d26259b2c3ee9f26d91898c47820cc252b7..1f85dd885448f6c6d65d080e70106501e3475f0c 100644
--- a/third_party/WebKit/Source/platform/graphics/StaticBitmapImage.h
+++ b/third_party/WebKit/Source/platform/graphics/StaticBitmapImage.h
@@ -5,8 +5,11 @@
 #ifndef StaticBitmapImage_h
 #define StaticBitmapImage_h
 
+#include "gpu/command_buffer/common/mailbox.h"
+#include "gpu/command_buffer/common/sync_token.h"
 #include "platform/graphics/Image.h"
 #include "third_party/khronos/GLES2/gl2.h"
+#include "third_party/skia/include/core/SkImage.h"

[Justin Novosad, 2016/09/14 14:28:13]

Why does this need to be included in the .h?

[xidachen, 2016/09/14 14:53:51]

I tried to remove this line and put it in .cpp, then I got a access to
incomplete type compile error. I think it is because the destructor of
StaticBitmapImage needs to delete the sk_sp for SkImage, so it needs to include
i here.

[Justin Novosad, 2016/09/14 17:56:43]

Then you should define the destructor in the .cpp file

 #include "third_party/skia/include/core/SkRefCnt.h"
 
 namespace blink {
@@ -36,6 +39,10 @@ public:
     virtual bool hasMailbox() { return false; }
     virtual void transfer() { }
 
+    virtual gpu::Mailbox getMailbox() { return gpu::Mailbox(); }
+    virtual gpu::SyncToken getSyncToken() { return gpu::SyncToken(); }
+    virtual void ensureMailbox() {}
+
 protected:
     StaticBitmapImage(sk_sp<SkImage>);
     StaticBitmapImage() { } // empty constructor for derived class.