Bug 86216

Summary:

Add a line layout performance test

Product:

WebKit

Reporter:

mitz

Component:

Tools / Tests

Assignee:

mitz

Status:

RESOLVED FIXED

Severity:

Normal

CC:

rniwa

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Add a test that exercises line layout	rniwa: review+

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+	Details Formatted Diff Diff
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

Note You need to log in before you can comment on or make changes to this bug.