Bug 147692

Summary: Add a second font-fallback performance test
Product: WebKit Reporter: Myles C. Maxfield <mmaxfield>
Component: New BugsAssignee: Myles C. Maxfield <mmaxfield>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Myles C. Maxfield 2015-08-05 14:15:35 PDT 
Add a second font-fallback performance test
Comment 1 Myles C. Maxfield 2015-08-05 14:18:15 PDT 
Created attachment 258300 [details]
Patch
Comment 2 Ryosuke Niwa 2015-08-05 14:38:57 PDT 
Do we really need two separate tests?  In general, the cycle time of perf bots is really long so it's not great to add more perf tests unless it adds substantial benefits.
Comment 3 Myles C. Maxfield 2015-08-05 14:44:05 PDT 
I think this test is valuable separately, since it is aimed at more realistic content. Combining the two doesn't make much sense since the previous one is just designed to hit a certain code path as much as possible.
Comment 4 WebKit Commit Bot 2015-08-05 15:58:06 PDT 
Comment on attachment 258300 [details]
Patch

Clearing flags on attachment: 258300

Committed r187995: <http://trac.webkit.org/changeset/187995>
Comment 5 WebKit Commit Bot 2015-08-05 15:58:09 PDT 
All reviewed patches have been landed.  Closing bug.