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

Unified Diff: Source/core/testing/UnionTypesTest.idl

Issue 713683003: IDL: Support optional union type arguments with default values (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: add unit tests Created 6 years, 1 month 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/testing/UnionTypesTest.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/testing/UnionTypesTest.idl
diff --git a/Source/core/testing/UnionTypesTest.idl b/Source/core/testing/UnionTypesTest.idl
index 282969ec181a78e8c79658e50afc11b3944c369f..efc8a1801ea0667c577082a3a8669366c03c1be1 100644
--- a/Source/core/testing/UnionTypesTest.idl
+++ b/Source/core/testing/UnionTypesTest.idl
@@ -8,4 +8,8 @@
attribute (double or DOMString) doubleOrStringAttribute;
DOMString doubleOrStringArg((double or DOMString) arg);
+
+ [ImplementedAs=doubleOrStringArg] DOMString doubleOrStringDefaultDoubleArg(optional (double or DOMString) arg = 3.14);
+ [ImplementedAs=doubleOrStringArg] DOMString doubleOrStringDefaultStringArg(optional (double or DOMString) arg = "foo");
+ [ImplementedAs=doubleOrStringArg] DOMString doubleOrStringDefaultNullArg(optional (double or DOMString)? arg = null);
haraken 2014/11/10 12:55:42 Can we add: DOMString doubleOrStringDefaultNull
Jens Widell 2014/11/10 12:59:10 Undefined is not a possible default value per spec
};
« no previous file with comments | « Source/core/testing/UnionTypesTest.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698