Change Uri.fromComponents to take named optional arguments.

tests/utils/uri_test.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #library('uriTest');
 
 #import('../../lib/utf/utf.dart');
 #import('../../lib/uri/uri.dart');
 
 testUri(String uri, bool isAbsolute) {
@@ skipping 75 matching lines @@
 
 main() {
   testUri("http:", true);
   testUri("file://", true);
   testUri("file", false);
   testUri("http://user@example.com:80/fisk?query=89&hest=silas", true);
   testUri("http://user@example.com:80/fisk?query=89&hest=silas#fragment",
           false);
   Expect.stringEquals("http://user@example.com:80/a/b/c?query#fragment",
                       const Uri.fromComponents(
-                          "http", "user", "example.com", 80, "/a/b/c",
-                          "query", "fragment").toString());
+                          scheme: "http",
+                          userInfo: "user",
+                          domain: "example.com",
+                          port: 80,
+                          path: "/a/b/c",
+                          query: "query",
+                          fragment: "fragment").toString());
   Expect.stringEquals("null://null@null/a/b/c/?null#null",
                       const Uri.fromComponents(
-                          null, null, null, 0, "/a/b/c/",
-                          null, null).toString());
+                          scheme: null,
+                          userInfo: null,
+                          domain: null,
+                          port: 0,
+                          path: "/a/b/c/",
+                          query: null,
+                          fragment: null).toString());
   Expect.stringEquals("file://", new Uri.fromString("file:").toString());
   Expect.stringEquals("file://", new Uri("file:").toString());
   Expect.stringEquals("/a/g", removeDotSegments("/a/b/c/./../../g"));
   Expect.stringEquals("mid/6", removeDotSegments("mid/content=5/../6"));
   Expect.stringEquals("a/b/e", removeDotSegments("a/b/c/d/../../e"));
   Expect.stringEquals("a/b/e", removeDotSegments("../a/b/c/d/../../e"));
   Expect.stringEquals("a/b/e", removeDotSegments("./a/b/c/d/../../e"));
   Expect.stringEquals("a/b/e", removeDotSegments("../a/b/./c/d/../../e"));
   Expect.stringEquals("a/b/e", removeDotSegments("./a/b/./c/d/../../e"));
   Expect.stringEquals("a/b/e/", removeDotSegments("./a/b/./c/d/../../e/."));
@@ skipping 16 matching lines @@
       "http://example.com:1234",
       new Uri.fromString("http://example.com:1234/a/b/c").origin);
   Expect.stringEquals(
       "https://example.com:1234",
       new Uri.fromString("https://example.com:1234/a/b/c").origin);
   Expect.throws(
       () => new Uri.fromString("http:").origin,
       (e) { return e is IllegalArgumentException; },
       "origin for uri with empty domain should fail");
   Expect.throws(
-      () => const Uri.fromComponents("http", null, "", 80, "/a/b/c",
-                       "query", "fragment").origin,
+      () => const Uri.fromComponents(
+          scheme: "http",
+          userInfo: null,
+          domain: "",
+          port: 80,
+          path: "/a/b/c",
+          query: "query",
+          fragment: "fragment").origin,
       (e) { return e is IllegalArgumentException; },
       "origin for uri with empty domain should fail");
   Expect.throws(
-      () => const Uri.fromComponents(null, null, "", 80, "/a/b/c",
-                       "query", "fragment").origin,
+      () => const Uri.fromComponents(
+          scheme: null,
+          userInfo: null,
+          domain: "",
+          port: 80,
+          path: "/a/b/c",
+          query: "query",
+          fragment: "fragment").origin,
       (e) { return e is IllegalArgumentException; },
       "origin for uri with empty scheme should fail");
   Expect.throws(
-      () => const Uri.fromComponents("http", null, null, 80, "/a/b/c",
-                       "query", "fragment").origin,
+      () => const Uri.fromComponents(
+          scheme: "http",
+          userInfo: null,
+          domain: null,
+          port: 80,
+          path: "/a/b/c",
+          query: "query",
+          fragment: "fragment").origin,
       (e) { return e is IllegalArgumentException; },
       "origin for uri with empty domain should fail");
   Expect.throws(
       () => new Uri.fromString("http://:80").origin,
       (e) { return e is IllegalArgumentException; },
       "origin for uri with empty domain should fail");
   Expect.throws(
       () => new Uri.fromString("file://localhost/test.txt").origin,
       (e) { return e is IllegalArgumentException; },
       "origin for non-http/https uri should fail");
 
   // URI encode tests
   // Note: dart2js won't handle '\ud800\udc00' and frog
   // won't handle '\u{10000}'. So we cons this up synthetically...
   var s = decodeUtf8([0xf0, 0x90, 0x80, 0x80]);
-  
+
   testEncodeDecode("\uFFFE", "%EF%BF%BE");
   testEncodeDecode("\uFFFF", "%EF%BF%BF");
   testEncodeDecode("\uFFFE", "%EF%BF%BE");
   testEncodeDecode("\uFFFF", "%EF%BF%BF");
   testEncodeDecode("\x7f", "%7F");
   testEncodeDecode("\x80", "%C2%80");
   testEncodeDecode("\u0800", "%E0%A0%80");
   testEncodeDecode(":/@',;?&=+\$", ":/@',;?&=+\$");
   testEncodeDecode(s, "%F0%90%80%80");
   testEncodeDecodeComponent("\uFFFE", "%EF%BF%BE");
   testEncodeDecodeComponent("\uFFFF", "%EF%BF%BF");
   testEncodeDecodeComponent("\uFFFE", "%EF%BF%BE");
   testEncodeDecodeComponent("\uFFFF", "%EF%BF%BF");
   testEncodeDecodeComponent("\x7f", "%7F");
   testEncodeDecodeComponent("\x80", "%C2%80");
   testEncodeDecodeComponent("\u0800", "%E0%A0%80");
   testEncodeDecodeComponent(":/@',;?&=+\$", "%3A%2F%40'%2C%3B%3F%26%3D%2B%24");
   testEncodeDecodeComponent(s, "%F0%90%80%80");
 }