Bug 157697

Summary: Fix JSBench cause math domain error in run-benchmark script.
Product: WebKit Reporter: dewei_zhu
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, commit-queue, glenn, rniwa, slewis
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ews103 for mac-yosemite
none
Patch for landing none

Description dewei_zhu 2016-05-13 17:34:43 PDT
Fix JSBench cause math domain error in run-benchmark script.
Comment 1 dewei_zhu 2016-05-13 17:38:47 PDT
Created attachment 278900 [details]
Patch
Comment 2 Build Bot 2016-05-13 18:19:01 PDT
Comment on attachment 278900 [details]
Patch

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

New failing tests:
transitions/default-timing-function.html
Comment 3 Build Bot 2016-05-13 18:19:04 PDT
Created attachment 278904 [details]
Archive of layout-test-results from ews103 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 4 Ryosuke Niwa 2016-05-13 19:49:58 PDT
Comment on attachment 278900 [details]
Patch

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

> Tools/Scripts/webkitpy/benchmark_runner/benchmark_results.py:92
> +        # Be careful about round-off error when sample_stdev is 0.

Nit: Wrong indentation.
Comment 5 dewei_zhu 2016-05-13 21:08:25 PDT
Created attachment 278919 [details]
Patch for landing
Comment 6 Ryosuke Niwa 2016-05-13 21:16:37 PDT
Stephanie, could you merge this change back into your TA setup?
Comment 7 WebKit Commit Bot 2016-05-13 21:57:48 PDT
Comment on attachment 278919 [details]
Patch for landing

Clearing flags on attachment: 278919

Committed r200914: <http://trac.webkit.org/changeset/200914>
Comment 8 WebKit Commit Bot 2016-05-13 21:57:53 PDT
All reviewed patches have been landed.  Closing bug.