Avoid parsing path and query string more than once

tests/corelib/uri_path_test.dart

Index: tests/corelib/uri_path_test.dart
diff --git a/tests/corelib/uri_path_test.dart b/tests/corelib/uri_path_test.dart
index bc62d3e1b3ee54756b33485be6fc2ce39170e91a..653402c84b0e2ed3f0c45788931d11fc0276cf62 100644
--- a/tests/corelib/uri_path_test.dart
+++ b/tests/corelib/uri_path_test.dart
@@ -76,8 +76,19 @@ void testPathSegments() {
   test(encoded + "/" + encoded, [unencoded, unencoded]);
 }
 
+testPathSegmentsParsedOnce() {
+  var uri = Uri.parse("a/b");
+  Expect.isTrue(identical(uri.pathSegments, uri.pathSegments));

[Lasse Reichstein Nielsen, 2013/05/30 11:57:50]

I wouldn't guarantee that - it means you can't throw away your cached values if
you want to, which you may want at some point in the future.
The return value may or may not be the same object.

Or in other words: This is a performance optimization, not something we need to
check for.

[Søren Gjesse, 2013/05/30 13:38:25]

You are probably right. I put it in to make sure my change did what I expected.

Removed.

+
+  var list = ["a", "b"];
+  uri = new Uri(pathSegments: list);
+  Expect.isFalse(identical(list, uri.pathSegments));
+  Expect.isTrue(identical(uri.pathSegments, uri.pathSegments));
+}
+
 main() {
   testInvalidArguments();
   testPath();
   testPathSegments();
+  testPathSegmentsParsedOnce();
 }