RESOLVED FIXED 147692
Add a second font-fallback performance test 
https://bugs.webkit.org/show_bug.cgi?id=147692
Summary Add a second font-fallback performance test
Myles C. Maxfield
Reported 2015-08-05 14:15:35 PDT
Add a second font-fallback performance test
Attachments
Patch (23.42 KB, patch)
2015-08-05 14:18 PDT, Myles C. Maxfield 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Myles C. Maxfield
Comment 1 2015-08-05 14:18:15 PDT
Created attachment 258300 [details] Patch
Ryosuke Niwa
Comment 2 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.
Myles C. Maxfield
Comment 3 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.
WebKit Commit Bot
Comment 4 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>
WebKit Commit Bot
Comment 5 2015-08-05 15:58:09 PDT
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.