Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(100)

Unified Diff: sky/engine/core/css/CSSProperty.cpp

Issue 1068393002: remove CSS 'all' property. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sky/engine/core/css/CSSProperty.h ('k') | sky/engine/core/css/parser/BisonCSSParser-in.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/engine/core/css/CSSProperty.cpp
diff --git a/sky/engine/core/css/CSSProperty.cpp b/sky/engine/core/css/CSSProperty.cpp
index ea33ca90b7fc289001371719ab1f09bd043fa6d7..2bc6f7a432a4e20402877f312ca7921bb436393e 100644
--- a/sky/engine/core/css/CSSProperty.cpp
+++ b/sky/engine/core/css/CSSProperty.cpp
@@ -172,18 +172,4 @@ CSSPropertyID CSSProperty::resolveDirectionAwareProperty(CSSPropertyID propertyI
}
}
-bool CSSProperty::isAffectedByAllProperty(CSSPropertyID propertyID)
-{
- if (propertyID == CSSPropertyAll)
- return false;
-
- // all shorthand spec says:
- // The all property is a shorthand that resets all CSS properties except
- // direction and unicode-bidi. It only accepts the CSS-wide keywords.
- // c.f. http://dev.w3.org/csswg/css-cascade/#all-shorthand
- // So CSSPropertyUnicodeBidi and CSSPropertyDirection are not
- // affected by all property.
- return propertyID != CSSPropertyUnicodeBidi && propertyID != CSSPropertyDirection;
-}
-
} // namespace blink
« no previous file with comments | « sky/engine/core/css/CSSProperty.h ('k') | sky/engine/core/css/parser/BisonCSSParser-in.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698