Allow WebExternalTextureLayers to receive a bitmap along with a mailbox

webkit/renderer/compositor_bindings/web_external_bitmap_impl.h

Index: webkit/renderer/compositor_bindings/web_external_bitmap_impl.h
diff --git a/webkit/renderer/compositor_bindings/web_external_bitmap_impl.h b/webkit/renderer/compositor_bindings/web_external_bitmap_impl.h
new file mode 100644
index 0000000000000000000000000000000000000000..ca5427dd149458356b2bacaa319148e0504002fa
--- /dev/null
+++ b/webkit/renderer/compositor_bindings/web_external_bitmap_impl.h
@@ -0,0 +1,47 @@
+// Copyright 2013 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 WEBKIT_RENDERER_COMPOSITOR_BINDINGS_WEB_EXTERNAL_BITMAP_IMPL_H_
+#define WEBKIT_RENDERER_COMPOSITOR_BINDINGS_WEB_EXTERNAL_BITMAP_IMPL_H_
+
+#include "base/bind.h"
+#include "base/memory/scoped_ptr.h"
+#include "third_party/WebKit/public/platform/WebExternalBitmap.h"
+#include "webkit/renderer/compositor_bindings/webkit_compositor_bindings_export.h"
+
+namespace base {
+class SharedMemory;
+}
+
+namespace webkit {
+
+typedef scoped_ptr<base::SharedMemory> (*SharedMemoryAllocationFunction)(
+    size_t);
+
+// Sets the function that this will use to allocate shared memory.
+WEBKIT_COMPOSITOR_BINDINGS_EXPORT void SetSharedMemoryAllocationFunction(
+    SharedMemoryAllocationFunction);
+
+class WebExternalBitmapImpl : public WebKit::WebExternalBitmap {
+ public:
+  WEBKIT_COMPOSITOR_BINDINGS_EXPORT explicit WebExternalBitmapImpl();
+  virtual ~WebExternalBitmapImpl();
+
+  // WebKit::WebExternalBitmap implementation.
+  virtual WebKit::WebSize size() OVERRIDE;
+  virtual void setSize(WebKit::WebSize size) OVERRIDE;
+  virtual uint8* pixels() OVERRIDE;
+
+  base::SharedMemory* getSharedMemory() { return shared_memory_.get(); }

[jamesr, 2013/07/04 00:32:08]

this is chromium code, so it should follow chromium naming conventions. since
it's an inline trivial getter i think the best name is 'shared_memory()' and it
should be const. if it had to be out-of-line or whatever then
'GetSharedMemory()' would be a good name

+
+ private:
+  scoped_ptr<base::SharedMemory> shared_memory_;
+  WebKit::WebSize size_;
+
+  DISALLOW_COPY_AND_ASSIGN(WebExternalBitmapImpl);
+};
+
+}  // namespace webkit
+
+#endif  // WEBKIT_RENDERER_COMPOSITOR_BINDINGS_WEB_EXTERNAL_BITMAP_IMPL_H_