fix(bootstrap): fix the configuration of ExceptionHandler

vsavkin · jelbourn · commit 0d9a1de4d995 · 2015-12-08T16:09:37.000-08:00
diff --git a/modules/angular2/src/core/application_ref.ts b/modules/angular2/src/core/application_ref.ts
@@ -1,5 +1,5 @@
 import {NgZone} from 'angular2/src/core/zone/ng_zone';
-import {Type, isBlank, isPresent, assertionsEnabled, print} from 'angular2/src/facade/lang';
+import {Type, isBlank, isPresent, assertionsEnabled, print, IS_DART} from 'angular2/src/facade/lang';
 import {provide, Provider, Injector, OpaqueToken} from 'angular2/src/core/di';
 import {
   APP_COMPONENT_REF_PROMISE,
@@ -412,7 +412,14 @@ export class ApplicationRef_ extends ApplicationRef {
 
         var tickResult = PromiseWrapper.then(compRefToken, tick);
 
-        PromiseWrapper.then(tickResult, (_) => {});
+        // THIS MUST ONLY RUN IN DART.
+        // This is required to report an error when no components with a matching selector found.
+        // Otherwise the promise will never be completed.
+        // Doing this in JS causes an extra error message to appear.
+        if (IS_DART) {
+          PromiseWrapper.then(tickResult, (_) => {});
+        }
+
         PromiseWrapper.then(tickResult, null,
                             (err, stackTrace) => completer.reject(err, stackTrace));
       } catch (e) {
diff --git a/modules/angular2/src/platform/browser_common.ts b/modules/angular2/src/platform/browser_common.ts
@@ -1,4 +1,4 @@
-import {CONST_EXPR} from 'angular2/src/facade/lang';
+import {CONST_EXPR, IS_DART} from 'angular2/src/facade/lang';
 import {provide, Provider, Injector, OpaqueToken} from 'angular2/src/core/di';
 
 import {
@@ -48,7 +48,9 @@ export const BROWSER_PROVIDERS: Array<any /*Type | Provider | any[]*/> = CONST_E
 ]);
 
 function _exceptionHandler(): ExceptionHandler {
-  return new ExceptionHandler(DOM, false);
+  // !IS_DART is required because we must rethrow exceptions in JS,
+  // but must not rethrow exceptions in Dart
+  return new ExceptionHandler(DOM, !IS_DART);
 }
 
 function _document(): any {
diff --git a/modules/angular2/src/platform/worker_render_common.ts b/modules/angular2/src/platform/worker_render_common.ts
@@ -1,4 +1,4 @@
-import {CONST_EXPR} from 'angular2/src/facade/lang';
+import {CONST_EXPR, IS_DART} from 'angular2/src/facade/lang';
 import {MessageBus} from 'angular2/src/web_workers/shared/message_bus';
 import {NgZone} from 'angular2/src/core/zone/ng_zone';
 import {AnchorBasedAppRootUrl} from 'angular2/src/compiler/anchor_based_app_root_url';
@@ -112,7 +112,7 @@ export function initWebWorkerRenderPlatform(): void {
 }
 
 function _exceptionHandler(): ExceptionHandler {
-  return new ExceptionHandler(DOM, false);
+  return new ExceptionHandler(DOM, !IS_DART);
 }
 
 function _document(): any {
diff --git a/modules/angular2/test/platform/browser/bootstrap_spec.ts b/modules/angular2/test/platform/browser/bootstrap_spec.ts
@@ -125,7 +125,7 @@ export function main() {
 
     it('should throw if no element is found', inject([AsyncTestCompleter], (async) => {
          var logger = new _ArrayLogger();
-         var exceptionHandler = new ExceptionHandler(logger, IS_DART ? false : true);
+         var exceptionHandler = new ExceptionHandler(logger, !IS_DART);
 
          var refPromise =
              bootstrap(HelloRootCmp, [provide(ExceptionHandler, {useValue: exceptionHandler})]);
@@ -140,7 +140,7 @@ export function main() {
       it('should invoke the default exception handler when bootstrap fails',
          inject([AsyncTestCompleter], (async) => {
            var logger = new _ArrayLogger();
-           var exceptionHandler = new ExceptionHandler(logger, IS_DART ? false : true);
+           var exceptionHandler = new ExceptionHandler(logger, !IS_DART);
 
            var refPromise =
                bootstrap(HelloRootCmp, [provide(ExceptionHandler, {useValue: exceptionHandler})]);
diff --git a/modules/angular2/test/router/route_config_spec.ts b/modules/angular2/test/router/route_config_spec.ts
@@ -16,7 +16,7 @@ import {Component, Directive, View} from 'angular2/src/core/metadata';
 import {DOM} from 'angular2/src/platform/dom/dom_adapter';
 import {provide} from 'angular2/core';
 import {DOCUMENT} from 'angular2/src/platform/dom/dom_tokens';
-import {Type} from 'angular2/src/facade/lang';
+import {Type, IS_DART} from 'angular2/src/facade/lang';
 
 import {
   ROUTER_PROVIDERS,
@@ -46,7 +46,7 @@ export function main() {
       el = DOM.createElement('app-cmp', fakeDoc);
       DOM.appendChild(fakeDoc.body, el);
       var logger = new _ArrayLogger();
-      var exceptionHandler = new ExceptionHandler(logger, true);
+      var exceptionHandler = new ExceptionHandler(logger, !IS_DART);
       testBindings = [
         ROUTER_PROVIDERS,
         provide(LocationStrategy, {useClass: MockLocationStrategy}),

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-import {CONST_EXPR} from 'angular2/src/facade/lang';`
	`1`	`+import {CONST_EXPR, IS_DART} from 'angular2/src/facade/lang';`
`2`	`2`	`import {MessageBus} from 'angular2/src/web_workers/shared/message_bus';`
`3`	`3`	`import {NgZone} from 'angular2/src/core/zone/ng_zone';`
`4`	`4`	`import {AnchorBasedAppRootUrl} from 'angular2/src/compiler/anchor_based_app_root_url';`
`@@ -112,7 +112,7 @@ export function initWebWorkerRenderPlatform(): void {`
`112`	`112`	`}`
`113`	`113`
`114`	`114`	`function _exceptionHandler(): ExceptionHandler {`
`115`		`- return new ExceptionHandler(DOM, false);`
	`115`	`+ return new ExceptionHandler(DOM, !IS_DART);`
`116`	`116`	`}`
`117`	`117`
`118`	`118`	`function _document(): any {`