Disable partial swaps for webview guest renderer until we can figure out how to do that properly.

ui/gfx/native_widget_types.h

Index: ui/gfx/native_widget_types.h
diff --git a/ui/gfx/native_widget_types.h b/ui/gfx/native_widget_types.h
index b187d57f1e890a0427cf8177fe38b7696df1ea47..54d382629f00a41f3132275e1554d51770f886a2 100644
--- a/ui/gfx/native_widget_types.h
+++ b/ui/gfx/native_widget_types.h
@@ -271,7 +271,8 @@ enum SurfaceType {
   EMPTY,
   NATIVE_DIRECT,
   NATIVE_TRANSPORT,
-  TEXTURE_TRANSPORT
+  TEXTURE_TRANSPORT,
+  TEXTURE_TRANSPORT_NO_PARTIAL_SWAP

[piman, 2013/03/05 20:32:37]

I'm really uneasy with passing feature flags through this type enum. How about
disabling partial swaps in the guest, by changing the LayerTreeSettings in
RenderWidgetCompositor::Create ?

[alexst (slow to review), 2013/03/05 21:09:28]

I'll need to go do some digging to find a good way to do that. I've written via
SurfaceType because it's one place I know something is a guest.

Currently a renderer itself doesn't know it's a webview, so I'll need to plumb
that information through, somehow.

I'll need a flag along the lines of do_partial_swap_but_render_everything_anyway
if we want to forward the damage rect to the embedder.

 };
 
 struct GLSurfaceHandle {
@@ -288,12 +289,14 @@ struct GLSurfaceHandle {
         parent_client_id(0) {
     DCHECK(!is_null() || handle == kNullPluginWindow);
     DCHECK(transport_type != TEXTURE_TRANSPORT ||
+           transport_type != TEXTURE_TRANSPORT_NO_PARTIAL_SWAP ||
            handle == kNullPluginWindow);
   }
   bool is_null() const { return transport_type == EMPTY; }
   bool is_transport() const {
     return transport_type == NATIVE_TRANSPORT ||
-           transport_type == TEXTURE_TRANSPORT;
+           transport_type == TEXTURE_TRANSPORT ||
+           transport_type == TEXTURE_TRANSPORT_NO_PARTIAL_SWAP;
   }
   PluginWindowHandle handle;
   SurfaceType transport_type;