Bug 113811 – Need japanese line layout performance test

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Glenn Adams]

For use in testing performance regression related to upcoming Japanese related line break patches.

[Glenn Adams]

Created attachment 196200 [details]
Patch

[Ryosuke Niwa]

Comment on attachment 196200 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=196200&action=review

> PerformanceTests/ChangeLog:9
> +        Will be used by https://bugs.webkit.org/show_bug.cgi?id=105692 to test potential performance regression
> +        for Japanese (and other CJK) line break changes.

Is this test stable? Could you post sample results from this test?
e.g. you can simply upload WebKitBuild/Release/PerformanceTestsResults.html after repeatedly running this test for 3-5 times.

Also, is it really worth running this test on bot? The current cycle time of bots is roughly 2 hours so I'd rather not enable this test by default unless this test is going to be very effective in detecting real regressions.

> PerformanceTests/Layout/LineLayoutJapanese.html:4
> +    <meta charset="utf-8" />

We don't explicitly close self-closing tag in HTML5.

[Glenn Adams]

Created attachment 196232 [details]
run-perf-tests results

Test run of PerformanceTests/Layout/LineLayoutJapanese.html on MBP Retina 15" (16GB RAM, 512G SSD).

[Glenn Adams]

Created attachment 196240 [details]
Patch

[Glenn Adams]

(In reply to comment #2)
> (From update of attachment 196200 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=196200&action=review
> 
> > PerformanceTests/ChangeLog:9
> > +        Will be used by https://bugs.webkit.org/show_bug.cgi?id=105692 to test potential performance regression
> > +        for Japanese (and other CJK) line break changes.
> 
> Is this test stable? Could you post sample results from this test?
> e.g. you can simply upload WebKitBuild/Release/PerformanceTestsResults.html after repeatedly running this test for 3-5 times.

Yes. Posted.

> 
> Also, is it really worth running this test on bot? The current cycle time of bots is roughly 2 hours so I'd rather not enable this test by default unless this test is going to be very effective in detecting real regressions.

Added to Skipped.

> 
> > PerformanceTests/Layout/LineLayoutJapanese.html:4
> > +    <meta charset="utf-8" />
> 
> We don't explicitly close self-closing tag in HTML5.

Fixed.

[WebKit Review Bot]

Comment on attachment 196240 [details]
Patch

Clearing flags on attachment: 196240

Committed r147507: <http://trac.webkit.org/changeset/147507>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.