Bug 101229 - [chromium] Layout test inspector-protocol/debugger-pause-dedicated-worker.html is flaky
Summary: [chromium] Layout test inspector-protocol/debugger-pause-dedicated-worker.htm...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Yury Semikhatsky
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-05 09:18 PST by Stephen White
Modified: 2012-12-04 14:30 PST (History)
3 users (show)

See Also:

Attachments
Patch (4.87 KB, patch)
2012-12-04 14:27 PST, Yury Semikhatsky 		vsevik: review+
vsevik: commit-queue+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Stephen White 2012-11-05 09:18:45 PST 
Layout test inspector-protocol/debugger-pause-dedicated-worker.html is flaky.

Sometimes, it seems to output text in a non-deterministic order (racy?):

--- /mnt/data/b/build/slave/WebKit_Linux/build/layout-test-results/inspector-protocol/debugger-pause-dedicated-worker-expected.txt 
+++ /mnt/data/b/build/slave/WebKit_Linux/build/layout-test-results/inspector-protocol/debugger-pause-dedicated-worker-actual.txt 
@@ -1,6 +1,6 @@
 Started worker
+didEnableWorkerDebugging
 Worker created
-didEnableWorkerDebugging
 didConnectToWorker
 SUCCESS: Worker paused
 
Sometimes, it seems to output extra text (maybe creating an extra worker?):

--- /b/build/slave/WebKit_Linux__dbg_/build/layout-test-results/inspector-protocol/debugger-pause-dedicated-worker-expected.txt 
+++ /b/build/slave/WebKit_Linux__dbg_/build/layout-test-results/inspector-protocol/debugger-pause-dedicated-worker-actual.txt 
@@ -1,6 +1,8 @@
 Started worker
+Worker created
 Worker created
 didEnableWorkerDebugging
 didConnectToWorker
+didConnectToWorker
 SUCCESS: Worker paused

And sometimes (Linux Debug), it crashes with useless stack trace:

crash log for DumpRenderTree (pid 29036):
STDOUT: <empty>
STDERR: <empty>
Comment 1 Yury Semikhatsky 2012-12-04 14:27:09 PST 
Created attachment 177563 [details]
Patch
Comment 2 Yury Semikhatsky 2012-12-04 14:30:26 PST 
Committed r136571: <http://trac.webkit.org/changeset/136571>