Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(674)

Unified Diff: Source/core/inspector/InspectorLayerTreeAgent.cpp

Issue 166273018: Added showing slow scroll rectangles in Layers panel. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: New fixes. Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/inspector/InspectorLayerTreeAgent.cpp
diff --git a/Source/core/inspector/InspectorLayerTreeAgent.cpp b/Source/core/inspector/InspectorLayerTreeAgent.cpp
index 55762f82a85252a9ec973787903f0917f7d238f0..05448abf69e731fec0d5aed9ce6432c8290fbcc5 100644
--- a/Source/core/inspector/InspectorLayerTreeAgent.cpp
+++ b/Source/core/inspector/InspectorLayerTreeAgent.cpp
@@ -73,6 +73,36 @@ inline String idForLayer(const GraphicsLayer* graphicsLayer)
return String::number(graphicsLayer->platformLayer()->id());
}
+static PassRefPtr<TypeBuilder::LayerTree::ScrollRect> buildScrollRect(const blink::WebRect& rect, const TypeBuilder::LayerTree::ScrollRect::Type::Enum& type)
+{
+ RefPtr<TypeBuilder::DOM::Rect> rectObject = TypeBuilder::DOM::Rect::create()
+ .setX(rect.x)
+ .setY(rect.y)
+ .setHeight(rect.height)
+ .setWidth(rect.width);
+ RefPtr<TypeBuilder::LayerTree::ScrollRect> scrollRectObject = TypeBuilder::LayerTree::ScrollRect::create()
+ .setRect(rectObject.release())
+ .setType(type);
+ return scrollRectObject.release();
+}
+
+static PassRefPtr<TypeBuilder::Array<TypeBuilder::LayerTree::ScrollRect> > buildScrollRectsForLayer(GraphicsLayer* graphicsLayer)
+{
+ RefPtr<TypeBuilder::Array<TypeBuilder::LayerTree::ScrollRect> > scrollRects = TypeBuilder::Array<TypeBuilder::LayerTree::ScrollRect>::create();
+ blink::WebLayer* webLayer = graphicsLayer->platformLayer();
+ for (size_t i = 0; i < webLayer->nonFastScrollableRegion().size(); ++i) {
+ scrollRects->addItem(buildScrollRect(webLayer->nonFastScrollableRegion()[i], TypeBuilder::LayerTree::ScrollRect::Type::RepaintsOnScroll));
+ }
+ for (size_t i = 0; i < webLayer->touchEventHandlerRegion().size(); ++i) {
+ scrollRects->addItem(buildScrollRect(webLayer->touchEventHandlerRegion()[i], TypeBuilder::LayerTree::ScrollRect::Type::TouchEventHandler));
+ }
+ if (webLayer->haveWheelEventHandlers()) {
+ blink::WebRect* webRect = new blink::WebRect(webLayer->position().x, webLayer->position().y, webLayer->bounds().width, webLayer->bounds().height);
caseq 2014/03/11 12:04:09 that's a memory leak. Why do you need a dynamic al
malch 2014/03/11 12:48:49 Done.
+ scrollRects->addItem(buildScrollRect(*webRect, TypeBuilder::LayerTree::ScrollRect::Type::WheelEventHandler));
+ }
+ return scrollRects->length() ? scrollRects.release() : nullptr;
+}
+
static PassRefPtr<TypeBuilder::LayerTree::Layer> buildObjectForLayer(GraphicsLayer* graphicsLayer, int nodeId)
{
blink::WebLayer* webLayer = graphicsLayer->platformLayer();
@@ -107,6 +137,10 @@ static PassRefPtr<TypeBuilder::LayerTree::Layer> buildObjectForLayer(GraphicsLay
layerObject->setAnchorY(anchor.y());
layerObject->setAnchorZ(anchor.z());
}
+ RefPtr<TypeBuilder::Array<TypeBuilder::LayerTree::ScrollRect> > scrollRects =
+ buildScrollRectsForLayer(graphicsLayer);
caseq 2014/03/11 12:04:09 nit: please don't wrap here, it's well within reas
malch 2014/03/11 12:48:49 Done.
+ if (scrollRects)
+ layerObject->setScrollRects(scrollRects.release());
return layerObject;
}

Powered by Google App Engine
This is Rietveld 408576698