Enables fullscreen subtitle and media control from Blink

content/browser/renderer_host/render_widget_host_view_android.cc

Index: content/browser/renderer_host/render_widget_host_view_android.cc
diff --git a/content/browser/renderer_host/render_widget_host_view_android.cc b/content/browser/renderer_host/render_widget_host_view_android.cc
index 421d2f0fa998efaac3d6d5f69177a8c51401cb4f..48b80840120a60963642daaee23dbeaeef4987a0 100644
--- a/content/browser/renderer_host/render_widget_host_view_android.cc
+++ b/content/browser/renderer_host/render_widget_host_view_android.cc
@@ -115,7 +115,8 @@ RenderWidgetHostViewAndroid::RenderWidgetHostViewAndroid(
       current_mailbox_output_surface_id_(kUndefinedOutputSurfaceId),
       weak_ptr_factory_(this),
       overscroll_effect_enabled_(true),
-      flush_input_requested_(false) {
+      flush_input_requested_(false),
+      overlay_video_mode_(false) {
   if (CompositorImpl::UsesDirectGL()) {
     surface_texture_transport_.reset(new SurfaceTextureTransportClient());
     layer_ = surface_texture_transport_->Initialize();
@@ -754,6 +755,13 @@ void RenderWidgetHostViewAndroid::SynchronousFrameMetadata(
   ComputeContentsSize(frame_metadata);
 }
 
+void RenderWidgetHostViewAndroid::SetOverlayVideoMode(bool enabled) {
+  if (overlay_video_mode_ == enabled)

[Ted C, 2013/09/30 15:54:36]

Doesn't look like you really need to store this value at this point.

Couldn't you also just do:

if (layer_->contents_opaque() == !enabled)
  return;
layer_->SetContentsOpaque(!enabled);

I guess it doesn't really matter either way though.

+    return;
+  overlay_video_mode_ = enabled;
+  layer_->SetContentsOpaque(!enabled);
+}
+
 void RenderWidgetHostViewAndroid::UpdateContentViewCoreFrameMetadata(
     const cc::CompositorFrameMetadata& frame_metadata) {
   if (content_view_core_) {