Bug 152029 – inspector/debugger/command-line-api-exception.html flakily times out on mac

• Component: Web Inspector
• Status: RESOLVED FIXED

[Ryan Haddad]

inspector/debugger/command-line-api-exception.html flakily times out on mac

Flakiness dashboard:
<http://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=inspector%2Fdebugger%2Fcommand-line-api-exception.html>

Most recent timeout:
<https://build.webkit.org/builders/Apple%20Yosemite%20Debug%20WK1%20%28Tests%29/builds/9383>

Has happened since being unskipped in r193427

[Ryan Haddad]

--- /Volumes/Data/slave/yosemite-debug-tests-wk1/build/layout-test-results/inspector/debugger/command-line-api-exception-expected.txt
+++ /Volumes/Data/slave/yosemite-debug-tests-wk1/build/layout-test-results/inspector/debugger/command-line-api-exception-actual.txt
@@ -1,32 +1,10 @@
 CONSOLE MESSAGE: line 4: TypeError: undefined is not an object (evaluating '[].x.x')
 CONSOLE MESSAGE: line 10: ReferenceError: Can't find variable: variableThatDoesNotExist
-CONSOLE MESSAGE: line 16: SyntaxError: Unexpected token ')'
-CONSOLE MESSAGE: line 22: IndexSizeError: DOM Exception 1: Index or size was negative, or greater than the allowed value.
-CONSOLE MESSAGE: line 27: thrown string
-CONSOLE MESSAGE: line 32: 123.456
-CONSOLE MESSAGE: line 37: null
-CONSOLE MESSAGE: line 42: [object Object]
-CONSOLE MESSAGE: line 47: [object HTMLBodyElement]
-CONSOLE MESSAGE: line 55: CATCHER: TypeError: undefined is not an object (evaluating '[].x.x')
-CONSOLE MESSAGE: line 55: CATCHER: thrown string
-CONSOLE MESSAGE: line 55: CATCHER: [object Object]
+FAIL: Timed out waiting for notifyDone to be called
 Checks that $exception is available and accurate in evaluations when paused on an exception.
 
 BEFORE: $exception => undefined
 PAUSE #1: $exception => TypeError: undefined is not an object (evaluating '[].x.x')
 PAUSE #2: $exception => ReferenceError: Can't find variable: variableThatDoesNotExist
-PAUSE #3: $exception => SyntaxError: Unexpected token ')'
-PAUSE #4: $exception => Error: IndexSizeError: DOM Exception 1
-PAUSE #5: $exception => thrown string
-PAUSE #6: $exception => 123.456
-PAUSE #7: $exception => null
-PAUSE #8: $exception => Object
-PAUSE #9: $exception => body
-PAUSE #10: $exception => TypeError: undefined is not an object (evaluating '[].x.x')
-STEPPED OUT TO CATCH BLOCK: $exception === e ? true
-PAUSE #11: $exception => thrown string
-STEPPED OUT TO CATCH BLOCK: $exception === e ? true
-PAUSE #12: $exception => Object
-STEPPED OUT TO CATCH BLOCK: $exception === e ? true
-AFTER: $exception => undefined
+AFTER: $exception => SyntaxError: Unexpected token ')'

[Radar WebKit Bug Importer]

<rdar://problem/23813812>

[Ryan Haddad]

Marked as flaky on Mac in <https://trac.webkit.org/r193834>

[Joseph Pecoraro]

This kind of timeout with output sounds like it is actually just really really slow.

I ran the test locally and found an issue with it, so I'm just completely rewriting the test.

[Joseph Pecoraro]

*** Bug 152487 has been marked as a duplicate of this bug. ***

[Joseph Pecoraro]

Created attachment 282509 [details]
[PATCH] Proposed Fix

[Blaze Burg]

Comment on attachment 282509 [details]
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=282509&action=review

> LayoutTests/inspector/debugger/command-line-api-exception-nested-catch.html:47
> +                if (!WebInspector.debuggerManager.activeCallFrame)

You *probably* could rewrite this using DebuggerManager's promises and avoid the phase thing. See comment below.

> LayoutTests/inspector/debugger/command-line-api-exception.html:36
> +                        WebInspector.debuggerManager.resume();

(applies throughout) Since many DebuggerManager commands return promises, you could do:

WebInspector.debuggerManager.resume().then(resolve, reject);

Internally, it does a single fire listener for the same event. This helps to keep the logic of event ordering inside DebuggerManager for many use cases.

[Joseph Pecoraro]

Created attachment 282569 [details]
[PATCH] Proposed Fix

[WebKit Commit Bot]

Comment on attachment 282569 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 282569

Committed r202784: <http://trac.webkit.org/changeset/202784>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.