Bug 111123

Summary:

Flaky test: http/tests/inspector/resource-tree/resource-request-content-while-loading.html

Product:

WebKit

Reporter:

Ryosuke Niwa <rniwa>

Component:

Web Inspector (Deprecated)

Assignee:

Vsevolod Vlasov <vsevik>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

apavlov, keishi, loislo, pfeldman, pmuellr, vsevik, web-inspector-bugs, yurys

Priority:

Keywords:

LayoutTestFailure

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	apavlov: review+

Description Ryosuke Niwa 2013-02-28 15:36:14 PST

http/tests/inspector/resource-tree/resource-request-content-while-loading.html is flaky

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20-%20webkit.org&showExpectations=true&tests=http%2Ftests%2Finspector%2Fresource-tree%2Fresource-request-content-while-loading.html

There appears to be a race condition between InspectorTest.pageLoaded and contentLoaded
but I can't tell because there's just way too much javascript code involved in this test.

Comment 1 Ryosuke Niwa 2013-02-28 15:40:10 PST

Added a test expectation in http://trac.webkit.org/changeset/144384.

Comment 2 Vsevolod Vlasov 2013-03-01 03:31:24 PST

Created attachment 190925 [details]
Patch

Comment 3 Alexander Pavlov (apavlov) 2013-03-01 03:36:07 PST

Comment on attachment 190925 [details]
Patch

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

> LayoutTests/http/tests/inspector/resource-tree/resource-request-content-while-loading.html:10
> +    styleElement.setAttribute("rel", "stylesheet");

You can rewrite this as styleElement.rel = "stylesheet"

> LayoutTests/http/tests/inspector/resource-tree/resource-request-content-while-loading.html:11
> +    styleElement.setAttribute("href", "resources/styles-initial.css");

Ditto for "href"

Comment 4 Vsevolod Vlasov 2013-03-01 03:38:55 PST

Committed r144435: <http://trac.webkit.org/changeset/144435>