Avoid parsing path and query string more than once

tests/corelib/uri_path_test.dart

 // Copyright (c) 2013, 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.
 
 import "package:expect/expect.dart";
 
 void testInvalidArguments() {
 }
 
 void testPath() {
@@ skipping 58 matching lines @@
     unencoded.writeCharCode(i);
   }
   encoded = encoded.toString();
   unencoded = unencoded.toString();
   print(encoded);
   print(unencoded);
   test(encoded, [unencoded]);
   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();
 }