[DO NOT COMMIT] Initial MediaPlayerRenderer plumbing and basic features

content/browser/frame_host/render_frame_host_impl.cc

Index: content/browser/frame_host/render_frame_host_impl.cc
diff --git a/content/browser/frame_host/render_frame_host_impl.cc b/content/browser/frame_host/render_frame_host_impl.cc
index 3855d7ee223446aa05e556519fe12961131e149a..9dc08e5da9591580d09a8fd5ff566b7b332a79bf 100644
--- a/content/browser/frame_host/render_frame_host_impl.cc
+++ b/content/browser/frame_host/render_frame_host_impl.cc
@@ -83,6 +83,9 @@
 #include "content/public/common/url_constants.h"
 #include "content/public/common/url_utils.h"
 #include "device/vibration/vibration_manager_impl.h"
+// TODO(tguilbert): For reviewers: Is it ok to about include \\media files in
+// this file?
+#include "media/media_features.h"

[xhwang, 2016/06/22 17:39:02]

this is fine. content/ already depends on media/

 #include "ui/accessibility/ax_tree.h"
 #include "ui/accessibility/ax_tree_update.h"
 #include "ui/gfx/geometry/quad_f.h"
@@ -92,6 +95,10 @@
 #include "content/browser/mojo/service_registrar_android.h"
 #endif
 
+#if BUILDFLAG(FORCE_MOJO_MEDIA_PLAYER_RENDERER)
+#include "media/mojo/services/mojo_media_player_renderer_helper.h"
+#endif
+
 #if defined(OS_MACOSX)
 #include "content/browser/frame_host/popup_menu_helper_mac.h"
 #endif
@@ -2051,6 +2058,15 @@ void RenderFrameHostImpl::RegisterMojoServices() {
   }
 #endif
 
+#if BUILDFLAG(FORCE_MOJO_MEDIA_PLAYER_RENDERER)
+  // This service creates a MojoRendererService, passing it a MediaPlayerRender.
+  // TODO(tguilbert): For reviewers: is there a need for disambiguating the
+  // mojom::Renderer service at a RenderFrameHost level?
+  GetServiceRegistry()->AddService<media::mojom::Renderer>(base::Bind(
+      &media::MojoMediaPlayerRendererHelper::CreateMediaPlayerRenderer,

[xhwang, 2016/06/22 17:39:02]

You don't really need a class to do this. You can probably just define a
CreateMediaPlayerRenderer() function, in this file, or in a new media/ file.

+      base::Unretained(this)));
+#endif
+
   GetContentClient()->browser()->RegisterRenderFrameMojoServices(
       GetServiceRegistry(), this);
 }