Mandoline: Support transforms and clipping of OOPIFs and events

components/mus/connection_manager.cc

Index: components/mus/connection_manager.cc
diff --git a/components/mus/connection_manager.cc b/components/mus/connection_manager.cc
index 2a86f1070e5967409e5241dbec47bcec9b112da6..dcd572995e91a20e090b84111131966dc8c6f98b 100644
--- a/components/mus/connection_manager.cc
+++ b/components/mus/connection_manager.cc
@@ -8,6 +8,7 @@
 #include "base/stl_util.h"
 #include "cc/output/compositor_frame.h"
 #include "cc/quads/shared_quad_state.h"
+#include "cc/quads/surface_draw_quad.h"
 #include "components/mus/client_connection.h"
 #include "components/mus/connection_manager_delegate.h"
 #include "components/mus/server_view.h"
@@ -445,18 +446,18 @@ bool ConnectionManager::ConvertSurfaceDrawQuad(
   ServerView* view = GetView(ViewIdFromTransportId(id));
   // If a CompositorFrame message arrives late, say during a navigation, then
   // it may contain view IDs that no longer exist.
-  if (!view)
+  if (!view || view->surface_id().is_null())
     return false;
-  gfx::Rect bounds(input->visible_rect.To<gfx::Rect>());
-  gfx::Point p;
-  sqs->quad_to_target_transform.TransformPoint(&p);
-  bounds.set_origin(p);
-  // TODO(fsamuel): This seems like a crude way to set the size that probably
-  // doesn't work correctly in the general case. We need to get transforms
-  // working correctly in the general case.
-  bounds.set_size(gfx::ToRoundedSize(
-      gfx::ScaleSize(bounds.size(), metadata->device_scale_factor)));
-  view->SetBounds(bounds);
+
+  cc::SurfaceDrawQuad* surface_quad =
+      render_pass->CreateAndAppendDrawQuad<cc::SurfaceDrawQuad>();

[rjkroege, 2015/09/16 21:31:37]

where do we validate the contents of the provided CompositorFrame for malicious
payload? please put a TODO there.

why is connection_mangement doing graphics work? this seems like a failure to
separate concerns (more TODOs probably)

do you have a block-diagram of mus? I've seen fragments of one. Can you put one
together that's more detailed?

[Fady Samuel, 2015/09/17 18:31:42]

There's a TODO above.

Added a todo and filed a bug for moving this out of connectionmanager.

I'll write a block diagram for mus. I've added a doc to your phase 1 doc with a
partial diagram.

+  surface_quad->SetAll(
+      sqs,
+      input->rect.To<gfx::Rect>(),
+      input->opaque_rect.To<gfx::Rect>(),
+      input->visible_rect.To<gfx::Rect>(),
+      input->needs_blending,
+      view->surface_id());
   return true;
 }