fix(dart): fix the static_browser platform not to include compiler

vsavkin · vsavkin · commit ad6fb067e9dc · 2015-11-16T23:29:36.000Z
Closes #5321
diff --git a/modules/angular2/platform/browser.ts b/modules/angular2/platform/browser.ts
@@ -10,12 +10,18 @@ import {
 import {COMPILER_PROVIDERS} from 'angular2/compiler';
 import {ComponentRef, platform, reflector} from 'angular2/core';
 import {ReflectionCapabilities} from 'angular2/src/core/reflection/reflection_capabilities';
+import {XHRImpl} from "angular2/src/platform/browser/xhr_impl";
+import {XHR} from 'angular2/compiler';
+import {Provider} from 'angular2/src/core/di';
 
 /**
  * An array of providers that should be passed into `application()` when bootstrapping a component.
  */
-export const BROWSER_APP_PROVIDERS: Array<any /*Type | Provider | any[]*/> =
-    CONST_EXPR([BROWSER_APP_COMMON_PROVIDERS, COMPILER_PROVIDERS]);
+export const BROWSER_APP_PROVIDERS: Array<any /*Type | Provider | any[]*/> = CONST_EXPR([
+  BROWSER_APP_COMMON_PROVIDERS,
+  COMPILER_PROVIDERS,
+  new Provider(XHR, {useClass: XHRImpl}),
+]);
 
 /**
  * Bootstrapping for Angular applications.
diff --git a/modules/angular2/src/compiler/xhr.ts b/modules/angular2/src/compiler/xhr.ts
@@ -1,6 +1,6 @@
 import {Promise} from 'angular2/src/facade/async';
 
-// TODO: vsavkin rename it into TemplateLoader and do not reexport it via DomAdapter
+// TODO: vsavkin rename it into TemplateLoader
 export class XHR {
   get(url: string): Promise<string> { return null; }
 }
diff --git a/modules/angular2/src/platform/browser/xhr_impl.dart b/modules/angular2/src/platform/browser/xhr_impl.dart
@@ -3,7 +3,7 @@ library angular2.src.services.xhr_impl;
 import 'dart:async' show Future;
 import 'dart:html' show HttpRequest;
 import 'package:angular2/core.dart';
-import 'package:angular2/compiler.dart' show XHR;
+import 'package:angular2/src/compiler/xhr.dart';
 
 @Injectable()
 class XHRImpl extends XHR {
diff --git a/modules/angular2/src/platform/browser_common.ts b/modules/angular2/src/platform/browser_common.ts
@@ -1,7 +1,6 @@
 import {CONST_EXPR} from 'angular2/src/facade/lang';
 import {provide, Provider, Injector, OpaqueToken} from 'angular2/src/core/di';
 
-import {XHR} from 'angular2/compiler';
 import {
   PLATFORM_DIRECTIVES,
   PLATFORM_PIPES,
@@ -15,7 +14,6 @@ import {
 } from "angular2/core";
 import {COMMON_DIRECTIVES, COMMON_PIPES, FORM_PROVIDERS} from "angular2/common";
 import {Renderer} from 'angular2/render';
-import {XHRImpl} from "angular2/src/platform/browser/xhr_impl";
 import {Testability} from 'angular2/src/core/testability/testability';
 
 // TODO change these imports once dom_adapter is moved out of core
@@ -60,7 +58,6 @@ export const BROWSER_APP_COMMON_PROVIDERS: Array<any /*Type | Provider | any[]*/
   new Provider(DomRenderer, {useClass: DomRenderer_}),
   new Provider(Renderer, {useExisting: DomRenderer}),
   new Provider(SharedStylesHost, {useExisting: DomSharedStylesHost}),
-  new Provider(XHR, {useClass: XHRImpl}),
   DomSharedStylesHost,
   Testability,
   BrowserDetails,

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`import {Promise} from 'angular2/src/facade/async';`
`2`	`2`
`3`		`-// TODO: vsavkin rename it into TemplateLoader and do not reexport it via DomAdapter`
	`3`	`+// TODO: vsavkin rename it into TemplateLoader`
`4`	`4`	`export class XHR {`
`5`	`5`	`get(url: string): Promise<string> { return null; }`
`6`	`6`	`}`