Summary: | inspector/debugger/command-line-api-exception.html flakily times out on mac | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryan Haddad <ryanhaddad> | ||||||
Component: | Web Inspector | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bburg, commit-queue, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Ryan Haddad
2015-12-08 17:21:32 PST
--- /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 ')' Marked as flaky on Mac in <https://trac.webkit.org/r193834> 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. *** Bug 152487 has been marked as a duplicate of this bug. *** Created attachment 282509 [details]
[PATCH] Proposed Fix
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. Created attachment 282569 [details]
[PATCH] Proposed Fix
Comment on attachment 282569 [details] [PATCH] Proposed Fix Clearing flags on attachment: 282569 Committed r202784: <http://trac.webkit.org/changeset/202784> All reviewed patches have been landed. Closing bug. |