Add initial GpuMemoryBufferSurfaceTexture implementation.

content/browser/renderer_host/compositor_impl_android.h

 // Copyright (c) 2012 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 CONTENT_BROWSER_RENDERER_HOST_COMPOSITOR_IMPL_ANDROID_H_
 #define CONTENT_BROWSER_RENDERER_HOST_COMPOSITOR_IMPL_ANDROID_H_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/containers/scoped_ptr_hash_map.h"
@@ skipping 31 matching lines @@
       public ui::WindowAndroidCompositor {
  public:
   CompositorImpl(CompositorClient* client, gfx::NativeWindow root_window);
   virtual ~CompositorImpl();
 
   static bool IsInitialized();
 
   // Returns the Java Surface object for a given view surface id.
   static jobject GetSurface(int surface_id);
 
+  // Returns the Java Surface object for a given surface texture.
+  static jobject GetSurfaceTextureSurface(int primary_id, int secondary_id);

[no sievers, 2014/03/24 19:26:08]

Why did you follow the bad example from the existing EstablishPeer code and
changed the variable names to be so generic? :)


Could it just be |surface_texture_id| and |child_process_id| or something like
that?

[reveman, 2014/03/25 01:31:23]

Changed it to an explicit base::ProcessHandle where I think is appropriate in
latest patch. I kept the detail that we use a process handle as part of the id
hidden from the renderer side and from the ui/gl/ code. I prefer if this is only
exposed to the code that depends on it. That's content/browser,
content/common/gpu/gpu_command_buffer_stub.cc and
content/public/android/java/src/org/chromium/content/browser.

Does that make sense?

+
+  // Allocates a surface texture and returns primary id. Returns -1 on failure.
+  static int AllocateSurfaceTexture(int secondary_id);
+
+  // Destroy all surface textures associated with |secondary_id|.
+  static void DestroyAllSurfaceTextures(int secondary_id);
+
   // Compositor implementation.
   virtual void SetRootLayer(scoped_refptr<cc::Layer> root) OVERRIDE;
   virtual void SetWindowSurface(ANativeWindow* window) OVERRIDE;
   virtual void SetSurface(jobject surface) OVERRIDE;
   virtual void SetVisible(bool visible) OVERRIDE;
   virtual void setDeviceScaleFactor(float factor) OVERRIDE;
   virtual void SetWindowBounds(const gfx::Size& size) OVERRIDE;
   virtual void SetHasTransparentBackground(bool flag) OVERRIDE;
   virtual bool CompositeAndReadback(
       void *pixels, const gfx::Rect& rect) OVERRIDE;
@@ skipping 59 matching lines @@
   UIResourceMap ui_resource_map_;
 
   gfx::NativeWindow root_window_;
 
   DISALLOW_COPY_AND_ASSIGN(CompositorImpl);
 };
 
 } // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_COMPOSITOR_IMPL_ANDROID_H_