Remove forced commit and readback from the scheduler.

cc/trees/thread_proxy.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/thread_proxy.h"
 
 #include <algorithm>
 #include <string>
 
 #include "base/auto_reset.h"
@@ skipping 1134 matching lines @@
   bool forced_draw = false;
   return DrawSwapInternal(forced_draw);
 }
 
 DrawResult ThreadProxy::ScheduledActionDrawAndSwapForced() {
   TRACE_EVENT0("cc", "ThreadProxy::ScheduledActionDrawAndSwapForced");
   bool forced_draw = true;
   return DrawSwapInternal(forced_draw);
 }
 
-DrawResult ThreadProxy::ScheduledActionDrawAndReadback() {
-  TRACE_EVENT0("cc", "ThreadProxy::ScheduledActionDrawAndReadback");
-  NOTREACHED();
-  return INVALID_RESULT;
-}
-
 void ThreadProxy::DidAnticipatedDrawTimeChange(base::TimeTicks time) {
   if (impl().current_resource_update_controller)
     impl().current_resource_update_controller->PerformMoreUpdates(time);
 }
 
 base::TimeDelta ThreadProxy::DrawDurationEstimate() {
   return impl().timing_history.DrawDurationEstimate();
 }
 
 base::TimeDelta ThreadProxy::BeginMainFrameToCommitDurationEstimate() {
@@ skipping 39 matching lines @@
   TRACE_EVENT0("cc", "ThreadProxy::CreateAndInitializeOutputSurface");
   DCHECK(IsMainThread());
 
   // Check that output surface has not been recreated by CompositeAndReadback
   // after this task is posted but before it is run.
   bool has_initialized_output_surface = true;
   {
     CompletionEvent completion;
     Proxy::ImplThreadTaskRunner()->PostTask(
         FROM_HERE,
         base::Bind(&ThreadProxy::HasInitializedOutputSurfaceOnImplThread,

[simonhong, 2014/05/17 00:03:21]

I think we can also remove this check.

[danakj, 2014/05/17 06:07:11]

Yep that's in another cl already :)

[simonhong, 2014/05/17 08:45:31]

Aha! great!

                    impl_thread_weak_ptr_,
                    &completion,
                    &has_initialized_output_surface));
     completion.Wait();
   }
   if (has_initialized_output_surface)
     return;
 
   layer_tree_host()->DidLoseOutputSurface();
   main().output_surface_creation_callback.Reset(
@@ skipping 276 matching lines @@
 
   impl().timing_history.DidActivatePendingTree();
 }
 
 void ThreadProxy::DidManageTiles() {
   DCHECK(IsImplThread());
   impl().scheduler->DidManageTiles();
 }
 
 }  // namespace cc