WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
154277
Extract the code specific to v2 UI out of shared statistics.js
https://bugs.webkit.org/show_bug.cgi?id=154277
Summary
Extract the code specific to v2 UI out of shared statistics.js
Ryosuke Niwa
Reported
2016-02-15 20:39:42 PST
Extract the code specific to v2 UI (and the current implementation of detect-changes.js) out of statistics.js This re-factoring is needed to support statistical tools in v3 UI.
Attachments
Cleanup
(34.67 KB, patch)
2016-02-15 20:54 PST
,
Ryosuke Niwa
cdumez
: review+
cdumez
: commit-queue-
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1
2016-02-15 20:54:18 PST
Created
attachment 271412
[details]
Cleanup
Chris Dumez
Comment 2
2016-02-16 13:39:15 PST
Comment on
attachment 271412
[details]
Cleanup View in context:
https://bugs.webkit.org/attachment.cgi?id=271412&action=review
rs=me
> Websites/perf.webkit.org/ChangeLog:8 > + Extracted statistics-strategies.js out of statistics.js for v2 UI and detect-changes.js. The indent is to
"intent" ?
> Websites/perf.webkit.org/public/shared/statistics.js:136 > + this.cumultaiveMovingAverage = function (values) {
"cumulative".
> Websites/perf.webkit.org/public/v2/statistics-strategies.js:21 > + execute: Statistics.cumultaiveMovingAverage,
"cumulative"
Ryosuke Niwa
Comment 3
2016-02-16 15:16:21 PST
Committed
r196663
: <
http://trac.webkit.org/changeset/196663
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug