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

Unified Diff: Source/core/svg/SVGNumberList.idl

Issue 141523002: Add null-correctness checking to [StrictTypeChecking] methods (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 11 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
Index: Source/core/svg/SVGNumberList.idl
diff --git a/Source/core/svg/SVGNumberList.idl b/Source/core/svg/SVGNumberList.idl
index 4711cc66379a1f67163f0cb1b373e5398807ee52..254e5a9d820ec515b172ecaeb836104220cd1334 100644
--- a/Source/core/svg/SVGNumberList.idl
+++ b/Source/core/svg/SVGNumberList.idl
@@ -28,11 +28,11 @@ interface SVGNumberList {
readonly attribute unsigned long numberOfItems;
[RaisesException] void clear();
- [StrictTypeChecking, RaisesException] SVGNumber initialize(SVGNumber item);
+ [StrictTypeChecking, RaisesException] SVGNumber initialize(SVGNumber? item);
kouhei (in TOK) 2014/01/17 04:28:45 No ? here.
[StrictTypeChecking, RaisesException] SVGNumber getItem(unsigned long index);
- [StrictTypeChecking, RaisesException] SVGNumber insertItemBefore(SVGNumber item, unsigned long index);
- [StrictTypeChecking, RaisesException] SVGNumber replaceItem(SVGNumber item, unsigned long index);
+ [StrictTypeChecking, RaisesException] SVGNumber insertItemBefore(SVGNumber? item, unsigned long index);
kouhei (in TOK) 2014/01/17 04:28:45 No ? here.
+ [StrictTypeChecking, RaisesException] SVGNumber replaceItem(SVGNumber? item, unsigned long index);
kouhei (in TOK) 2014/01/17 04:28:45 No ? here.
[StrictTypeChecking, RaisesException] SVGNumber removeItem(unsigned long index);
- [StrictTypeChecking, RaisesException] SVGNumber appendItem(SVGNumber item);
+ [StrictTypeChecking, RaisesException] SVGNumber appendItem(SVGNumber? item);
};
kouhei (in TOK) 2014/01/17 04:28:45 No ? here.

Powered by Google App Engine
This is Rietveld 408576698