Bug 153016 – LayoutTest inspector/script-profiler/event-type-Other.html is flaky

• Component: Web Inspector
• Status: RESOLVED FIXED

[Ryan Haddad]

LayoutTest inspector/script-profiler/event-type-Other.html is flaky

Most recent failing run:
<https://build.webkit.org/builders/Apple%20El%20Capitan%20Debug%20WK2%20%28Tests%29/builds/2144>

Flakiness dashboard:
<http://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=inspector%2Fscript-profiler%2Fevent-type-Other.html>

--- /Volumes/Data/slave/elcapitan-debug-tests-wk2/build/layout-test-results/inspector/script-profiler/event-type-Other-expected.txt
+++ /Volumes/Data/slave/elcapitan-debug-tests-wk2/build/layout-test-results/inspector/script-profiler/event-type-Other-actual.txt
@@ -11,9 +11,7 @@
 PASS: Event type should be Other.
 ScriptProfiler.trackingUpdate
 PASS: Event type should be Other.
-ScriptProfiler.trackingUpdate
-PASS: Event type should be Other.
 ScriptProfiler.trackingComplete
 PASS: Profiles should exist when complete.
-PASS: Should be 3 profiles for this session.
+FAIL: Should be 3 profiles for this session.

[Ryan Haddad]

Marked test as flaky in <https://trac.webkit.org/r194893>

[Joseph Pecoraro]

Hmm, this is weird. Well the test relies on a setTimeout of 50ms to occur after some work with a timeout of 0ms, but maybe on some systems that is not fast enough. We may be able to have the frontend cue off of an event, like a console message added.

[Radar WebKit Bug Importer]

<rdar://problem/24192919>

[Joseph Pecoraro]

Created attachment 269081 [details]
[PATCH] Proposed Fix

[Blaze Burg]

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

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

r=me. GLORIOUS.

> LayoutTests/inspector/unit-tests/protocol-test-dispatch-event-to-frontend.html:15
> +            ProtocolTest.singleFireEventListener("MyEvent", (event) => {

This reads better than I expected.

[WebKit Commit Bot]

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

Clearing flags on attachment: 269081

Committed r195147: <http://trac.webkit.org/changeset/195147>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.