Summary: | Add internal benchmark tests for CSS mix-blend-modes and filters | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Said Abou-Hallawa <sabouhallawa> | ||||
Component: | Animations | Assignee: | Said Abou-Hallawa <sabouhallawa> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | cdumez, commit-queue, dino, rniwa | ||||
Priority: | P2 | ||||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Said Abou-Hallawa
2016-02-09 17:52:26 PST
Created attachment 270966 [details]
Patch
Comment on attachment 270966 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=270966&action=review Unofficial r=me > PerformanceTests/Animometer/resources/extensions.js:87 > + browserPrefix: function () No space between function and (). This result is derived and never cached; seems better to rename this to getBrowserPrefix()? > PerformanceTests/Animometer/resources/extensions.js:93 > + .match(/-(moz|webkit|ms)-/) || (styles.OLink === '' && ['', 'o']) This is hard to read. Can we unfold this using if clauses instead? Committed r196381: <http://trac.webkit.org/changeset/196381> |