[DevTools] Adjust svg elements highlight to the root FrameView origin.

Source/core/inspector/InspectorOverlay.cpp

Index: Source/core/inspector/InspectorOverlay.cpp
diff --git a/Source/core/inspector/InspectorOverlay.cpp b/Source/core/inspector/InspectorOverlay.cpp
index 8ede7d22ea0c93f336fca63139b8d0acabc1bc26..596ce39818398bdc034264e452d86c108110b875 100644
--- a/Source/core/inspector/InspectorOverlay.cpp
+++ b/Source/core/inspector/InspectorOverlay.cpp
@@ -315,8 +315,12 @@ static void buildNodeHighlight(Node& node, const HighlightConfig& highlightConfi
     if (renderer->node() && renderer->node()->isSVGElement() && !renderer->isSVGRoot()) {
         Vector<FloatQuad> quads;
         renderer->absoluteQuads(quads);
-        for (size_t i = 0; i < quads.size(); ++i)
+        FrameView* containingView = renderer->frameView();
+        for (size_t i = 0; i < quads.size(); ++i) {
+            if (containingView)
+                contentsQuadToScreen(containingView, quads[i]);
             highlight->appendQuad(quads[i], highlightConfig.content, highlightConfig.contentOutline);
+        }
         return;
     }
 
@@ -833,6 +837,19 @@ bool InspectorOverlay::getBoxModel(Node* node, RefPtr<TypeBuilder::DOM::BoxModel
     return true;
 }
 
+String InspectorOverlay::getHighlightJSONForNode(Node* node)

[caseq, 2014/10/24 13:31:44]

Can we return JSONObject for better reuse? It's never too late to stringize it.

[dgozman, 2014/10/24 14:45:21]

Done.

+{
+    HighlightConfig config;

[caseq, 2014/10/24 13:31:44]

So we end up with a set of path that are all the same color and hence impossible
to distinguish. Should we make the colors different to improve coverage?

[dgozman, 2014/10/24 14:45:21]

Nice idea! Done.

+    config.showInfo = true;
+    config.showRulers = true;
+    config.showExtensionLines = true;
+    Highlight highlight;
+    appendPathsForShapeOutside(highlight, config, node);
+    buildNodeHighlight(*node, config, &highlight);
+    RefPtr<JSONObject> json(highlight.asJSONObject());
+    return json->toPrettyJSONString();
+}
+
 void InspectorOverlay::freePage()
 {
     if (m_overlayPage) {