Mozart: Directly associate the renderer with the view tree.

Mozart: Directly associate the renderer with the view tree.

Instead of hooking up the view tree to the renderer indirectly
in the launcher application, give the Renderer interface to the
View Manager to take care of instead.

This change will allow the view manager to perform hit testing
on the scene graph.  It also simplifies the launcher somewhat.

BUG=
R=abarth@google.com

Committed: https://chromium.googlesource.com/external/mojo/+/e63ab215cdea2da68d02085c88afdf810ab1eacf

[jeffbrown, 2016-03-09 02:16:54]

jeffbrown@google.com changed reviewers:
+ abarth@google.com

[abarth, 2016-03-09 03:48:41]

lgtm

https://codereview.chromium.org/1775143004/diff/1/mojo/services/gfx/compositi...
File mojo/services/gfx/composition/interfaces/renderers.mojom (right):

https://codereview.chromium.org/1775143004/diff/1/mojo/services/gfx/compositi...
mojo/services/gfx/composition/interfaces/renderers.mojom:36: ResetRootScene();
ClearRootScene?  We've sort of vaguely been following the naming conventions in
the C++ standard library.

[jeffbrown, 2016-03-09 19:06:36]

On 2016/03/09 03:48:41, abarth wrote:
> lgtm
> 
>
https://codereview.chromium.org/1775143004/diff/1/mojo/services/gfx/compositi...
> File mojo/services/gfx/composition/interfaces/renderers.mojom (right):
> 
>
https://codereview.chromium.org/1775143004/diff/1/mojo/services/gfx/compositi...
> mojo/services/gfx/composition/interfaces/renderers.mojom:36: ResetRootScene();
> ClearRootScene?  We've sort of vaguely been following the naming conventions
in
> the C++ standard library.

I vacillated on this a little bit but I think you're right.  I'll fix this in a
follow-on patch.

[jeffbrown, 2016-03-09 22:22:21]

Description was changed from

==========
Mozart: Directly associate the renderer with the view tree.

Instead of hooking up the view tree to the renderer indirectly
in the launcher application, give the Renderer interface to the
View Manager to take care of instead.

This change will allow the view manager to perform hit testing
on the scene graph.  It also simplifies the launcher somewhat.

BUG=
==========

to

==========
Mozart: Directly associate the renderer with the view tree.

Instead of hooking up the view tree to the renderer indirectly
in the launcher application, give the Renderer interface to the
View Manager to take care of instead.

This change will allow the view manager to perform hit testing
on the scene graph.  It also simplifies the launcher somewhat.

BUG=
R=abarth@google.com

Committed:
https://chromium.googlesource.com/external/mojo/+/e63ab215cdea2da68d02085c88a...
==========

[jeffbrown, 2016-03-09 22:22:22]

Committed patchset #2 (id:20001) manually as
e63ab215cdea2da68d02085c88afdf810ab1eacf (presubmit successful).