Change glimage to accept a type.

android_webview/browser/gpu_memory_buffer_factory_impl.cc

Index: android_webview/browser/gpu_memory_buffer_factory_impl.cc
diff --git a/android_webview/browser/gpu_memory_buffer_factory_impl.cc b/android_webview/browser/gpu_memory_buffer_factory_impl.cc
index 620fd5abe8a54209a784ea3fa6dd4adf4ff97db5..80d9beb6cc1563b568cf29612143bf0b44982a0d 100644
--- a/android_webview/browser/gpu_memory_buffer_factory_impl.cc
+++ b/android_webview/browser/gpu_memory_buffer_factory_impl.cc
@@ -31,23 +31,9 @@ class GpuMemoryBufferImpl : public gfx::GpuMemoryBuffer {
   }
 
   // Overridden from gfx::GpuMemoryBuffer:
-  virtual void* Map(gfx::GpuMemoryBuffer::AccessMode mode) OVERRIDE {
-    AwMapMode map_mode = MAP_READ_ONLY;
-    switch (mode) {
-      case GpuMemoryBuffer::READ_ONLY:
-        map_mode = MAP_READ_ONLY;
-        break;
-      case GpuMemoryBuffer::WRITE_ONLY:
-        map_mode = MAP_WRITE_ONLY;
-        break;
-      case GpuMemoryBuffer::READ_WRITE:
-        map_mode = MAP_READ_WRITE;
-        break;
-      default:
-        LOG(DFATAL) << "Unknown map mode: " << mode;
-    }
+  virtual void* Map() OVERRIDE {
     void* vaddr = NULL;
-    int err = g_gl_draw_functions->map(buffer_id_, map_mode, &vaddr);
+    int err = g_gl_draw_functions->map(buffer_id_, MAP_READ_WRITE, &vaddr);

[alexst (slow to review), 2014/05/01 17:45:07]

This seems to be mapped into the function ptr table inside Android, isn't it?

[reveman, 2014/05/01 18:07:33]

Yes. FYI, we're planning to move this GpuMemoryBuffer implementation to 	
content/common/gpu/client where all other implementations live.

     DCHECK(!err);
     mapped_ = true;
     return vaddr;