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

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

Issue 112303003: IDL compiler: [Constructor] overloading (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Formatting Created 7 years 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/bindings/tests/idls/TestInterfaceConstructor2.idl
diff --git a/Source/bindings/tests/idls/TestInterfaceConstructor.idl b/Source/bindings/tests/idls/TestInterfaceConstructor2.idl
similarity index 77%
copy from Source/bindings/tests/idls/TestInterfaceConstructor.idl
copy to Source/bindings/tests/idls/TestInterfaceConstructor2.idl
index ef46b8a1387a0b567dae78a2ec07c174cc6135a4..a698e11671e3a6ea1ce906671ce0cfdcdaf40be9 100644
--- a/Source/bindings/tests/idls/TestInterfaceConstructor.idl
+++ b/Source/bindings/tests/idls/TestInterfaceConstructor2.idl
@@ -28,20 +28,20 @@
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
+// 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.
+//
+// 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).
[
+ Constructor(DOMString stringArg),
Constructor(
- double doubleArg,
- DOMString stringArg,
TestInterfaceEmpty testInterfaceEmptyArg,
- Dictionary dictionaryArg,
- sequence<DOMString> sequenceStringArg,
- optional Dictionary optionalDictionaryArg,
[Default=Undefined] optional DOMString defaultUndefinedOptionalStringArg,
[Default=NullString] optional DOMString defaultNullStringOptionalStringArg,
- [Default=Undefined] optional TestInterfaceEmpty optionalTestInterfaceEmptyArg,
[Default=Undefined] optional Dictionary defaultUndefinedOptionalDictionaryArg),
- ConstructorCallWith=ExecutionContext&Document,
- MeasureAs=TestFeature,
- RaisesException=Constructor,
-] interface TestInterfaceConstructor {
+] interface TestInterfaceConstructor2 {
};
« no previous file with comments | « Source/bindings/tests/idls/TestInterfaceConstructor.idl ('k') | Source/bindings/tests/idls/TestInterfaceConstructor3.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698