Title: [149829] trunk/Source/WebCore
Revision
149829
Author
[email protected]
Date
2013-05-09 11:47:52 -0700 (Thu, 09 May 2013)

Log Message

Fix an exception when hovering native functions while paused in the debugger.

https://webkit.org/b/115866
rdar://problem/13681982

Reviewed by Darin Adler and Joseph Pecoraro.

* inspector/InjectedScriptSource.js:
(InjectedScript.prototype.getFunctionDetails): Return an error if the details
is null or undefined.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (149828 => 149829)


--- trunk/Source/WebCore/ChangeLog	2013-05-09 18:42:49 UTC (rev 149828)
+++ trunk/Source/WebCore/ChangeLog	2013-05-09 18:47:52 UTC (rev 149829)
@@ -1,3 +1,16 @@
+2013-05-09  Timothy Hatcher  <[email protected]>
+
+        Fix an exception when hovering native functions while paused in the debugger.
+
+        https://webkit.org/b/115866
+        rdar://problem/13681982
+
+        Reviewed by Darin Adler and Joseph Pecoraro.
+
+        * inspector/InjectedScriptSource.js:
+        (InjectedScript.prototype.getFunctionDetails): Return an error if the details
+        is null or undefined.
+
 2013-05-09  Darin Adler  <[email protected]>
 
         [Mac] CFError leak under Objective-C garbage collection

Modified: trunk/Source/WebCore/inspector/InjectedScriptSource.js (149828 => 149829)


--- trunk/Source/WebCore/inspector/InjectedScriptSource.js	2013-05-09 18:42:49 UTC (rev 149828)
+++ trunk/Source/WebCore/inspector/InjectedScriptSource.js	2013-05-09 18:47:52 UTC (rev 149829)
@@ -345,6 +345,8 @@
         if (typeof func !== "function")
             return "Cannot resolve function by id.";
         var details = InjectedScriptHost.functionDetails(func);
+        if (!details)
+            return "Cannot resolve function details.";
         if ("rawScopes" in details) {
             var objectGroupName = this._idToObjectGroupName[parsedFunctionId.id];
             var rawScopes = details.rawScopes;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to