RESOLVED FIXED 86216
Add a line layout performance test 
https://bugs.webkit.org/show_bug.cgi?id=86216
Summary Add a line layout performance test
mitz
Reported 2012-05-11 07:31:45 PDT
Add a line-breaking performance test
Attachments
Add a test that exercises line layout (22.33 KB, patch)
2012-05-11 07:34 PDT, mitz 		rniwa: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2012-05-11 07:34:03 PDT
Created attachment 141416 [details] Add a test that exercises line layout
Ryosuke Niwa
Comment 2 2012-05-12 01:43:44 PDT
Comment on attachment 141416 [details] Add a test that exercises line layout View in context: https://bugs.webkit.org/attachment.cgi?id=141416&action=review Adding a line layout test will be great. Maybe we should add one with lots of bidi text as well? > PerformanceTests/Layout/line-layout.html:91 > + PerfTestRunner.run(test, 5, 10); Should we wait adding this test until https://bugs.webkit.org/show_bug.cgi?id=86021 is fixed? ojan says runs/s works better for these rendering perf. tests.
Ryosuke Niwa
Comment 3 2012-05-14 12:29:01 PDT
Comment on attachment 141416 [details] Add a test that exercises line layout Hi, so I've added runPerSecond in http://trac.webkit.org/changeset/116916. You can just use that function instead. PerfTestRunner.runPerSecond({run: test}).
mitz
Comment 4 2012-05-14 13:15:59 PDT
Thanks! Landed in <http://trac.webkit.org/r116989>.
Note You need to log in before you can comment on or make changes to this bug.