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

Unified Diff: bindings/tests/idls/TestInterfaceConstructor2.idl

Issue 540533002: Roll IDL to Dartium37 (r181268) (Closed) Base URL: https://dart.googlecode.com/svn/third_party/WebCore
Patch Set: Created 6 years, 3 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 | « bindings/tests/idls/TestInterfaceConstructor.idl ('k') | bindings/tests/idls/TestInterfaceConstructor3.idl » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: bindings/tests/idls/TestInterfaceConstructor2.idl
diff --git a/core/fileapi/Stream.idl b/bindings/tests/idls/TestInterfaceConstructor2.idl
similarity index 60%
copy from core/fileapi/Stream.idl
copy to bindings/tests/idls/TestInterfaceConstructor2.idl
index 1406b7bbde87de78d763db03be171e39e6c5e83f..10057de6dcce46770bf791f10f60f1fa9ba89864 100644
--- a/core/fileapi/Stream.idl
+++ b/bindings/tests/idls/TestInterfaceConstructor2.idl
@@ -28,17 +28,24 @@
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
-// FIXME: close() method which is necessary for Stream building feature
-// is omitted for now. Add it when necessary.
+// Test for length > 0, overloaded constructor.
+// Need separate tests for constructors with length == 0 and length > 0,
+// and for overloaded vs. non-overloaded, if length > 0.
+// length == 0, non-overloaded just omits a block and is not worth another test.
//
-// FIXME: Add a flag to indicate if this stream frees memory when read or not
-// (e.g. boolean isReadOnce()).
-//
-// FIXME: Make the Blob a subclass of the Stream.
-
+// Also includes some [Default] arguments (instead of in
+// TestInterfaceConstructor.idl), otherwise overload resolution check string is
+// extremely long and triggers a lint warning (line length).
[
- RuntimeEnabled=Stream,
- ActiveDOMObject
-] interface Stream {
- readonly attribute DOMString type;
+ // 2 constructors with same type length, to test overload resolution
+ Constructor(DOMString stringArg),
+ Constructor(Dictionary dictionaryArg),
+ Constructor(
+ TestInterfaceEmpty testInterfaceEmptyArg,
+ long longArg,
+ [Default=Undefined] optional DOMString defaultUndefinedOptionalStringArg,
+ optional DOMString defaultNullStringOptionalStringArg = null,
+ [Default=Undefined] optional Dictionary defaultUndefinedOptionalDictionaryArg,
+ optional DOMString optionalStringArg),
+] interface TestInterfaceConstructor2 {
};
« no previous file with comments | « bindings/tests/idls/TestInterfaceConstructor.idl ('k') | bindings/tests/idls/TestInterfaceConstructor3.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698