Changeset 199899 in webkit

Timestamp:

Apr 22, 2016, 1:28:44 PM (9 years ago)

Author:

timothy@apple.com

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):

Location:

trunk/Source/WebInspectorUI

Files:

• ChangeLog (modified) (1 diff)
• UserInterface/Controllers/DebuggerManager.js (modified) (1 diff)

trunk/Source/WebInspectorUI/ChangeLog

@@ +1 @@
+2016-04-22  Timothy Hatcher  <timothy@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  <timothy@apple.com>
 

trunk/Source/WebInspectorUI/UserInterface/Controllers/DebuggerManager.js

@@ -547 +547 @@
 
         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;