Reschedule COMPOSITE_EVENTUALLY instead of COMPOSITE_IMMEDIATELY if we miss a browser frame

content/browser/renderer_host/compositor_impl_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/compositor_impl_android.h"
 
 #include <android/bitmap.h>
 #include <android/native_window_jni.h>
 
 #include "base/android/jni_android.h"
@@ skipping 664 matching lines @@
     scoped_ptr<cc::CopyOutputRequest> request) {
   root_layer_->RequestCopyOfOutput(request.Pass());
 }
 
 void CompositorImpl::OnVSync(base::TimeTicks frame_time,
                              base::TimeDelta vsync_period) {
   vsync_period_ = vsync_period;
   last_vsync_ = frame_time;
 
   if (WillCompositeThisFrame()) {
-    // We somehow missed the last vsync interval, so reschedule immediately.
+    // We somehow missed the last vsync interval, so reschedule.

[no sievers, 2014/06/03 17:04:11]

Maybe put a small comment to say why we schedule for the deadline, like "We
cannot schedule immediately, or will get us out-of-phase with new frames".

[Yufeng Shen (Slow to review), 2014/06/03 18:22:36]

Done.

     CancelComposite();
-    composite_on_vsync_trigger_ = COMPOSITE_IMMEDIATELY;
+    composite_on_vsync_trigger_ = COMPOSITE_EVENTUALLY;
   } else {
     current_composite_task_.reset();
   }
 
   DCHECK(!DidCompositeThisFrame() && !WillCompositeThisFrame());
   if (composite_on_vsync_trigger_ != DO_NOT_COMPOSITE) {
     CompositingTrigger trigger = composite_on_vsync_trigger_;
     composite_on_vsync_trigger_ = DO_NOT_COMPOSITE;
     PostComposite(trigger);
   }
 }
 
 void CompositorImpl::SetNeedsAnimate() {
   if (!host_)
     return;
 
   host_->SetNeedsAnimate();
 }
 
 }  // namespace content