Bug 167203 - LayoutTest inspector/worker/debugger-shared-breakpoint.html is flaky
Summary: LayoutTest inspector/worker/debugger-shared-breakpoint.html is flaky
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-01-19 10:19 PST by Ryan Haddad
Modified: 2017-02-01 14:35 PST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan Haddad 2017-01-19 10:19:11 PST
LayoutTest inspector/worker/debugger-shared-breakpoint.html is flaky

https://build.webkit.org/results/Apple%20Sierra%20Debug%20WK2%20(Tests)/r210923%20(1692)/results.html

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=inspector%2Fworker%2Fdebugger-shared-breakpoint.html

--- /Volumes/Data/slave/sierra-debug-tests-wk2/build/layout-test-results/inspector/worker/debugger-shared-breakpoint-expected.txt
+++ /Volumes/Data/slave/sierra-debug-tests-wk2/build/layout-test-results/inspector/worker/debugger-shared-breakpoint-actual.txt
@@ -7,21 +7,11 @@
 PASS: Two Workers ready.
 
 -- Running test case: Worker.Debugger.SharedBreakpoint.BreakpointAdded
-PASS: All Targets paused.
-PASS: Active CalleFrame should be in a Worker.
-PASS: Pause reason should be a breakpoint.
-PAUSE AT triggerBreakpoint:9:5
-      5    
-      6    function triggerBreakpoint() {
-      7        let alpha = 1;
- ->   8        |let beta = 2; // BREAKPOINT
-      9        let gamma = 3;
-     10    }
-     11    
+ASSERT: undefined
+!! EXCEPTION: null is not an object (evaluating 'workerTarget1.mainResource.createSourceCodeLocation')
+Stack Trace: #0: (anonymous) (test)
+#1: initializePromise [native code]
+#2: Promise [native code]
+#3: (anonymous) (TestCombined.js:1591:35)
+#4: promiseReactionJob [native code]
 
-
-
--- Running test case: Worker.Debugger.SharedBreakpoint.BreakpointRemoved
-PASS: Worker 1 should not have paused.
-PASS: Worker 2 should not have paused.
-
Comment 1 BJ Burg 2017-01-19 10:28:40 PST
Joe, is this a race between the worker context and inspector context?
Comment 2 Ryan Haddad 2017-02-01 14:35:55 PST
Marked test as flaky in http://trac.webkit.org/projects/webkit/changeset/211532