Title: [199899] trunk/Source/WebInspectorUI
Revision
199899
Author
timo...@apple.com
Date
2016-04-22 13:28:44 -0700 (Fri, 22 Apr 2016)

Log Message

Change an assert to a warn based on post review feedback.

https://bugs.webkit.org/show_bug.cgi?id=156919
rdar://problem/25857118

Rubber-stamped by Joseph Pecoraro.

* UserInterface/Controllers/DebuggerManager.js:
(WebInspector.DebuggerManager.prototype.debuggerDidPause):

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (199898 => 199899)


--- trunk/Source/WebInspectorUI/ChangeLog	2016-04-22 20:12:39 UTC (rev 199898)
+++ trunk/Source/WebInspectorUI/ChangeLog	2016-04-22 20:28:44 UTC (rev 199899)
@@ -1,5 +1,17 @@
 2016-04-22  Timothy Hatcher  <timo...@apple.com>
 
+        Change an assert to a warn based on post review feedback.
+
+        https://bugs.webkit.org/show_bug.cgi?id=156919
+        rdar://problem/25857118
+
+        Rubber-stamped by Joseph Pecoraro.
+
+        * UserInterface/Controllers/DebuggerManager.js:
+        (WebInspector.DebuggerManager.prototype.debuggerDidPause):
+
+2016-04-22  Timothy Hatcher  <timo...@apple.com>
+
         Web Inspector: Debugger statement in console does not provide any call frames and debugger UI is confused
 
         https://bugs.webkit.org/show_bug.cgi?id=156919

Modified: trunk/Source/WebInspectorUI/UserInterface/Controllers/DebuggerManager.js (199898 => 199899)


--- trunk/Source/WebInspectorUI/UserInterface/Controllers/DebuggerManager.js	2016-04-22 20:12:39 UTC (rev 199898)
+++ trunk/Source/WebInspectorUI/UserInterface/Controllers/DebuggerManager.js	2016-04-22 20:28:44 UTC (rev 199899)
@@ -546,7 +546,7 @@
         this._activeCallFrame = this._callFrames[0];
 
         if (!this._activeCallFrame) {
-            console.assert("We should always have one call frame. This could indicate we are hitting an exception or debugger statement in an internal injected script.");
+            console.warn("We should always have one call frame. This could indicate we are hitting an exception or debugger statement in an internal injected script.");
             this._didResumeInternal();
             return;
         }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to