[not for review] Add Draw entries to window Performance Timeline

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 <string>
 
@@ skipping 1058 matching lines @@
           outer_viewport_scroll_delta);
     }
     ApplyPageScaleDeltaFromImplSide(info.page_scale_delta);
 
     client_->ApplyScrollAndScale(
         inner_viewport_scroll_delta + outer_viewport_scroll_delta,
         info.page_scale_delta);
   }
 }
 
+void LayerTreeHost::RecordDrawTiming(const DrawTimingSet& info) {
+  if (!root_layer_.get())
+    return;
+
+  for (size_t i = 0; i < info.draws.size(); ++i) {
+    client_->RecordDrawTiming(info.draws[i].rect_id, info.draws[i].timestamps);

[enne (OOO), 2014/07/22 20:44:25]

Can you help me understand what the eventual API is supposed to be? In some
places in the code you are using source frame number and others you're using
timestamps.

[Mike B, 2014/07/25 23:09:40]

Both.. but timestamp is the main thing being delivered to the caller. Ultimately
I wanted to be able to provide the source of the event back to JS. Although now
that I'm down into that part of the API it's possible that number is meaningless
to JS and I'll trim it back out. In the meantime it gets passed almost all of
the way through, but is dropped right towards the end of cc before it goes to
blink.

On 2014/07/22 20:44:25, enne wrote:

+  }
+}
+
 void LayerTreeHost::StartRateLimiter() {
   if (animating_)
     return;
 
   if (!rate_limit_timer_.IsRunning()) {
     rate_limit_timer_.Start(FROM_HERE,
                             base::TimeDelta(),
                             this,
                             &LayerTreeHost::RateLimit);
   }
@@ skipping 193 matching lines @@
   swap_promise_list_.push_back(swap_promise.Pass());
 }
 
 void LayerTreeHost::BreakSwapPromises(SwapPromise::DidNotSwapReason reason) {
   for (size_t i = 0; i < swap_promise_list_.size(); i++)
     swap_promise_list_[i]->DidNotSwap(reason);
   swap_promise_list_.clear();
 }
 
 }  // namespace cc