RESOLVED FIXED 196243
Primary cluster of measurement set should always contain latest point. 
https://bugs.webkit.org/show_bug.cgi?id=196243
Summary Primary cluster of measurement set should always contain latest point.
dewei_zhu
Reported 2019-03-25 23:07:33 PDT
Primary cluster of measurement set should always contain latest point.
Attachments
Patch (10.44 KB, patch)
2019-03-25 23:09 PDT, dewei_zhu 		rniwa: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
dewei_zhu
Comment 1 2019-03-25 23:09:49 PDT
Created attachment 365948 [details] Patch
Ryosuke Niwa
Comment 2 2019-03-26 11:33:27 PDT
Comment on attachment 365948 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=365948&action=review > Websites/perf.webkit.org/server-tests/api-measurement-set-tests.js:468 > + const currentRows = response['configurations']['current']; Can we also check that these times are earlier than the cluster start time, etc...?
Ryosuke Niwa
Comment 3 2019-03-26 11:35:09 PDT
That is to make sure that the last data point was actually not within the time range of the primary cluster. This is to prevent the test case from being broken in the future when someone refractors it modifies the test for something else.
dewei_zhu
Comment 4 2019-03-26 12:19:42 PDT
Tracked in <rdar://problem/49285265>. Landed in r243510.
Note You need to log in before you can comment on or make changes to this bug.