Add SwapPromise support to LayerTreeHost and LayerTreeImpl

cc/trees/layer_tree_impl.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_impl.h"
 
 #include "base/debug/trace_event.h"
 #include "cc/animation/keyframed_animation_curve.h"
 #include "cc/animation/scrollbar_animation_controller.h"
 #include "cc/base/math_util.h"
 #include "cc/base/util.h"
 #include "cc/debug/traced_value.h"
 #include "cc/layers/heads_up_display_layer_impl.h"
 #include "cc/layers/layer.h"
 #include "cc/layers/render_surface_impl.h"
 #include "cc/layers/scrollbar_layer_impl_base.h"
 #include "cc/resources/ui_resource_request.h"
 #include "cc/trees/layer_tree_host_common.h"
 #include "cc/trees/layer_tree_host_impl.h"
 #include "ui/gfx/size_conversions.h"
 #include "ui/gfx/vector2d_conversions.h"
 
+namespace {
+const size_t kMaxQueuedSwapPromiseNumber = 100;

[danakj, 2013/11/13 23:22:09]

Can we share these constants instead of duplicating? They need to be the same
size.

[Yufeng Shen (Slow to review), 2013/11/14 22:09:55]

Moved to swap_promise.h

+}  // namespace
+
 namespace cc {
 
 LayerTreeImpl::LayerTreeImpl(LayerTreeHostImpl* layer_tree_host_impl)
     : layer_tree_host_impl_(layer_tree_host_impl),
       source_frame_number_(-1),
       hud_layer_(0),
       root_scroll_layer_(NULL),
       currently_scrolling_layer_(NULL),
       root_layer_scroll_offset_delegate_(NULL),
       background_color_(0),
@@ skipping 82 matching lines @@
   // The request queue should have been processed and does not require a push.
   DCHECK_EQ(ui_resource_request_queue_.size(), 0u);
 
   if (next_activation_forces_redraw_) {
     layer_tree_host_impl_->SetFullRootLayerDamage();
     next_activation_forces_redraw_ = false;
   }
 
   target_tree->SetLatencyInfo(latency_info_);
   latency_info_.Clear();
+
+  target_tree->TakeSwapPromise(&swap_promise_list_);
+
   target_tree->SetPageScaleFactorAndLimits(
       page_scale_factor(), min_page_scale_factor(), max_page_scale_factor());
   target_tree->SetPageScaleDelta(
       target_tree->page_scale_delta() / target_tree->sent_page_scale_delta());
   target_tree->set_sent_page_scale_delta(1);
 
   if (settings().use_pinch_virtual_viewport) {
     target_tree->SetViewportLayersFromIds(
         page_scale_layer_->id(),
         inner_viewport_scroll_layer_->id(),
@@ skipping 566 matching lines @@
 }
 
 const ui::LatencyInfo& LayerTreeImpl::GetLatencyInfo() {
   return latency_info_;
 }
 
 void LayerTreeImpl::ClearLatencyInfo() {
   latency_info_.Clear();
 }
 
+void LayerTreeImpl::QueueSwapPromise(SwapPromise* swap_promise) {

[danakj, 2013/11/13 23:22:09]

Same comments here as in LTH

[Yufeng Shen (Slow to review), 2013/11/14 22:09:55]

Done.

+  if (swap_promise_list_.size() > kMaxQueuedSwapPromiseNumber) {
+    NOTREACHED() << "LayerTreeImpl::swap_promise_list overflows";
+    BreakSwapPromise(SwapPromise::SWAP_PROMISE_LIST_OVERFLOW);
+  }
+  swap_promise_list_.push_back(swap_promise);
+}
+
+void LayerTreeImpl::TakeSwapPromise(
+    ScopedVector<SwapPromise>* new_swap_promise) {
+  swap_promise_list_.insert(swap_promise_list_.end(),
+                            new_swap_promise->begin(),
+                            new_swap_promise->end());
+  new_swap_promise->weak_clear();
+}
+
+void LayerTreeImpl::FinishSwapPromise() {
+  for (size_t i = 0; i < swap_promise_list_.size(); i++) {
+    swap_promise_list_[i]->DidSwap();
+  }
+  swap_promise_list_.clear();
+}
+
+void LayerTreeImpl::BreakSwapPromise(SwapPromise::DidNotSwapReason reason) {
+  for (size_t i = 0; i < swap_promise_list_.size(); i++) {
+    swap_promise_list_[i]->DidNotSwap(reason);
+  }
+  swap_promise_list_.clear();
+}
+
 void LayerTreeImpl::DidModifyTilePriorities() {
   layer_tree_host_impl_->DidModifyTilePriorities();
 }
 
 void LayerTreeImpl::set_ui_resource_request_queue(
     const UIResourceRequestQueue& queue) {
   ui_resource_request_queue_ = queue;
 }
 
 ResourceProvider::ResourceId LayerTreeImpl::ResourceIdForUIResource(
@@ skipping 56 matching lines @@
 const std::vector<LayerImpl*> LayerTreeImpl::LayersWithCopyOutputRequest()
     const {
   // Only the active tree needs to know about layers with copy requests, as
   // they are aborted if not serviced during draw.
   DCHECK(IsActiveTree());
 
   return layers_with_copy_output_request_;
 }
 
 }  // namespace cc