Implement commit for unaccelerated canvas with GPU compositing

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

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef OffscreenCanvasFrameDispatcherImpl_h
 #define OffscreenCanvasFrameDispatcherImpl_h
 
 #include "cc/ipc/mojo_compositor_frame_sink.mojom-blink.h"
 #include "cc/resources/shared_bitmap.h"
 #include "cc/surfaces/surface_id.h"
@@ skipping 23 matching lines @@
   // cc::mojom::blink::MojoCompositorFrameSinkClient implementation.
   void ReturnResources(
       Vector<cc::mojom::blink::ReturnedResourcePtr> resources) override;
 
  private:
   const cc::SurfaceId m_surfaceId;
   const int m_width;
   const int m_height;
 
   unsigned m_nextResourceId;
-  unsigned getNextResourceIdAndIncrement() { return m_nextResourceId++; }
   HashMap<unsigned, RefPtr<StaticBitmapImage>> m_cachedImages;
   HashMap<unsigned, std::unique_ptr<cc::SharedBitmap>> m_sharedBitmaps;
+  HashMap<unsigned, GLuint> m_cachedTextureIds;
 
   bool verifyImageSize(const sk_sp<SkImage>&);
 
   cc::mojom::blink::MojoCompositorFrameSinkPtr m_sink;
   mojo::Binding<cc::mojom::blink::MojoCompositorFrameSinkClient> m_binding;
+
+  void setTransferableResourceCase1(cc::TransferableResource&,

[Justin Novosad, 2016/10/04 14:20:48]

These function names have readability issues. You could call them ...InMemory,
...InTexture and ...MemoryToTexture. Or something like that.

[xidachen, 2016/10/04 14:43:06]

Done.

+                                    RefPtr<StaticBitmapImage>);
+  void setTransferableResourceCase2(cc::TransferableResource&,
+                                    RefPtr<StaticBitmapImage>);
+  void setTransferableResourceCase3(cc::TransferableResource&,
+                                    RefPtr<StaticBitmapImage>);
 };
 
 }  // namespace blink
 
 #endif  // OffscreenCanvasFrameDispatcherImpl_h