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

content/renderer/compositor_bindings/web_layer_impl.cc

 // Copyright 2014 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/compositor_bindings/web_layer_impl.h"
 
+#include <vector>
+
 #include "base/bind.h"
 #include "base/debug/trace_event_impl.h"
 #include "base/lazy_instance.h"
 #include "base/strings/string_util.h"
 #include "base/threading/thread_checker.h"
 #include "cc/animation/animation.h"
 #include "cc/base/region.h"
 #include "cc/base/switches.h"
 #include "cc/layers/layer.h"
 #include "cc/layers/layer_position_constraint.h"
@@ skipping 159 matching lines @@
 
 void WebLayerImpl::setTransformOrigin(const blink::WebFloatPoint3D& point) {
   gfx::Point3F gfx_point = point;
   layer_->SetTransformOrigin(gfx_point);
 }
 
 blink::WebFloatPoint3D WebLayerImpl::transformOrigin() const {
   return layer_->transform_origin();
 }
 
-void WebLayerImpl::setAnchorPoint(const blink::WebFloatPoint&) {}
+void WebLayerImpl::setAnchorPoint(const blink::WebFloatPoint& point) {}
 
 blink::WebFloatPoint WebLayerImpl::anchorPoint() const {
   return blink::WebFloatPoint();
 }
 
-void WebLayerImpl::setAnchorPointZ(float) {}
+void WebLayerImpl::setAnchorPointZ(float z) {}
 
 float WebLayerImpl::anchorPointZ() const {
   return 0.f;
-};
+}
 
 SkMatrix44 WebLayerImpl::transform() const {
   return layer_->transform().matrix();
 }
 
 void WebLayerImpl::setDrawsContent(bool draws_content) {
   layer_->SetIsDrawable(draws_content);
 }
 
 bool WebLayerImpl::drawsContent() const {
@@ skipping 144 matching lines @@
   size_t i = 0;
   for (cc::Region::Iterator region_rects(layer_->non_fast_scrollable_region());
        region_rects.has_rect();
        region_rects.next()) {
     result[i] = region_rects.rect();
     ++i;
   }
   return result;
 }
 
+void WebLayerImpl::setDrawFrameRequestRects(const WebVector<std::pair<int64_t,
+                                            WebRect> >& rects) {
+  std::vector<std::pair<int64_t, gfx::Rect> > gfxRects(rects.size());
+  for (size_t i = 0; i < rects.size(); ++i) {
+    gfx::Rect gfxRect(rects[i].second);

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

style nit: This file should be in Chrome style, so gfxRect => gfx_rect. 
Actually, you should probably just call it rect or skip the temporary.

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

Done.

+    gfxRects[i] = std::make_pair(rects[i].first, gfxRect);
+  }
+  layer_->SetDrawFrameRequestRects(gfxRects);
+}
+
+WebVector<std::pair<int64_t, WebRect> >
+WebLayerImpl::drawFrameRequestRects() const {
+  std::vector<std::pair<int64_t, gfx::Rect> > rects =
+      layer_->draw_frame_request_rects();
+  size_t num_rects = rects.size();
+  WebVector<std::pair<int64_t, WebRect> > result(num_rects);
+  for (size_t i = 0; i < num_rects; ++i) {
+    WebRect rect(rects[i].second);
+    result[i] = std::make_pair(rects[i].first, rect);
+  }
+  return result;
+}
+
 void WebLayerImpl::setTouchEventHandlerRegion(const WebVector<WebRect>& rects) {
   cc::Region region;
   for (size_t i = 0; i < rects.size(); ++i)
     region.Union(rects[i]);
   layer_->SetTouchEventHandlerRegion(region);
 }
 
 WebVector<WebRect> WebLayerImpl::touchEventHandlerRegion() const {
   size_t num_rects = 0;
   for (cc::Region::Iterator region_rects(layer_->touch_event_handler_region());
@@ skipping 108 matching lines @@
   if (parent)
     clip_parent = static_cast<WebLayerImpl*>(parent)->layer();
   layer_->SetClipParent(clip_parent);
 }
 
 Layer* WebLayerImpl::layer() const {
   return layer_.get();
 }
 
 }  // namespace content
-