Allows fullscreen to be triggered with the key events used for scrolling.

cc/trees/layer_tree_host.cc

 // Copyright 2011 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 "cc/trees/layer_tree_host.h"
 
 #include <algorithm>
 #include <stack>
 
 #include "base/bind.h"
@@ skipping 389 matching lines @@
                                 rendering_stats_instrumentation_.get());
   if (settings_.calculate_top_controls_position &&
       host_impl->top_controls_manager()) {
     top_controls_manager_weak_ptr_ =
         host_impl->top_controls_manager()->AsWeakPtr();
   }
   input_handler_weak_ptr_ = host_impl->AsWeakPtr();
   return host_impl.Pass();
 }
 
+void LayerTreeHost::ShowTopControls(bool show) {

[Ted C, 2013/06/03 17:43:42]

Same thing about moving this next to the other method (matching the ordering of
the .h file).

[Jinsuk Kim, 2013/06/05 08:21:07]

Done.

+  proxy_->ImplThread()->PostTask(

[Ted C, 2013/06/03 17:43:42]

Similar to the other method, I would add this to the start of the method:

if (!settings_.calculate_top_controls_position)
  return;

[Jinsuk Kim, 2013/06/05 08:21:07]

Done.

+      base::Bind(
+          &TopControlsManager::UpdateTopControlsStatePreservingConstraints,
+          top_controls_manager_weak_ptr_,
+          show ? cc::SHOWN : cc::HIDDEN,
+          true));
+}
+
 void LayerTreeHost::DidLoseOutputSurface() {
   TRACE_EVENT0("cc", "LayerTreeHost::DidLoseOutputSurface");
   DCHECK(proxy_->IsMainThread());
 
   if (output_surface_lost_)
     return;
 
   num_failed_recreate_attempts_ = 0;
   output_surface_lost_ = true;
   SetNeedsCommit();
@@ skipping 597 matching lines @@
     bool start_ready_animations = true;
     (*iter).second->UpdateState(start_ready_animations, NULL);
   }
 }
 
 skia::RefPtr<SkPicture> LayerTreeHost::CapturePicture() {
   return proxy_->CapturePicture();
 }
 
 }  // namespace cc