Bug 167203 – LayoutTest inspector/worker/debugger-shared-breakpoint.html is flaky

• Component: Web Inspector
• Status: NEW

[Ryan Haddad]

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.
-

[Blaze Burg]

Joe, is this a race between the worker context and inspector context?

[Ryan Haddad]

Marked test as flaky in http://trac.webkit.org/projects/webkit/changeset/211532