RESOLVED FIXED 149626
Use separate style resolver for user agent shadow trees
https://bugs.webkit.org/show_bug.cgi?id=149626
Summary Use separate style resolver for user agent shadow trees
Antti Koivisto
Reported 2015-09-29 08:32:23 PDT
We now support separate style resolvers for shadow trees. Use this mechanism to have a separate per-document style resolver for user agent shadow trees. This isolates user agent shadow trees from author style better and simplifies the style resolver. It also avoids some unnecessary style recalcs.
Attachments
patch (31.76 KB, patch)
2015-09-29 09:33 PDT, Antti Koivisto
kling: review+
buildbot: commit-queue-
Archive of layout-test-results from ews103 for mac-mavericks (832.07 KB, application/zip)
2015-09-29 10:31 PDT, Build Bot
no flags
Archive of layout-test-results from ews105 for mac-mavericks-wk2 (874.50 KB, application/zip)
2015-09-29 10:42 PDT, Build Bot
no flags
patch (33.36 KB, patch)
2015-09-30 06:15 PDT, Antti Koivisto
no flags
Antti Koivisto
Comment 1 2015-09-29 09:33:43 PDT
Andreas Kling
Comment 2 2015-09-29 10:22:50 PDT
Comment on attachment 262075 [details] patch r=me
Darin Adler
Comment 3 2015-09-29 10:24:59 PDT
Comment on attachment 262075 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=262075&action=review > Source/WebCore/dom/AuthorStyleSheets.cpp:355 > + Vector<RefPtr<CSSStyleSheet>> newStyleSheets; > + newStyleSheets.appendRange(activeStyleSheets.begin() + firstNewIndex, activeStyleSheets.end()); I think there’s a Vector constructor we can use instead of making an empty vector and then appending.
Build Bot
Comment 4 2015-09-29 10:31:01 PDT
Comment on attachment 262075 [details] patch Attachment 262075 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/224063 New failing tests: svg/custom/non-scaling-stroke.svg svg/custom/use-css-no-effect-on-shadow-tree.svg svg/W3C-SVG-1.1-SE/struct-use-11-f.svg svg/css/width-height-presentation-attribute.svg
Build Bot
Comment 5 2015-09-29 10:31:04 PDT
Created attachment 262077 [details] Archive of layout-test-results from ews103 for mac-mavericks The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews103 Port: mac-mavericks Platform: Mac OS X 10.9.5
Build Bot
Comment 6 2015-09-29 10:42:02 PDT
Comment on attachment 262075 [details] patch Attachment 262075 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/224087 New failing tests: svg/custom/non-scaling-stroke.svg svg/custom/use-css-no-effect-on-shadow-tree.svg svg/W3C-SVG-1.1-SE/struct-use-11-f.svg svg/css/width-height-presentation-attribute.svg
Build Bot
Comment 7 2015-09-29 10:42:05 PDT
Created attachment 262078 [details] Archive of layout-test-results from ews105 for mac-mavericks-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews105 Port: mac-mavericks-wk2 Platform: Mac OS X 10.9.5
Antti Koivisto
Comment 8 2015-09-30 06:15:36 PDT
Antti Koivisto
Comment 9 2015-09-30 08:42:17 PDT
Note You need to log in before you can comment on or make changes to this bug.