Bug 128821

Summary: [CSS Shapes] Add performance test for stacked floats with shape-outsides
Product: WebKit Reporter: Zoltan Horvath <zoltan>
Component: Tools / TestsAssignee: Zoltan Horvath <zoltan>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 128188    
Attachments:
Description Flags
Patch none

Zoltan Horvath
Reported 2014-02-14 09:34:37 PST
Add performance test for stacked floats with shape-outsides.
Attachments
Patch (1.60 KB, patch)
2014-02-18 13:48 PST, Zoltan Horvath 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Zoltan Horvath
Comment 1 2014-02-18 13:48:08 PST
Created attachment 224540 [details] Patch
Brent Fulgham
Comment 2 2014-02-19 11:29:32 PST
Comment on attachment 224540 [details] Patch r=me. When do you plan on turning this on?
Zoltan Horvath
Comment 3 2014-02-19 14:42:28 PST
(In reply to comment #2) > (From update of attachment 224540 [details]) > r=me. When do you plan on turning this on? I'm going to come up with an integrated shapes+regions test soon. These tests are micro benchmarks, good for verify the performance impact for our code changes. As per my discussion with Ryosuke, they would require too much time on the perf.tester bot, so we can't enable them by default. I'm planning to set up an internal tester to run these continuously though.
WebKit Commit Bot
Comment 4 2014-02-19 15:17:30 PST
Comment on attachment 224540 [details] Patch Clearing flags on attachment: 224540 Committed r164395: <http://trac.webkit.org/changeset/164395>
WebKit Commit Bot
Comment 5 2014-02-19 15:17:32 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.