Use shared D3D9 texture to transport the compositor's backing buffer to the browser...

content/common/gpu/image_transport_surface.h

 // Copyright (c) 2011 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_COMMON_GPU_IMAGE_TRANSPORT_SURFACE_H_
 #define CONTENT_COMMON_GPU_IMAGE_TRANSPORT_SURFACE_H_
 #pragma once
 
 #if defined(ENABLE_GPU)
 
+#include "base/callback.h"
 #include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
 #include "ipc/ipc_channel.h"
 #include "ipc/ipc_message.h"
+#include "ui/gfx/gl/gl_surface.h"
 #include "ui/gfx/size.h"
 #include "ui/gfx/native_widget_types.h"
 #include "ui/gfx/surface/transport_dib.h"
 
 class GpuChannelManager;
 
 struct GpuHostMsg_AcceleratedSurfaceNew_Params;
 struct GpuHostMsg_AcceleratedSurfaceBuffersSwapped_Params;
 struct GpuHostMsg_AcceleratedSurfaceRelease_Params;
 
@@ skipping 17 matching lines @@
 // The ImageTransportSurface class defines an simple interface for events that
 // should be responded to. The factory returns an offscreen surface that looks
 // a lot like an onscreen surface to the GPU process.
 //
 // The ImageTransportSurfaceHelper provides some glue to the outside world:
 // making sure outside events reach the ImageTransportSurface and
 // allowing the ImageTransportSurface to send events to the outside world.
 
 class ImageTransportSurface {
  public:
+  ImageTransportSurface();
+  virtual ~ImageTransportSurface();

[jonathan.backer, 2011/11/03 23:09:07]

This was meant to be an interface so as not to break the style guidelines
regarding multiple inheritance. Why did you add these stubbed methods? For the
DISALLOW_COPY_AND_ASSIGN?

[apatrick_chromium, 2011/11/07 20:40:04]

The style guide actually requires that interface classes have a virtual
destructor and allows a no-arg constructor.

+
   virtual void OnNewSurfaceACK(
       uint64 surface_id, TransportDIB::Handle surface_handle) = 0;
   virtual void OnBuffersSwappedACK() = 0;
+  virtual void OnResizeViewACK() = 0;
   virtual void OnResize(gfx::Size size) = 0;
 
   // Creates the appropriate surface depending on the GL implementation.
   static scoped_refptr<gfx::GLSurface>
       CreateSurface(GpuChannelManager* manager,
                     int32 render_view_id,
                     int32 renderer_id,
                     int32 command_buffer_id,
                     gfx::PluginWindowHandle handle);
+ private:
+  DISALLOW_COPY_AND_ASSIGN(ImageTransportSurface);
 };
 
 class ImageTransportHelper : public IPC::Channel::Listener {
  public:
   // Takes weak pointers to objects that outlive the helper.
   ImageTransportHelper(ImageTransportSurface* surface,
                        GpuChannelManager* manager,
                        int32 render_view_id,
                        int32 renderer_id,
                        int32 command_buffer_id,
                        gfx::PluginWindowHandle handle);
   virtual ~ImageTransportHelper();
 
   bool Initialize();
   void Destroy();
 
   // IPC::Channel::Listener implementation:
   virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
 
   // Helper send functions. Caller fills in the surface specific params
   // like size and surface id. The helper fills in the rest.
   void SendAcceleratedSurfaceNew(
       GpuHostMsg_AcceleratedSurfaceNew_Params params);
   void SendAcceleratedSurfaceBuffersSwapped(
       GpuHostMsg_AcceleratedSurfaceBuffersSwapped_Params params);
   void SendAcceleratedSurfaceRelease(
       GpuHostMsg_AcceleratedSurfaceRelease_Params params);
+  void SendResizeView(const gfx::Size& size);
 
   // Whether or not we should execute more commands.
   void SetScheduled(bool is_scheduled);
 
+  void DeferToFence(base::Closure task);
+
   // Make the surface's context current.
   bool MakeCurrent();
 
  private:
   gpu::GpuScheduler* Scheduler();
   gpu::gles2::GLES2Decoder* Decoder();
 
   // IPC::Message handlers.
   void OnNewSurfaceACK(uint64 surface_id, TransportDIB::Handle surface_handle);
   void OnBuffersSwappedACK();
+  void OnResizeViewACK();
 
   // Backbuffer resize callback.
   void Resize(gfx::Size size);
 
+  // Set the default swap interval on the surface.
+  void SetSwapInterval();
+
   // Weak pointers that point to objects that outlive this helper.
   ImageTransportSurface* surface_;
   GpuChannelManager* manager_;
 
   int32 render_view_id_;
   int32 renderer_id_;
   int32 command_buffer_id_;
   int32 route_id_;
   gfx::PluginWindowHandle handle_;
 
   DISALLOW_COPY_AND_ASSIGN(ImageTransportHelper);
 };
 
+// An implementation of ImageTransportSurface that implements GLSurface through
+// GLSurfaceAdapter, thereby forwarding GLSurface methods through to it.
+class PassThroughImageTransportSurface
+    : public gfx::GLSurfaceAdapter,

[jonathan.backer, 2011/11/03 23:09:07]

I suspect that this is the only GLSurfaceAdapter that you need. Maybe merge the
two classes. You could also have it call CreateViewGLSurface directly.

[apatrick_chromium, 2011/11/07 20:40:04]

I added GLSurfaceAdapter to avoid making the inheritance any deeper than it
already is. Adapter is a well-known pattern to avoid that. I was thinking of
going in the direction of having GLSurface implementations that derive only from
GLSurfaceAdapter and then chain them together at runtime. I think I can get rid
of a fair amount of redundant code in gfx/ui/gl that way.

+      public ImageTransportSurface {
+ public:
+  PassThroughImageTransportSurface(GpuChannelManager* manager,
+                                   int32 render_view_id,
+                                   int32 renderer_id,
+                                   int32 command_buffer_id,
+                                   gfx::GLSurface* surface);
+  virtual ~PassThroughImageTransportSurface();
+
+  // GLSurface implementation.
+  virtual bool Initialize();
+  virtual void Destroy();
+
+  // ImageTransportSurface implementation.
+  virtual void OnNewSurfaceACK(
+      uint64 surface_id, TransportDIB::Handle surface_handle) OVERRIDE;
+  virtual void OnBuffersSwappedACK() OVERRIDE;
+  virtual void OnResizeViewACK() OVERRIDE;
+  virtual void OnResize(gfx::Size size) OVERRIDE;
+
+ private:
+  scoped_ptr<ImageTransportHelper> helper_;
+
+  DISALLOW_COPY_AND_ASSIGN(PassThroughImageTransportSurface);
+};
+
 #endif  // defined(ENABLE_GPU)
 
 #endif  // CONTENT_COMMON_GPU_IMAGE_TRANSPORT_SURFACE_H_