Title: [199940] trunk/Source/WebCore
Revision
199940
Author
commit-qu...@webkit.org
Date
2016-04-22 17:44:45 -0700 (Fri, 22 Apr 2016)

Log Message

Web Inspector: Include columnNumber in event listener locations
https://bugs.webkit.org/show_bug.cgi?id=156927
<rdar://problem/25884584>

Patch by Joseph Pecoraro <pecor...@apple.com> on 2016-04-22
Reviewed by Brian Burg.

* inspector/InspectorDOMAgent.cpp:
(WebCore::InspectorDOMAgent::buildObjectForEventListener):
Include the column number in the location as well.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (199939 => 199940)


--- trunk/Source/WebCore/ChangeLog	2016-04-23 00:40:43 UTC (rev 199939)
+++ trunk/Source/WebCore/ChangeLog	2016-04-23 00:44:45 UTC (rev 199940)
@@ -1,3 +1,15 @@
+2016-04-22  Joseph Pecoraro  <pecor...@apple.com>
+
+        Web Inspector: Include columnNumber in event listener locations
+        https://bugs.webkit.org/show_bug.cgi?id=156927
+        <rdar://problem/25884584>
+
+        Reviewed by Brian Burg.
+
+        * inspector/InspectorDOMAgent.cpp:
+        (WebCore::InspectorDOMAgent::buildObjectForEventListener):
+        Include the column number in the location as well.
+
 2016-04-22  Brent Fulgham  <bfulg...@apple.com>
 
         [Win] Unreviewed build fix.

Modified: trunk/Source/WebCore/inspector/InspectorDOMAgent.cpp (199939 => 199940)


--- trunk/Source/WebCore/inspector/InspectorDOMAgent.cpp	2016-04-23 00:40:43 UTC (rev 199939)
+++ trunk/Source/WebCore/inspector/InspectorDOMAgent.cpp	2016-04-23 00:44:45 UTC (rev 199940)
@@ -1466,6 +1466,7 @@
     JSC::JSObject* handler = nullptr;
     String body;
     int lineNumber = 0;
+    int columnNumber = 0;
     String scriptID;
     String sourceName;
     if (auto scriptListener = JSEventListener::cast(eventListener.get())) {
@@ -1478,6 +1479,7 @@
                 if (!function->isHostOrBuiltinFunction()) {
                     if (auto executable = function->jsExecutable()) {
                         lineNumber = executable->firstLine() - 1;
+                        columnNumber = executable->startColumn() - 1;
                         scriptID = executable->sourceID() == JSC::SourceProvider::nullID ? emptyString() : String::number(executable->sourceID());
                         sourceName = executable->sourceURL();
                     }
@@ -1503,6 +1505,7 @@
             .setScriptId(scriptID)
             .setLineNumber(lineNumber)
             .release();
+        location->setColumnNumber(columnNumber);
         value->setLocation(WTFMove(location));
         if (!sourceName.isEmpty())
             value->setSourceName(sourceName);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to