Commit c63f05dd authored by kling@webkit.org's avatar kling@webkit.org

Allow matched property cache for elements with additional attribute style.

<http://webkit.org/b/79583>

Reviewed by Antti Koivisto.

There's no reason to disallow the matched style property cache for elements
that return something from additionalAttributeStyle(). The only requirement
for a property set to be cached is that it either doesn't mutate OR that it
invalidates the document's CSSStyleSelector when doing so.

This allows some more match caching for table-related elements, though we
are still held back by explicitly 'inherited' properties in html.css.

* css/CSSStyleSelector.cpp:
(WebCore::CSSStyleSelector::collectMatchingRulesForList):


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@108916 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent c17bd051
2012-02-25 Andreas Kling <awesomekling@apple.com>
Allow matched property cache for elements with additional attribute style.
<http://webkit.org/b/79583>
Reviewed by Antti Koivisto.
There's no reason to disallow the matched style property cache for elements
that return something from additionalAttributeStyle(). The only requirement
for a property set to be cached is that it either doesn't mutate OR that it
invalidates the document's CSSStyleSelector when doing so.
This allows some more match caching for table-related elements, though we
are still held back by explicitly 'inherited' properties in html.css.
* css/CSSStyleSelector.cpp:
(WebCore::CSSStyleSelector::collectMatchingRulesForList):
2012-02-25 Julien Chaffraix <jchaffraix@webkit.org>
Clean-up RenderTableSection::calcRowLogicalHeight
......@@ -1043,7 +1043,6 @@ void CSSStyleSelector::matchAllRules(MatchResult& result)
result.ranges.firstAuthorRule = result.matchedProperties.size();
result.ranges.lastAuthorRule = result.ranges.firstAuthorRule;
addMatchedProperties(result, additionalStyle);
result.isCacheable = false;
}
if (m_styledElement->isHTMLElement()) {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment