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

Unified Diff: Source/core/css/CSSStyleDeclaration.idl

Issue 1054303004: Sync CSS interfaces with the specs (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 9 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 | « Source/core/css/CSSRuleList.idl ('k') | Source/core/css/CSSStyleRule.idl » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/css/CSSStyleDeclaration.idl
diff --git a/Source/core/css/CSSStyleDeclaration.idl b/Source/core/css/CSSStyleDeclaration.idl
index 4def0755528ded19e7d79236cb94336da647bca6..24fdfbb82bdee6a9d0905e8efd42c79a46506aac 100644
--- a/Source/core/css/CSSStyleDeclaration.idl
+++ b/Source/core/css/CSSStyleDeclaration.idl
@@ -18,24 +18,32 @@
* Boston, MA 02110-1301, USA.
*/
-// Introduced in DOM Level 2:
+// http://dev.w3.org/csswg/cssom/#the-cssstyledeclaration-interface
+
[
DependentLifetime,
WillBeGarbageCollected
] interface CSSStyleDeclaration {
- [RaisesException=Setter] attribute DOMString? cssText;
-
- DOMString? getPropertyValue(DOMString propertyName);
- [RaisesException] DOMString? removeProperty(DOMString propertyName);
- DOMString? getPropertyPriority(DOMString propertyName);
- [RaisesException] void setProperty(DOMString propertyName,
- DOMString? value,
- optional DOMString? priority = null);
+ // TODO(philipj): cssText should not be nullable.
+ [RaisesException=Setter] attribute DOMString? cssText;
+ readonly attribute unsigned long length;
+ getter DOMString item(unsigned long index);
+ // TODO(philipj): All DOMString? return types in this interface should be
+ // just DOMString (not nullable).
+ DOMString? getPropertyValue(DOMString property);
+ DOMString? getPropertyPriority(DOMString property);
+ // TODO(philipj): The value and priority arguments should have
+ // [TreatNullAs=EmptyString] and should not be nullable.
+ [RaisesException] void setProperty(DOMString property, DOMString? value, optional DOMString? priority = null);
+ // void setPropertyValue(DOMString property, [TreatNullAs=EmptyString] DOMString value);
+ // void setPropertyPriority(DOMString property, [TreatNullAs=EmptyString] DOMString priority);
+ [RaisesException] DOMString? removeProperty(DOMString property);
+ readonly attribute CSSRule? parentRule;
+ // [TreatNullAs=EmptyString] attribute DOMString cssFloat;
- readonly attribute unsigned long length;
- getter DOMString item(unsigned long index);
+ // The camel-cased and dashed attribute getters have custom bindings.
+ // http://dev.w3.org/csswg/cssom/#dom-cssstyledeclaration-camel-cased-attribute
+ // http://dev.w3.org/csswg/cssom/#dom-cssstyledeclaration-dashed-attribute
[Custom=(PropertyGetter,PropertyEnumerator,PropertyQuery)] getter (DOMString or float) (DOMString name);
- [Custom] setter void (DOMString propertyName, DOMString? propertyValue);
- readonly attribute CSSRule parentRule;
+ [Custom] setter void (DOMString property, DOMString? propertyValue);
};
-
« no previous file with comments | « Source/core/css/CSSRuleList.idl ('k') | Source/core/css/CSSStyleRule.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698