cc: Remove SetLayerTreeHostClientReady()

content/renderer/render_widget.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/renderer/render_widget.h"
 
 #include "base/auto_reset.h"
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/logging.h"
@@ skipping 1321 matching lines @@
   physical_backing_size_ = gfx::ScaleToCeiledSize(size_, device_scale_factor_);
   if (compositor_)
     compositor_->setViewportSize(size_, physical_backing_size_);
 }
 
 void RenderWidget::initializeLayerTreeView() {
   DCHECK(!host_closing_);
 
   compositor_ = RenderWidgetCompositor::Create(this, compositor_deps_);
   compositor_->setViewportSize(size_, physical_backing_size_);
+
+  // For background pages and certain tests, we don't want to trigger
+  // OutputSurface creation.
+  if (never_visible_ || !RenderThreadImpl::current())
+    compositor_->SetNeverVisible();
+
   StartCompositor();
 }
 
 void RenderWidget::WillCloseLayerTreeView() {
   if (host_closing_)
     return;
 
   // Prevent new compositors or output surfaces from being created.
   host_closing_ = true;
 
@@ skipping 809 matching lines @@
   // it can be bundled in the event ack.
   if (handling_event_overscroll_) {
     handling_event_overscroll_->reset(new DidOverscrollParams(params));
     return;
   }
 
   Send(new InputHostMsg_DidOverscroll(routing_id_, params));
 }
 
 void RenderWidget::StartCompositor() {
-  // For widgets that are never visible, we don't need the compositor to run
-  // at all.
-  if (never_visible_)
-    return;
-  // In tests without a RenderThreadImpl, don't set ready as this kicks
-  // off creating output surfaces that the test can't create.
-  if (!RenderThreadImpl::current())
-    return;
-  compositor_->StartCompositor();
+  if (!is_hidden())
+    compositor_->setVisible(true);
 }
 
 void RenderWidget::SchedulePluginMove(const WebPluginGeometry& move) {
   size_t i = 0;
   for (; i < plugin_window_moves_.size(); ++i) {
     if (plugin_window_moves_[i].window == move.window) {
       if (move.rects_valid) {
         plugin_window_moves_[i] = move;
       } else {
         plugin_window_moves_[i].visible = move.visible;
@@ skipping 195 matching lines @@
 void RenderWidget::RegisterVideoHoleFrame(RenderFrameImpl* frame) {
   video_hole_frames_.AddObserver(frame);
 }
 
 void RenderWidget::UnregisterVideoHoleFrame(RenderFrameImpl* frame) {
   video_hole_frames_.RemoveObserver(frame);
 }
 #endif  // defined(VIDEO_HOLE)
 
 }  // namespace content