Unreviewed, rolling out r232052.
[WebKit-https.git] / PerformanceTests / ChangeLog
1 2018-05-22  Ryan Haddad  <ryanhaddad@apple.com>
2
3         Unreviewed, rolling out r232052.
4
5         Breaks internal builds.
6
7         Reverted changeset:
8
9         "Use more C++17"
10         https://bugs.webkit.org/show_bug.cgi?id=185176
11         https://trac.webkit.org/changeset/232052
12
13 2018-05-21  Yusuke Suzuki  <utatane.tea@gmail.com>
14
15         Use more C++17
16         https://bugs.webkit.org/show_bug.cgi?id=185176
17
18         Reviewed by JF Bastien.
19
20         * MallocBench/MallocBench.xcodeproj/project.pbxproj:
21         * MediaTime/Configurations/Base.xcconfig:
22         * MediaTime/MediaTime.xcodeproj/project.pbxproj:
23
24 2018-04-30  Myles C. Maxfield  <mmaxfield@apple.com>
25
26         Improve the performance of FontCascadeDescription's effectiveFamilies
27         https://bugs.webkit.org/show_bug.cgi?id=184720
28
29         Reviewed by Simon Fraser.
30
31         This performance test calls SystemFontDatabase::systemFontCascadeList() around 2,000,000 times (before
32         this patch is applied), which is roughly equivalent to the page we found the performance problem on.
33         The calling pattern is roughly equivalent in this test.
34
35         * Layout/system-ui.html: Added.
36
37 2018-03-08  Antti Koivisto  <antti@apple.com>
38
39         Update StyleBench version number in page title to 0.3
40
41         * StyleBench/index.html:
42
43         There has been a bunch of changes, most notably the addition of attribute selector test step.
44
45 2018-02-16  Dean Jackson  <dino@apple.com>
46
47         Use OPENGL macros to be more clear about which OpenGL/ES WebGL uses on Cocoa
48         https://bugs.webkit.org/show_bug.cgi?id=182894
49
50         Reviewed by Tim Horton.
51
52         Rename OPENGL_ES_2 to OPENGL_ES.
53
54         * StitchMarker/wtf/Platform.h:
55
56 2018-02-09  Ryosuke Niwa  <rniwa@webkit.org>
57
58         Make run-perf-tests work with Speedometer 2.0 and re-enable the test
59         https://bugs.webkit.org/show_bug.cgi?id=182088
60
61         Reviewed by Antti Koivisto.
62
63         This patch updates the test harness to report new metrics of Speedometer 2.0.
64
65         Replaced the total time by the geometric mean of the time spent in each suite, and added scores
66         as a metric in addition to time. Also, updated the PerfTestRunner harness and the results template
67         to support "Score" metric with "pt" as its unit.
68
69         * Skipped: Unskipped the test.
70         * Speedometer/resources/benchmark-report.js:
71         (window.benchmarkClient.willStartFirstIteration..createTest): Added the unit as an optional argument
72         to report the score in addition to time. Also report the geomean of time instead of the total time.
73         (window.benchmarkClient.willStartFirstIteration): Report the geomean of time.
74         (window.benchmarkClient.didRunSuites): Record the entire measuredValues instead of just tests.
75         (window.benchmarkClient.didFinishLastIteration): Report the scores.
76
77 2018-02-09  Ross Kirsling  <ross.kirsling@sony.com>
78
79         Use REQUEST_ANIMATION_FRAME_DISPLAY_MONITOR on any non-Windows port.
80         https://bugs.webkit.org/show_bug.cgi?id=182623
81
82         Reviewed by Alex Christensen.
83
84         * StitchMarker/wtf/Platform.h:
85         Simplify #if.
86
87 2018-02-01  Antti Koivisto  <antti@apple.com>
88
89         StyleBench: Attribute selectors and other improvements
90         https://bugs.webkit.org/show_bug.cgi?id=182387
91
92         Reviewed by Joseph Pecoraro.
93
94         - Add some attributes to elements in all tests
95         - Add some attribute selectors to stylesheets in all tests
96         - Also add some * selectors to all stylesheets.
97         - Add attribute mutation step to all suites
98         - Make test steps do more mutations (25->100) and reduce the number of steps to keep testing time in check.
99           Too fast steps were running into timer resolution limits.
100
101         * StyleBench/resources/style-bench.js:
102         (defaultConfiguration):
103         (prototype.randomAttributeName):
104         (prototype.randomAttributeValue):
105         (prototype.randomAttributeSelector):
106         (prototype.makeCompoundSelector):
107         (prototype.makeElement):
108         (prototype.addClasses):
109         (prototype.removeClasses):
110         (prototype.mutateAttributes):
111         (prototype.async.runForever):
112         * StyleBench/resources/tests.js:
113         (makeSteps):
114         (makeSuite):
115
116 2018-02-01  Geoffrey Garen  <ggaren@apple.com>
117
118         Make MallocBench easier for non-WebKit engineers to run
119         https://bugs.webkit.org/show_bug.cgi?id=182415
120
121         Reviewed by Saam Barati.
122
123         * MallocBench/MallocBench.xcodeproj/project.pbxproj: Use c++14 so we
124         can make_unique.
125
126         Specify that we support all Darwin platforms so you can test them.
127
128         * MallocBench/run-malloc-benchmarks: Specify the path to MallocBench
129         and libmbmalloc explicitly, rather than computing them implicitly
130         using webkitdirs. Non-WebKit folks don't have the directory structure
131         required by webkitdirs.
132
133         Remove Linux-specific and cmake-specific behaviors because we only
134         needed them in the world of implicit path computation.
135
136 2018-01-31  Ryosuke Niwa  <rniwa@webkit.org>
137
138         Make run-perf-tests work with StyleBench and re-enable it
139         https://bugs.webkit.org/show_bug.cgi?id=182322
140
141         Reviewed by Antti Koivisto.
142
143         This patch updates the test harness to report new metrics of StyleBench.
144
145         Replaced the total time by the geometric mean of the time spent in each suite, and added scores
146         as a metric in addition to time. Also, updated the PerfTestRunner harness and the results template
147         to support "Score" metric with "pt" as its unit.
148
149         * Skipped: Unskipped the test.
150         * StyleBench/resources/benchmark-report.js:
151         (window.benchmarkClient.willStartFirstIteration..createTest): Added the unit as an optional argument
152         to report the score in addition to time. Also report the geomean of time instead of the total time.
153         (window.benchmarkClient.willStartFirstIteration): Report the geomean of time.
154         (window.benchmarkClient.didRunSuites): Record the entire measuredValues instead of just tests.
155         (window.benchmarkClient.didFinishLastIteration): Report the scores.
156         * resources/results-template.html: Added "Score" as a metric and "pt" as its unit.
157         * resources/runner.js:
158         (PerfTestRunner..finish): Ditto.
159
160 2018-01-30  Ryosuke Niwa  <rniwa@webkit.org>
161
162         StyleBench: Generate more mutation steps
163         https://bugs.webkit.org/show_bug.cgi?id=182291
164
165         Rubber-stamped by Antti Koivisto
166
167         Use zero-based index instead of one-based index.
168
169         * StyleBench/resources/tests.js:
170         (makeSteps):
171
172 2018-01-30  Antti Koivisto  <antti@apple.com>
173
174         StyleBench: Generate more mutation steps
175         https://bugs.webkit.org/show_bug.cgi?id=182291
176
177         Reviewed by Ryosuke Niwa.
178
179         The test should be doing multiple rounds of mutations in a single document to test what is supposed to test (like it did before r227756).
180         This also makes test results more stable and less dependent on possible first-time costs.
181
182         * StyleBench/resources/benchmark-runner.js:
183
184         Adjust correctionFactor to match the new steps count. With 1.5 Firefox is ~72rpm.
185
186         * StyleBench/resources/tests.js:
187         (makeSteps):
188
189         Generate 10 rounds of class/element mutation steps.
190         Give each generated step unique name so reporting works.
191         Make individual steps do fewer mutations.
192
193 2018-01-29  Antti Koivisto  <antti@apple.com>
194
195         StyleBench: Remove : from a test name
196         https://bugs.webkit.org/show_bug.cgi?id=182258
197
198         Reviewed by Ryosuke Niwa.
199
200         It breaks reporting.
201
202         * StyleBench/resources/style-bench.js:
203         (nthPseudoClassConfiguration):
204
205 2018-01-29  Ryosuke Niwa  <rniwa@webkit.org>
206
207         Temporarily skip StyleBench until webkit.org/b/182088 is fixed.
208
209         * Skipped:
210
211 2018-01-29  Ryosuke Niwa  <rniwa@webkit.org>
212
213         Make StyleBench compatible with run-benchmark and run-perf-tests
214         https://bugs.webkit.org/show_bug.cgi?id=182262
215
216         Reviewed by Antti Koivisto.
217
218         Copied resource files referenced from Speedometer directory since run-benchmark needs to be able
219         to checkout each benchmark separately.
220
221         Removed the code to create tests of the same name five times in makeSteps since this
222         won't be compatible with either run-benchmark or run-perf-tests.
223
224         * StyleBench/index.html: Removed the code to show warnings for local files since run-benchmark
225         doesn't use HTTP server in WebDriver mode.
226         * StyleBench/resources/benchmark-report.js: Copied from resources/benchmark-report.js.
227         * StyleBench/resources/benchmark-runner.js: Copied from resources/benchmark-runner.js.
228         (BenchmarkRunner.prototype._finalize): Use the correction factor of 8 instead of 5 in StyleBench.
229         * StyleBench/resources/gauge.png: Copied from resources/gauge.png.
230         * StyleBench/resources/gauge@2x.png: Copied from resources/gauge@2x.png.
231         * StyleBench/resources/main.css: Copied from resources/main.css.
232         * StyleBench/resources/main.js: Copied from resources/main.js.
233         * StyleBench/resources/tests.js:
234         (makeSteps): Only make each test once.
235
236 2018-01-26  Chris Nardi  <cnardi@chromium.org>
237
238         Addressing post-review comments after r226614
239         https://bugs.webkit.org/show_bug.cgi?id=182151
240
241         Reviewed by Myles C. Maxfield.
242
243         * StitchMarker/wtf/text/StringImpl.h:
244         (WTF::isSpaceOrNewline):
245         * StitchMarker/wtf/text/TextBreakIterator.cpp:
246         (WTF::numCodeUnitsInGraphemeClusters):
247         * StitchMarker/wtf/text/TextBreakIterator.h:
248
249 2018-01-22  Antti Koivisto  <antti@apple.com>
250
251         StyleBench: Separate test for :nth pseudo classes
252         https://bugs.webkit.org/show_bug.cgi?id=181932
253
254         Reviewed by Sam Weinig.
255
256         Optimization strategies for nth are different from other structural pseudo classes (:first-child etc). Move them to a separate test.
257
258         * StyleBench/resources/style-bench.js:
259         (structuralPseudoClassConfiguration):
260         (nthPseudoClassConfiguration):
261
262             Add a new configuration.
263
264         (predefinedConfigurations):
265         (pseudoClassConfiguration): Deleted.
266
267 2018-01-17  Ryosuke Niwa  <rniwa@webkit.org>
268
269         Temporarily skip Speedometer until we fix the metrics.
270
271         * Skipped:
272
273 2018-01-16  Alex Christensen  <achristensen@webkit.org>
274
275         Fix Mac CMake build after r222900
276
277         * MallocBench/MallocBench/CMakeLists.txt:
278         The mbmalloc target doesn't exist if we're not in developer mode, so only use it if we are.
279
280 2018-01-09  Ryosuke Niwa  <rniwa@webkit.org>
281
282         Speedometer 2.0: Update the About page.
283         https://bugs.webkit.org/show_bug.cgi?id=175918
284
285         Reviewed by Keith Miller.
286
287         Updated the about page for Speedometer 2.0.
288
289         * Speedometer/index.html:
290
291 == Rolled over to ChangeLog-2018-01-01 ==