android_webview: changes to support Android GraphicBuffers

ui/gl/pixel_buffer.h

Index: ui/gl/pixel_buffer.h
diff --git a/ui/gl/pixel_buffer.h b/ui/gl/pixel_buffer.h
new file mode 100644
index 0000000000000000000000000000000000000000..25e4f620a68e391742915a19f61eb77140a06da5
--- /dev/null
+++ b/ui/gl/pixel_buffer.h
@@ -0,0 +1,54 @@
+// 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 UI_GL_PIXEL_BUFFER_H_
+#define UI_GL_PIXEL_BUFFER_H_
+
+#include "base/basictypes.h"
+#include "base/callback.h"
+#include "base/memory/scoped_ptr.h"
+
+namespace gfx {
+class Size;
+
+// Interface for creating and accessing a zero-copy buffer such as GraphicBuffer
+// of Android framework.
+//
+// THREADING CONSIDERATIONS:
+//
+// It is legal for several different threads to map a buffer from
+// read access, none of the threads are blocked.
+// However, mapping a buffer simultaneously for write or read/write is
+// undefined, but:
+// - shall not result in termination of the process
+// - shall not block the caller
+// It is acceptable to return an error or to leave the buffer's content
+// into an indeterminate state.
+class PixelBuffer {
+ public:
+  // The factory is set by the ContentRendererClient::GetPixelBufferFactory()

[jamesr, 2013/04/04 21:05:27]

I don't think it's very useful to put a comment in a lower-level component like
ui/gl/ about what a higher-level component does.  ui/gl/ should make sense
without depending on behaviors of content.

[kaanb, 2013/04/04 22:26:18]

Removed.

+  // method.
+  typedef base::Callback<scoped_ptr<gfx::PixelBuffer>(gfx::Size)> Create;
+
+  virtual ~PixelBuffer() {}
+
+  // Maps the buffer so the client can write the bitmap data in |*vaddr|
+  // subsequently. This call may block, for instance if the h/w needs
+  // to finish rendering or if CPU caches need to be synchronized.
+  virtual void MapForWrite(void** vaddr) = 0;
+
+  // Unmaps the buffer. Called after all changes to the buffer are
+  // completed.
+  virtual void Unmap() = 0;
+
+  // Returns the native pointer for the pixel buffer.
+  virtual void* GetNativeBuffer() = 0;
+
+  // Returns the stride for the pixel buffer.
+  virtual uint32 GetStride() = 0;
+};
+
+}  // namespace gfx
+
+#endif  // UI_GL_PIXEL_BUFFER_H_