<fix>(resource_url_resolver): Make resolved URLs absolute rather than…

… relative

Makes resolved URLs absolute instead of relative. With this determination of
URLs is not relative to the current browser path, which was one of the problems
this code is trying to solve.

Closes dart-archive#1599

lib/core_dom/resource_url_resolver.dart

@@ -163,10 +163,9 @@ class ResourceUrlResolver {
     // If it's package-relative, tack on 'packages/' - Note that eventually
     // we may want to change this to be '/packages/' to make it truly absolute
     if (resolved.scheme == 'package') {
-      return 'packages/${resolved.path}';
+      return '/packages/${resolved.path}';
     } else if (resolved.isAbsolute && resolved.toString().startsWith(_baseUri)) {
-      var path = resolved.path;
-      return path.startsWith("/") ? path.substring(1) : path;
+      return resolved.path;
     } else {
       return resolved.toString();
     }

test/_specs.dart

@@ -47,7 +47,7 @@ e(String html) => es(html).first;
 // this file, _specs.dart, is at path /base/test/_specs.dart.  However, if
 // you're using a different test server or reconfigured the base prefix, then
 // you can set this to something different.
-String TEST_SERVER_BASE_PREFIX = "base/";
+String TEST_SERVER_BASE_PREFIX = "/base/";
 
 Expect expect(actual, [matcher]) {
   final expect = new Expect(actual);

test/core_dom/resource_url_resolver_spec.dart

@@ -195,18 +195,18 @@ _run_resolver({useRelativeUrls}) {
 
     testBothSchemes(
         urlToResolve:             'package:a.b/c/d/foo2.html',
-        expectedForPackageScheme: 'packages/a.b/c/d/foo2.html',
-        expectedForHttpScheme:    'packages/a.b/c/d/foo2.html');
+        expectedForPackageScheme: '/packages/a.b/c/d/foo2.html',
+        expectedForHttpScheme:    '/packages/a.b/c/d/foo2.html');
 
     testBothSchemes(
         urlToResolve:             'image.png',
-        expectedForPackageScheme: 'packages/angular/test/core_dom/image.png',
-        expectedForHttpScheme:    'a/b/image.png');
+        expectedForPackageScheme: '/packages/angular/test/core_dom/image.png',
+        expectedForHttpScheme:    '/a/b/image.png');
 
     testBothSchemes(
         urlToResolve:             './image2.png',
-        expectedForPackageScheme: 'packages/angular/test/core_dom/image2.png',
-        expectedForHttpScheme:    'a/b/image2.png');
+        expectedForPackageScheme: '/packages/angular/test/core_dom/image2.png',
+        expectedForHttpScheme:    '/a/b/image2.png');
 
     testBothSchemes(
         urlToResolve:             '/image3.png',
@@ -225,13 +225,13 @@ _run_resolver({useRelativeUrls}) {
 
     testBothSchemes(
         urlToResolve:             'HTTP://LOCALHOST/a/b/image4.png',
-        expectedForPackageScheme: 'a/b/image4.png',
-        expectedForHttpScheme:    'a/b/image4.png');
+        expectedForPackageScheme: '/a/b/image4.png',
+        expectedForHttpScheme:    '/a/b/image4.png');
 
     testBothSchemes(
         urlToResolve:             'HTTP://LOCALHOST/packages/angular/test/core_dom/foo3.html',
-        expectedForPackageScheme: 'packages/angular/test/core_dom/foo3.html',
-        expectedForHttpScheme:    'packages/angular/test/core_dom/foo3.html');
+        expectedForPackageScheme: '/packages/angular/test/core_dom/foo3.html',
+        expectedForHttpScheme:    '/packages/angular/test/core_dom/foo3.html');
 
   });
 }