cc: zero/one-copy based on shared memory on IOS uses GL_TEXTURE_2D, not GL_TEXTURE_RECTANGLE_ARB.

content/renderer/render_thread_impl.cc

Index: content/renderer/render_thread_impl.cc
diff --git a/content/renderer/render_thread_impl.cc b/content/renderer/render_thread_impl.cc
index 7193aa71614ca3c1b1551d525a5dc4a5c3eb4a06..662a95e3587ecf6dedd9c0351793319fce438dd4 100644
--- a/content/renderer/render_thread_impl.cc
+++ b/content/renderer/render_thread_impl.cc
@@ -101,6 +101,7 @@
 #include "content/renderer/service_worker/embedded_worker_dispatcher.h"
 #include "content/renderer/shared_worker/embedded_shared_worker_stub.h"
 #include "gin/public/debug.h"
+#include "gpu/GLES2/gl2extchromium.h"
 #include "ipc/ipc_channel_handle.h"
 #include "ipc/ipc_platform_file.h"
 #include "ipc/mojo/ipc_channel_mojo.h"
@@ -520,7 +521,16 @@ void RenderThreadImpl::Init() {
   is_one_copy_enabled_ = !command_line.HasSwitch(switches::kDisableOneCopy);
 #endif
 
-  use_image_external_ = command_line.HasSwitch(switches::kUseImageExternal);
+  image_texture_target_ = GL_TEXTURE_2D;

[reveman, 2014/12/03 15:57:19]

do you need to set this here?

+  if (command_line.HasSwitch(switches::kImageTextureTarget)) {
+    std::string texture_target =
+        command_line.GetSwitchValueASCII(switches::kImageTextureTarget);
+    if (texture_target == "GL_TEXTURE_RECTANGLE_ARB") {

[reveman, 2014/12/03 15:57:19]

I prefer "rectangle" instead of "GL_TEXTURE_RECTANGLE_ARB".

+      image_texture_target_ = GL_TEXTURE_RECTANGLE_ARB;
+    } else if (texture_target == "GL_TEXTURE_EXTERNAL_OES") {

[reveman, 2014/12/03 15:57:19]

"external"

+      image_texture_target_ = GL_TEXTURE_EXTERNAL_OES;
+    }
+  }
 
   if (command_line.HasSwitch(switches::kDisableLCDText)) {
     is_lcd_text_enabled_ = false;