Enables fullscreen subtitle and media control from Blink

content/browser/renderer_host/render_widget_host_view_android.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/renderer_host/render_widget_host_view_android.h"
 
 #include <android/bitmap.h>
 
 #include "base/basictypes.h"
 #include "base/bind.h"
@@ skipping 97 matching lines @@
     : host_(widget_host),
       needs_begin_frame_(false),
       are_layers_attached_(true),
       content_view_core_(NULL),
       ime_adapter_android_(this),
       cached_background_color_(SK_ColorWHITE),
       texture_id_in_layer_(0),
       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();
     layer_->SetIsDrawable(true);
   } else {
     if (CommandLine::ForCurrentProcess()->HasSwitch(
             switches::kEnableDelegatedRenderer)) {
       delegated_renderer_layer_ = cc::DelegatedRendererLayer::Create(this);
       layer_ = delegated_renderer_layer_;
     } else {
@@ skipping 618 matching lines @@
 }
 
 void RenderWidgetHostViewAndroid::SynchronousFrameMetadata(
     const cc::CompositorFrameMetadata& frame_metadata) {
   // This is a subset of OnSwapCompositorFrame() used in the synchronous
   // compositor flow.
   UpdateContentViewCoreFrameMetadata(frame_metadata);
   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_) {
     // All offsets and sizes are in CSS pixels.
     content_view_core_->UpdateFrameInfo(
         frame_metadata.root_scroll_offset,
         frame_metadata.page_scale_factor,
         gfx::Vector2dF(frame_metadata.min_page_scale_factor,
                        frame_metadata.max_page_scale_factor),
         frame_metadata.root_layer_size,
@@ skipping 569 matching lines @@
 // RenderWidgetHostView, public:
 
 // static
 RenderWidgetHostView*
 RenderWidgetHostView::CreateViewForWidget(RenderWidgetHost* widget) {
   RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(widget);
   return new RenderWidgetHostViewAndroid(rwhi, NULL);
 }
 
 } // namespace content