Bug 173940 – Fix for intermittent Layout Test fail http/tests/loading/resourceLoadStatistics/telemetry-generation.html

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit2

[John Wilander]

The regular resource load statistics telemetry handler fires, which triggers early TestRunner callbacks and test failures. We should turn off regular telemetry in layout tests.

[John Wilander]

rdar://problem/33018125

[John Wilander]

Created attachment 314065 [details]
Patch

[Build Bot]

Comment on attachment 314065 [details]
Patch

Attachment 314065 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/4016614

New failing tests:
media/video-no-audio.html

[Build Bot]

Created attachment 314073 [details]
Archive of layout-test-results from ews107 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6

[John Wilander]

Comment on attachment 314065 [details]
Patch

Had a chat with Jer Noble and the test failure is unrelated according to him.

[Brent Fulgham]

Comment on attachment 314065 [details]
Patch

r=me

[WebKit Commit Bot]

Comment on attachment 314065 [details]
Patch

Rejecting attachment 314065 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'apply-attachment', '--no-update', '--non-interactive', 314065, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
csManager.cpp
patching file Source/WebKit2/UIProcess/API/C/WKResourceLoadStatisticsManager.h
patching file Tools/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file Tools/WebKitTestRunner/TestController.cpp
patching file LayoutTests/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file LayoutTests/platform/wk2/TestExpectations

Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Brent Fulgham']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Full output: http://webkit-queues.webkit.org/results/4021744

[John Wilander]

Created attachment 314176 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 314176 [details]
Patch for landing

Clearing flags on attachment: 314176

Committed r218965: <http://trac.webkit.org/changeset/218965>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[John Wilander]

Committed r218995: <http://trac.webkit.org/changeset/218995>

[John Wilander]

Last commit updated expectations with failure again. Haven't looked into why it fails now. Tested 500 iterations on my machine with no failure.