Bug 147692 - Add a second font-fallback performance test
Summary: Add a second font-fallback performance test
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Myles C. Maxfield
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-08-05 14:15 PDT by Myles C. Maxfield
Modified: 2015-08-05 15:58 PDT (History)
2 users (show)

See Also:

Attachments
Patch (23.42 KB, patch)
2015-08-05 14:18 PDT, Myles C. Maxfield 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.