Revert "fix(url_resolver): in Dart make package urls relative to AppRootUrl"

This reverts commit 469afda.

Author: mhevery

modules/angular2/src/services/url_resolver.dart

@@ -1,15 +1,9 @@
 library angular2.src.services.url_resolver;
 
 import 'package:angular2/di.dart' show Injectable;
-import 'package:angular2/src/services/app_root_url.dart' show AppRootUrl;
 
 @Injectable()
 class UrlResolver {
-
-  final AppRootUrl _appRootUrl;
-
-  UrlResolver(this._appRootUrl);
-
   /**
    * Resolves the `url` given the `baseUrl`:
    * - when the `url` is null, the `baseUrl` is returned,
@@ -26,8 +20,7 @@ class UrlResolver {
     Uri uri = Uri.parse(url);
 
     if (uri.scheme == 'package') {
-      var maybeSlash = _appRootUrl.value.endsWith('/') ? '' : '/';
-      return '${_appRootUrl.value}${maybeSlash}packages/${uri.path}';
+      return '/packages/${uri.path}';
     }
 
     if (uri.isAbsolute) return uri.toString();

modules/angular2/src/services/url_resolver.ts

@@ -7,7 +7,6 @@ import {
   normalizeBlank
 } from 'angular2/src/facade/lang';
 import {ListWrapper} from 'angular2/src/facade/collection';
-import {AppRootUrl} from 'angular2/src/services/app_root_url';
 
 /**
  * Used by the {@link Compiler} when resolving HTML and CSS template URLs.
@@ -18,8 +17,6 @@ import {AppRootUrl} from 'angular2/src/services/app_root_url';
  */
 @Injectable()
 export class UrlResolver {
-  constructor(_: AppRootUrl) {}
-
   /**
    * Resolves the `url` given the `baseUrl`:
    * - when the `url` is null, the `baseUrl` is returned,

modules/angular2/src/transform/template_compiler/generator.dart

@@ -12,7 +12,6 @@ import 'package:angular2/src/render/dom/compiler/style_url_resolver.dart';
 import 'package:angular2/src/render/dom/compiler/view_loader.dart';
 import 'package:angular2/src/render/xhr.dart' show XHR;
 import 'package:angular2/src/reflection/reflection.dart';
-import 'package:angular2/src/services/app_root_url.dart';
 import 'package:angular2/src/services/url_resolver.dart';
 import 'package:angular2/src/transform/common/asset_reader.dart';
 import 'package:angular2/src/transform/common/xhr_impl.dart';
@@ -86,7 +85,7 @@ class _TemplateExtractor {
 
   _TemplateExtractor(XHR xhr)
       : _factory = new CompileStepFactory(new ng.Parser(new ng.Lexer())) {
-    var urlResolver = new UrlResolver(new AppRootUrl(''));
+    var urlResolver = new UrlResolver();
     var styleUrlResolver = new StyleUrlResolver(urlResolver);
     var styleInliner = new StyleInliner(xhr, styleUrlResolver, urlResolver);
 

modules/angular2/test/core/compiler/compiler_spec.ts

@@ -48,7 +48,7 @@ export function main() {
     function createCompiler(renderCompileResults:
                                 List<renderApi.ProtoViewDto | Promise<renderApi.ProtoViewDto>>,
                             protoViewFactoryResults: List<AppProtoView>) {
-      var urlResolver = new UrlResolver(new AppRootUrl(""));
+      var urlResolver = new UrlResolver();
       renderCompileRequests = [];
       renderCompileResults = ListWrapper.clone(renderCompileResults);
       renderCompiler.spy('compile').andCallFake((view) => {
@@ -399,9 +399,9 @@ export function main() {
          var reader: any = new SpyDirectiveResolver();
 
          // create the compiler
-         var compiler = new Compiler(reader, cache, tplResolver, cmpUrlMapper,
-                                     new UrlResolver(new AppRootUrl("")), renderCompiler,
-                                     protoViewFactory, new AppRootUrl("http://www.app.com/"));
+         var compiler =
+             new Compiler(reader, cache, tplResolver, cmpUrlMapper, new UrlResolver(),
+                          renderCompiler, protoViewFactory, new AppRootUrl("http://www.app.com/"));
          compiler.compileInHost(MainComponent)
              .then((protoViewRef) => {
                // the test should have failed if the resolver was called, so we're good
@@ -719,4 +719,4 @@ function collectEmbeddedPvs(pv: AppProtoView, target: AppProtoView[] = null): Ap
     }
   });
   return target;
-}
+}

modules/angular2/test/render/dom/compiler/style_url_resolver_spec.ts

@@ -2,14 +2,12 @@ import {describe, it, expect, beforeEach, ddescribe, iit, xit, el} from 'angular
 import {StyleUrlResolver} from 'angular2/src/render/dom/compiler/style_url_resolver';
 
 import {UrlResolver} from 'angular2/src/services/url_resolver';
-import {AppRootUrl} from 'angular2/src/services/app_root_url';
 
 export function main() {
   describe('StyleUrlResolver', () => {
     let styleUrlResolver;
 
-    beforeEach(
-        () => { styleUrlResolver = new StyleUrlResolver(new UrlResolver(new AppRootUrl(""))); });
+    beforeEach(() => { styleUrlResolver = new StyleUrlResolver(new UrlResolver()); });
 
     it('should resolve "url()" urls', () => {
       var css = `

modules/angular2/test/render/dom/compiler/view_loader_spec.ts

@@ -15,7 +15,6 @@ import {ViewLoader} from 'angular2/src/render/dom/compiler/view_loader';
 import {StyleInliner} from 'angular2/src/render/dom/compiler/style_inliner';
 import {StyleUrlResolver} from 'angular2/src/render/dom/compiler/style_url_resolver';
 import {UrlResolver} from 'angular2/src/services/url_resolver';
-import {AppRootUrl} from 'angular2/src/services/app_root_url';
 
 import {ViewDefinition} from 'angular2/src/render/api';
 import {PromiseWrapper, Promise} from 'angular2/src/facade/async';
@@ -29,7 +28,7 @@ export function main() {
 
     beforeEach(() => {
       xhr = new MockXHR();
-      urlResolver = new UrlResolver(new AppRootUrl(''));
+      urlResolver = new UrlResolver();
       styleUrlResolver = new StyleUrlResolver(urlResolver);
       let styleInliner = new StyleInliner(xhr, styleUrlResolver, urlResolver);
       loader = new ViewLoader(xhr, styleInliner, styleUrlResolver);

modules/angular2/test/services/url_resolver_spec.ts

@@ -1,21 +1,9 @@
-import {
-  describe,
-  it,
-  expect,
-  beforeEach,
-  ddescribe,
-  iit,
-  xit,
-  el,
-  IS_DARTIUM
-} from 'angular2/test_lib';
+import {describe, it, expect, beforeEach, ddescribe, iit, xit, el} from 'angular2/test_lib';
 import {UrlResolver} from 'angular2/src/services/url_resolver';
-import {AppRootUrl} from 'angular2/src/services/app_root_url';
 
 export function main() {
   describe('UrlResolver', () => {
-    var appRootUrl = new AppRootUrl('http://localhost/example/');
-    var resolver = new UrlResolver(appRootUrl);
+    var resolver = new UrlResolver();
 
     describe('absolute base url', () => {
       it('should add a relative path to the base url', () => {
@@ -82,14 +70,5 @@ export function main() {
         expect(resolver.resolve('foo/baz/', '/bar')).toEqual('/bar');
       });
     });
-
-    if (IS_DARTIUM) {
-      describe('package url', () => {
-        it('should be served relative to AppRootUrl', () => {
-          expect(resolver.resolve('foo', 'package:bar/baz.dart'))
-              .toEqual('http://localhost/example/packages/bar/baz.dart');
-        });
-      });
-    }
   });
 }

modules/benchmarks/src/compiler/compiler_benchmark.ts

@@ -39,7 +39,7 @@ export function main() {
   var cache = new CompilerCache();
   var viewResolver = new MultipleViewResolver(
       count, [BenchmarkComponentNoBindings, BenchmarkComponentWithBindings]);
-  var urlResolver = new UrlResolver(new AppRootUrl(""));
+  var urlResolver = new UrlResolver();
   var shadowDomStrategy = new NativeShadowDomStrategy();
   var renderCompiler = new rc.DefaultDomCompiler(new Parser(new Lexer()), shadowDomStrategy,
                                                  new ViewLoader(null, null, null));

modules/examples/src/material/button/index.ts

@@ -37,5 +37,5 @@ class DemoApp {
 
 export function main() {
   commonDemoSetup();
-  bootstrap(DemoApp, [bind(UrlResolver).toClass(DemoUrlResolver)]);
+  bootstrap(DemoApp, [bind(UrlResolver).toValue(new DemoUrlResolver())]);
 }

modules/examples/src/material/checkbox/index.ts

@@ -20,5 +20,5 @@ class DemoApp {
 
 export function main() {
   commonDemoSetup();
-  bootstrap(DemoApp, [bind(UrlResolver).toClass(DemoUrlResolver)]);
+  bootstrap(DemoApp, [bind(UrlResolver).toValue(new DemoUrlResolver())]);
 }