Bug 165582 – LayoutTest inspector/worker/debugger-scripts.html is a flaky failure

• Component: Web Inspector
• Status: RESOLVED FIXED

[Ryan Haddad]

LayoutTest inspector/worker/debugger-scripts.html is a flaky failure

https://build.webkit.org/builders/Apple%20El%20Capitan%20Debug%20WK2%20(Tests)/builds/9820

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

--- /Volumes/Data/slave/elcapitan-debug-tests-wk2/build/layout-test-results/inspector/worker/debugger-scripts-expected.txt
+++ /Volumes/Data/slave/elcapitan-debug-tests-wk2/build/layout-test-results/inspector/worker/debugger-scripts-actual.txt
@@ -3,28 +3,8 @@
 
 == Running test suite: Worker.Debugger.Script
 -- Running test case: Worker.Debugger.Script.MainResource
-PASS: Worker target's main resource should be a Script.
-PASS: Worker target's main resource should be owned by a Worker target.
-PASS: Worker DebuggerData should include script.
-PASS: Worker DebuggerData should include script by identifier.
-PASS: Worker DebuggerData should include script by URL.
-PASS: Main DebuggerData should not include the script.
-PASS: Worker target's main resource should have content.
+FAIL: Worker target's main resource should be a Script.
+    Expected: truthy
+    Actual: false
+!! EXCEPTION: null is not an object (evaluating 'script.target')
 
--- Running test case: Worker.Debugger.Script.ImportedScript
-PASS: ScriptAdded - inspector/worker/resources/worker-import-1.js
-PASS: Script should be owned by a Worker target.
-PASS: Worker DebuggerData should include script.
-PASS: Worker DebuggerData should include script by identifier.
-PASS: Worker DebuggerData should include script by URL.
-PASS: Main DebuggerData should not include the script.
-PASS: Imported script content should have 'workerImport1'.
-
--- Running test case: Worker.Debugger.Script.NamedEval
-PASS: ScriptAdded - worker-eval.js
-PASS: Script should be owned by a Worker target.
-PASS: Worker DebuggerData should include script.
-PASS: Worker DebuggerData should include script by identifier.
-PASS: Main DebuggerData should not include the script.
-PASS: Eval script content should have 'workerEval'.
-

[Ryan Haddad]

I'm not sure that this is a regression. The test was added on 11/2 and the first flake was seen 11/3.

[Ryan Haddad]

Marked test as flaky on mac-wk2 in https://trac.webkit.org/changeset/209517

[Ryan Haddad]

This happens on WK1 as well: https://build.webkit.org/results/Apple%20Sierra%20Debug%20WK1%20(Tests)/r210748%20(2475)/results.html

[Ryan Haddad]

Moved flaky expectation to platform/mac in http://trac.webkit.org/projects/webkit/changeset/210754

[Yury Semikhatsky]

Created attachment 375667 [details]
Patch

[Joseph Pecoraro]

Comment on attachment 375667 [details]
Patch

Awesome! r=me

[WebKit Commit Bot]

Comment on attachment 375667 [details]
Patch

Clearing flags on attachment: 375667

Committed r248392: <https://trac.webkit.org/changeset/248392>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/54051909>