From 0b3e4fa090eca6382d601253854f7aa4ba47a16a Mon Sep 17 00:00:00 2001
From: Tobias Bosch <tbosch1009@gmail.com>
Date: Mon, 5 Oct 2015 10:10:07 -0700
Subject: [PATCH] refactor(core): move `ViewEncapsulation` and `ViewType` to
 the right places

Closes #4526
---
 .../src/core/compiler/command_compiler.ts     |  2 +-
 .../src/core/compiler/directive_metadata.ts   |  2 +-
 .../src/core/compiler/style_compiler.ts       |  2 +-
 .../src/core/compiler/template_normalizer.ts  |  3 +-
 .../src/core/linker/proto_view_factory.ts     |  4 +-
 modules/angular2/src/core/linker/view.ts      | 16 ++++++-
 .../angular2/src/core/linker/view_manager.ts  | 11 +++--
 .../src/core/linker/view_manager_utils.ts     |  9 ++--
 modules/angular2/src/core/metadata.dart       |  2 +-
 modules/angular2/src/core/metadata/view.ts    | 33 +++++++++++++-
 modules/angular2/src/core/render/api.ts       | 45 -------------------
 .../src/web_workers/shared/serializer.ts      | 19 +-------
 .../core/compiler/command_compiler_spec.ts    |  2 +-
 .../core/compiler/directive_metadata_spec.ts  |  2 +-
 .../test/core/compiler/style_compiler_spec.ts |  2 +-
 .../core/compiler/template_compiler_spec.ts   |  2 +-
 .../core/compiler/template_normalizer_spec.ts |  2 +-
 .../test/core/linker/view_manager_spec.ts     |  3 +-
 .../core/linker/view_manager_utils_spec.ts    |  9 +++-
 .../common/directive_metadata_reader.dart     |  2 +-
 20 files changed, 77 insertions(+), 95 deletions(-)

diff --git a/modules/angular2/src/core/compiler/command_compiler.ts b/modules/angular2/src/core/compiler/command_compiler.ts
index e9f29862f0..976eab4358 100644
--- a/modules/angular2/src/core/compiler/command_compiler.ts
+++ b/modules/angular2/src/core/compiler/command_compiler.ts
@@ -30,7 +30,7 @@ import {
 import {CompileTypeMetadata, CompileDirectiveMetadata} from './directive_metadata';
 import {SourceExpressions, SourceExpression, moduleRef} from './source_module';
 
-import {ViewEncapsulation} from 'angular2/src/core/render/api';
+import {ViewEncapsulation} from 'angular2/src/core/metadata/view';
 import {
   shimHostAttribute,
   shimContentAttribute,
diff --git a/modules/angular2/src/core/compiler/directive_metadata.ts b/modules/angular2/src/core/compiler/directive_metadata.ts
index ec344771b5..a0e07cce26 100644
--- a/modules/angular2/src/core/compiler/directive_metadata.ts
+++ b/modules/angular2/src/core/compiler/directive_metadata.ts
@@ -12,7 +12,7 @@ import {
   ChangeDetectionStrategy,
   CHANGE_DECTION_STRATEGY_VALUES
 } from 'angular2/src/core/change_detection/change_detection';
-import {ViewEncapsulation, VIEW_ENCAPSULATION_VALUES} from 'angular2/src/core/render/api';
+import {ViewEncapsulation, VIEW_ENCAPSULATION_VALUES} from 'angular2/src/core/metadata/view';
 import {CssSelector} from 'angular2/src/core/compiler/selector';
 import {splitAtColon} from './util';
 import {LifecycleHooks, LIFECYCLE_HOOKS_VALUES} from 'angular2/src/core/linker/interfaces';
diff --git a/modules/angular2/src/core/compiler/style_compiler.ts b/modules/angular2/src/core/compiler/style_compiler.ts
index e55f32f15b..139ad9bef2 100644
--- a/modules/angular2/src/core/compiler/style_compiler.ts
+++ b/modules/angular2/src/core/compiler/style_compiler.ts
@@ -1,6 +1,6 @@
 import {CompileTypeMetadata, CompileTemplateMetadata} from './directive_metadata';
 import {SourceModule, SourceExpression, moduleRef} from './source_module';
-import {ViewEncapsulation} from 'angular2/src/core/render/api';
+import {ViewEncapsulation} from 'angular2/src/core/metadata/view';
 import {XHR} from 'angular2/src/core/compiler/xhr';
 import {StringWrapper, isBlank} from 'angular2/src/core/facade/lang';
 import {PromiseWrapper, Promise} from 'angular2/src/core/facade/async';
diff --git a/modules/angular2/src/core/compiler/template_normalizer.ts b/modules/angular2/src/core/compiler/template_normalizer.ts
index 6872d8f22a..a1114cae85 100644
--- a/modules/angular2/src/core/compiler/template_normalizer.ts
+++ b/modules/angular2/src/core/compiler/template_normalizer.ts
@@ -11,7 +11,8 @@ import {XHR} from 'angular2/src/core/compiler/xhr';
 import {UrlResolver} from 'angular2/src/core/compiler/url_resolver';
 import {resolveStyleUrls} from './style_url_resolver';
 import {Injectable} from 'angular2/src/core/di';
-import {ViewEncapsulation} from 'angular2/src/core/render/api';
+import {ViewEncapsulation} from 'angular2/src/core/metadata/view';
+
 
 import {
   HtmlAstVisitor,
diff --git a/modules/angular2/src/core/linker/proto_view_factory.ts b/modules/angular2/src/core/linker/proto_view_factory.ts
index 0caa431199..16f3b69dea 100644
--- a/modules/angular2/src/core/linker/proto_view_factory.ts
+++ b/modules/angular2/src/core/linker/proto_view_factory.ts
@@ -1,14 +1,14 @@
 import {ListWrapper} from 'angular2/src/core/facade/collection';
 import {isPresent, isBlank, Type, isArray, isNumber} from 'angular2/src/core/facade/lang';
 
-import {ViewType, RenderProtoViewRef} from 'angular2/src/core/render/api';
+import {RenderProtoViewRef} from 'angular2/src/core/render/api';
 
 import {Injectable, Binding, resolveForwardRef, Inject} from 'angular2/src/core/di';
 
 import {PipeBinding} from '../pipes/pipe_binding';
 import {ProtoPipes} from '../pipes/pipes';
 
-import {AppProtoView, AppProtoViewMergeInfo} from './view';
+import {AppProtoView, AppProtoViewMergeInfo, ViewType} from './view';
 import {ElementBinder} from './element_binder';
 import {ProtoElementInjector, DirectiveBinding} from './element_injector';
 import {DirectiveResolver} from './directive_resolver';
diff --git a/modules/angular2/src/core/linker/view.ts b/modules/angular2/src/core/linker/view.ts
index d61018198e..b2eec85a63 100644
--- a/modules/angular2/src/core/linker/view.ts
+++ b/modules/angular2/src/core/linker/view.ts
@@ -38,6 +38,18 @@ export {DebugContext} from 'angular2/src/core/change_detection/interfaces';
 
 const REFLECT_PREFIX: string = 'ng-reflect-';
 
+export enum ViewType {
+  // A view that contains the host element with bound component directive.
+  // Contains a COMPONENT view
+  HOST,
+  // The view of the component
+  // Can contain 0 to n EMBEDDED views
+  COMPONENT,
+  // A view that is embedded into another View via a <template> element
+  // inside of a COMPONENT view
+  EMBEDDED
+}
+
 export class AppViewContainer {
   // The order in this list matches the DOM order.
   views: AppView[] = [];
@@ -307,8 +319,8 @@ export class AppProtoView {
   textBindingCount = null;
   render: renderApi.RenderProtoViewRef = null;
 
-  constructor(public templateCmds: TemplateCmd[], public type: renderApi.ViewType,
-              public isMergable: boolean, public changeDetectorFactory: Function,
+  constructor(public templateCmds: TemplateCmd[], public type: ViewType, public isMergable: boolean,
+              public changeDetectorFactory: Function,
               public templateVariableBindings: Map<string, string>, public pipes: ProtoPipes) {
     this.ref = new ProtoViewRef(this);
   }
diff --git a/modules/angular2/src/core/linker/view_manager.ts b/modules/angular2/src/core/linker/view_manager.ts
index a297b9226b..7e648d2ff9 100644
--- a/modules/angular2/src/core/linker/view_manager.ts
+++ b/modules/angular2/src/core/linker/view_manager.ts
@@ -17,8 +17,7 @@ import {
   Renderer,
   RenderViewRef,
   RenderFragmentRef,
-  RenderViewWithFragments,
-  ViewType
+  RenderViewWithFragments
 } from 'angular2/src/core/render/api';
 import {AppViewManagerUtils} from './view_manager_utils';
 import {AppViewPool} from './view_pool';
@@ -57,7 +56,7 @@ export class AppViewManager {
    */
   getHostElement(hostViewRef: HostViewRef): ElementRef {
     var hostView = internalView(<ViewRef>hostViewRef);
-    if (hostView.proto.type !== ViewType.HOST) {
+    if (hostView.proto.type !== viewModule.ViewType.HOST) {
       throw new BaseException('This operation is only allowed on host views');
     }
     return hostView.elementRefs[hostView.elementOffset];
@@ -205,7 +204,7 @@ export class AppViewManager {
                                 templateRef: TemplateRef): ViewRef {
     var s = this._createEmbeddedViewInContainerScope();
     var protoView = internalProtoView(templateRef.protoViewRef);
-    if (protoView.type !== ViewType.EMBEDDED) {
+    if (protoView.type !== viewModule.ViewType.EMBEDDED) {
       throw new BaseException('This method can only be called with embedded ProtoViews!');
     }
     this._protoViewFactory.initializeProtoViewIfNeeded(protoView);
@@ -236,7 +235,7 @@ export class AppViewManager {
                             imperativelyCreatedInjector: ResolvedBinding[]): HostViewRef {
     var s = this._createHostViewInContainerScope();
     var protoView = internalProtoView(protoViewRef);
-    if (protoView.type !== ViewType.HOST) {
+    if (protoView.type !== viewModule.ViewType.HOST) {
       throw new BaseException('This method can only be called with host ProtoViews!');
     }
     this._protoViewFactory.initializeProtoViewIfNeeded(protoView);
@@ -258,7 +257,7 @@ export class AppViewManager {
     var contextBoundElementIndex = context.boundElementIndex;
     var embeddedFragmentView = contextView.getNestedView(contextBoundElementIndex);
     var view;
-    if (protoView.type === ViewType.EMBEDDED && isPresent(embeddedFragmentView) &&
+    if (protoView.type === viewModule.ViewType.EMBEDDED && isPresent(embeddedFragmentView) &&
         !embeddedFragmentView.hydrated()) {
       // Case 1: instantiate the first view of a template that has been merged into a parent
       view = embeddedFragmentView;
diff --git a/modules/angular2/src/core/linker/view_manager_utils.ts b/modules/angular2/src/core/linker/view_manager_utils.ts
index cad11238bd..cfc2d0110a 100644
--- a/modules/angular2/src/core/linker/view_manager_utils.ts
+++ b/modules/angular2/src/core/linker/view_manager_utils.ts
@@ -9,7 +9,6 @@ import {TemplateRef} from './template_ref';
 import {Renderer, RenderViewWithFragments} from 'angular2/src/core/render/api';
 import {Locals} from 'angular2/src/core/change_detection/change_detection';
 import {Pipes} from 'angular2/src/core/pipes/pipes';
-import {RenderViewRef, RenderFragmentRef, ViewType} from 'angular2/src/core/render/api';
 
 @Injectable()
 export class AppViewManagerUtils {
@@ -50,7 +49,7 @@ export class AppViewManagerUtils {
                   .nestedProtoView :
               mergedParentViewProto;
       var renderFragment = null;
-      if (viewOffset === 0 || protoView.type === ViewType.EMBEDDED) {
+      if (viewOffset === 0 || protoView.type === viewModule.ViewType.EMBEDDED) {
         renderFragment = renderFragments[fragmentIdx++];
       }
       var currentView = new viewModule.AppView(renderer, protoView, viewOffset, elementOffset,
@@ -93,7 +92,7 @@ export class AppViewManagerUtils {
         // preBuiltObjects
         if (isPresent(elementInjector)) {
           var templateRef = isPresent(binder.nestedProtoView) &&
-                                    binder.nestedProtoView.type === ViewType.EMBEDDED ?
+                                    binder.nestedProtoView.type === viewModule.ViewType.EMBEDDED ?
                                 new TemplateRef(el) :
                                 null;
           preBuiltObjects[boundElementIndex] =
@@ -102,7 +101,7 @@ export class AppViewManagerUtils {
       }
       currentView.init(protoView.changeDetectorFactory(currentView), elementInjectors,
                        rootElementInjectors, preBuiltObjects, views, elementRefs, viewContainers);
-      if (isPresent(parentView) && protoView.type === ViewType.COMPONENT) {
+      if (isPresent(parentView) && protoView.type === viewModule.ViewType.COMPONENT) {
         parentView.changeDetector.addShadowDomChild(currentView.changeDetector);
       }
       elementOffset += protoView.elementBinders.length;
@@ -180,7 +179,7 @@ export class AppViewManagerUtils {
     while (viewIdx <= endViewOffset) {
       var currView = initView.views[viewIdx];
       var currProtoView = currView.proto;
-      if (currView !== initView && currView.proto.type === ViewType.EMBEDDED) {
+      if (currView !== initView && currView.proto.type === viewModule.ViewType.EMBEDDED) {
         // Don't hydrate components of embedded fragment views.
         viewIdx += currView.proto.mergeInfo.viewCount;
       } else {
diff --git a/modules/angular2/src/core/metadata.dart b/modules/angular2/src/core/metadata.dart
index 890f9a14a5..600267e4e2 100644
--- a/modules/angular2/src/core/metadata.dart
+++ b/modules/angular2/src/core/metadata.dart
@@ -8,7 +8,7 @@ import './metadata/view.dart';
 
 export './metadata/di.dart';
 export './metadata/directives.dart';
-export './metadata/view.dart';
+export './metadata/view.dart' hide VIEW_ENCAPSULATION_VALUES;
 
 /**
  * See: [DirectiveMetadata] for docs.
diff --git a/modules/angular2/src/core/metadata/view.ts b/modules/angular2/src/core/metadata/view.ts
index 4e4bccf143..c3e44bcbb6 100644
--- a/modules/angular2/src/core/metadata/view.ts
+++ b/modules/angular2/src/core/metadata/view.ts
@@ -1,7 +1,36 @@
 import {CONST, Type} from 'angular2/src/core/facade/lang';
-import {ViewEncapsulation} from 'angular2/src/core/render/api';
 
-export {ViewEncapsulation} from 'angular2/src/core/render/api';
+/**
+ * Defines template and style encapsulation options available for Component's {@link View}.
+ *
+ * See {@link ViewMetadata#encapsulation}.
+ */
+export enum ViewEncapsulation {
+  /**
+   * Emulate `Native` scoping of styles by adding an attribute containing surrogate id to the Host
+   * Element and pre-processing the style rules provided via
+   * {@link ViewMetadata#styles} or {@link ViewMetadata#stylesUrls}, and adding the new Host Element
+   * attribute to all selectors.
+   *
+   * This is the default option.
+   */
+  Emulated,
+  /**
+   * Use the native encapsulation mechanism of the renderer.
+   *
+   * For the DOM this means using [Shadow DOM](https://w3c.github.io/webcomponents/spec/shadow/) and
+   * creating a ShadowRoot for Component's Host Element.
+   */
+  Native,
+  /**
+   * Don't provide any template or style encapsulation.
+   */
+  None
+}
+
+export var VIEW_ENCAPSULATION_VALUES =
+    [ViewEncapsulation.Emulated, ViewEncapsulation.Native, ViewEncapsulation.None];
+
 
 /**
  * Metadata properties available for configuring Views.
diff --git a/modules/angular2/src/core/render/api.ts b/modules/angular2/src/core/render/api.ts
index 84c33e1f14..d4221f6a9b 100644
--- a/modules/angular2/src/core/render/api.ts
+++ b/modules/angular2/src/core/render/api.ts
@@ -1,18 +1,5 @@
 import {Map} from 'angular2/src/core/facade/collection';
 
-export enum ViewType {
-  // A view that contains the host element with bound component directive.
-  // Contains a COMPONENT view
-  HOST,
-  // The view of the component
-  // Can contain 0 to n EMBEDDED views
-  COMPONENT,
-  // A view that is embedded into another View via a <template> element
-  // inside of a COMPONENT view
-  EMBEDDED
-}
-
-
 /**
  * Represents an Angular ProtoView in the Rendering Context.
  *
@@ -89,38 +76,6 @@ export class RenderFragmentRef {}
 // TODO(i): refactor into an interface
 export class RenderViewRef {}
 
-
-/**
- * Defines template and style encapsulation options available for Component's {@link View}.
- *
- * See {@link ViewMetadata#encapsulation}.
- */
-export enum ViewEncapsulation {
-  /**
-   * Emulate `Native` scoping of styles by adding an attribute containing surrogate id to the Host
-   * Element and pre-processing the style rules provided via
-   * {@link ViewMetadata#styles} or {@link ViewMetadata#stylesUrls}, and adding the new Host Element
-   * attribute to all selectors.
-   *
-   * This is the default option.
-   */
-  Emulated,
-  /**
-   * Use the native encapsulation mechanism of the renderer.
-   *
-   * For the DOM this means using [Shadow DOM](https://w3c.github.io/webcomponents/spec/shadow/) and
-   * creating a ShadowRoot for Component's Host Element.
-   */
-  Native,
-  /**
-   * Don't provide any template or style encapsulation.
-   */
-  None
-}
-
-export var VIEW_ENCAPSULATION_VALUES =
-    [ViewEncapsulation.Emulated, ViewEncapsulation.Native, ViewEncapsulation.None];
-
 export interface RenderTemplateCmd { visit(visitor: RenderCommandVisitor, context: any): any; }
 
 export interface RenderBeginCmd extends RenderTemplateCmd {
diff --git a/modules/angular2/src/web_workers/shared/serializer.ts b/modules/angular2/src/web_workers/shared/serializer.ts
index b66a079743..544ed74d5f 100644
--- a/modules/angular2/src/web_workers/shared/serializer.ts
+++ b/modules/angular2/src/web_workers/shared/serializer.ts
@@ -13,8 +13,6 @@ import {
   RenderViewRef,
   RenderFragmentRef,
   RenderElementRef,
-  ViewType,
-  ViewEncapsulation,
   RenderTemplateCmd,
   RenderCommandVisitor,
   RenderTextCmd,
@@ -46,23 +44,8 @@ export const PRIMITIVE: Type = String;
 
 @Injectable()
 export class Serializer {
-  private _enumRegistry: Map<any, Map<number, any>>;
   constructor(private _protoViewStore: RenderProtoViewRefStore,
-              private _renderViewStore: RenderViewWithFragmentsStore) {
-    this._enumRegistry = new Map<any, Map<number, any>>();
-
-    var viewTypeMap = new Map<number, any>();
-    viewTypeMap[0] = ViewType.HOST;
-    viewTypeMap[1] = ViewType.COMPONENT;
-    viewTypeMap[2] = ViewType.EMBEDDED;
-    this._enumRegistry.set(ViewType, viewTypeMap);
-
-    var viewEncapsulationMap = new Map<number, any>();
-    viewEncapsulationMap[0] = ViewEncapsulation.Emulated;
-    viewEncapsulationMap[1] = ViewEncapsulation.Native;
-    viewEncapsulationMap[2] = ViewEncapsulation.None;
-    this._enumRegistry.set(ViewEncapsulation, viewEncapsulationMap);
-  }
+              private _renderViewStore: RenderViewWithFragmentsStore) {}
 
   serialize(obj: any, type: Type): Object {
     if (!isPresent(obj)) {
diff --git a/modules/angular2/test/core/compiler/command_compiler_spec.ts b/modules/angular2/test/core/compiler/command_compiler_spec.ts
index 94c84e5ef7..aa8d823c29 100644
--- a/modules/angular2/test/core/compiler/command_compiler_spec.ts
+++ b/modules/angular2/test/core/compiler/command_compiler_spec.ts
@@ -35,7 +35,7 @@ import {
   CompileTemplateMetadata
 } from 'angular2/src/core/compiler/directive_metadata';
 import {SourceModule, SourceExpression, moduleRef} from 'angular2/src/core/compiler/source_module';
-import {ViewEncapsulation} from 'angular2/src/core/render/api';
+import {ViewEncapsulation} from 'angular2/src/core/metadata/view';
 import {evalModule} from './eval_module';
 import {
   escapeSingleQuoteString,
diff --git a/modules/angular2/test/core/compiler/directive_metadata_spec.ts b/modules/angular2/test/core/compiler/directive_metadata_spec.ts
index 76f9149ec7..9f4ca11e11 100644
--- a/modules/angular2/test/core/compiler/directive_metadata_spec.ts
+++ b/modules/angular2/test/core/compiler/directive_metadata_spec.ts
@@ -17,7 +17,7 @@ import {
   CompileTypeMetadata,
   CompileTemplateMetadata
 } from 'angular2/src/core/compiler/directive_metadata';
-import {ViewEncapsulation} from 'angular2/src/core/render/api';
+import {ViewEncapsulation} from 'angular2/src/core/metadata/view';
 import {ChangeDetectionStrategy} from 'angular2/src/core/change_detection';
 import {LifecycleHooks} from 'angular2/src/core/linker/interfaces';
 
diff --git a/modules/angular2/test/core/compiler/style_compiler_spec.ts b/modules/angular2/test/core/compiler/style_compiler_spec.ts
index fcb338ac1c..6716051a78 100644
--- a/modules/angular2/test/core/compiler/style_compiler_spec.ts
+++ b/modules/angular2/test/core/compiler/style_compiler_spec.ts
@@ -27,7 +27,7 @@ import {
   CompileTypeMetadata
 } from 'angular2/src/core/compiler/directive_metadata';
 import {SourceExpression, SourceModule} from 'angular2/src/core/compiler/source_module';
-import {ViewEncapsulation} from 'angular2/src/core/render/api';
+import {ViewEncapsulation} from 'angular2/src/core/metadata/view';
 import {TEST_BINDINGS} from './test_bindings';
 import {
   codeGenValueFn,
diff --git a/modules/angular2/test/core/compiler/template_compiler_spec.ts b/modules/angular2/test/core/compiler/template_compiler_spec.ts
index 381376eb5d..fe6c698153 100644
--- a/modules/angular2/test/core/compiler/template_compiler_spec.ts
+++ b/modules/angular2/test/core/compiler/template_compiler_spec.ts
@@ -26,7 +26,7 @@ import {evalModule} from './eval_module';
 import {SourceModule, moduleRef} from 'angular2/src/core/compiler/source_module';
 import {XHR} from 'angular2/src/core/compiler/xhr';
 import {MockXHR} from 'angular2/src/core/compiler/xhr_mock';
-import {ViewEncapsulation} from 'angular2/src/core/render/api';
+import {ViewEncapsulation} from 'angular2/src/core/metadata/view';
 
 import {Locals} from 'angular2/src/core/change_detection/change_detection';
 
diff --git a/modules/angular2/test/core/compiler/template_normalizer_spec.ts b/modules/angular2/test/core/compiler/template_normalizer_spec.ts
index 672f38e00e..9dd24248c1 100644
--- a/modules/angular2/test/core/compiler/template_normalizer_spec.ts
+++ b/modules/angular2/test/core/compiler/template_normalizer_spec.ts
@@ -17,7 +17,7 @@ import {
   CompileTypeMetadata,
   CompileTemplateMetadata
 } from 'angular2/src/core/compiler/directive_metadata';
-import {ViewEncapsulation} from 'angular2/src/core/render/api';
+import {ViewEncapsulation} from 'angular2/src/core/metadata/view';
 
 import {TemplateNormalizer} from 'angular2/src/core/compiler/template_normalizer';
 import {XHR} from 'angular2/src/core/compiler/xhr';
diff --git a/modules/angular2/test/core/linker/view_manager_spec.ts b/modules/angular2/test/core/linker/view_manager_spec.ts
index 0a42b9426d..85a06c55bc 100644
--- a/modules/angular2/test/core/linker/view_manager_spec.ts
+++ b/modules/angular2/test/core/linker/view_manager_spec.ts
@@ -16,7 +16,7 @@ import {
 import {SpyRenderer, SpyAppViewPool, SpyAppViewListener, SpyProtoViewFactory} from '../spies';
 import {Injector, bind} from 'angular2/core';
 
-import {AppProtoView, AppView, AppViewContainer} from 'angular2/src/core/linker/view';
+import {AppProtoView, AppView, AppViewContainer, ViewType} from 'angular2/src/core/linker/view';
 import {ProtoViewRef, ViewRef, internalView} from 'angular2/src/core/linker/view_ref';
 import {ElementRef} from 'angular2/src/core/linker/element_ref';
 import {TemplateRef} from 'angular2/src/core/linker/template_ref';
@@ -25,7 +25,6 @@ import {
   RenderViewRef,
   RenderProtoViewRef,
   RenderFragmentRef,
-  ViewType,
   RenderViewWithFragments
 } from 'angular2/src/core/render/api';
 import {AppViewManager} from 'angular2/src/core/linker/view_manager';
diff --git a/modules/angular2/test/core/linker/view_manager_utils_spec.ts b/modules/angular2/test/core/linker/view_manager_utils_spec.ts
index 23bf36dd3c..4a6ff4f244 100644
--- a/modules/angular2/test/core/linker/view_manager_utils_spec.ts
+++ b/modules/angular2/test/core/linker/view_manager_utils_spec.ts
@@ -26,7 +26,12 @@ import {
 import {Injector, bind} from 'angular2/core';
 import {isBlank, isPresent} from 'angular2/src/core/facade/lang';
 
-import {AppProtoView, AppView, AppProtoViewMergeInfo} from 'angular2/src/core/linker/view';
+import {
+  AppProtoView,
+  AppView,
+  AppProtoViewMergeInfo,
+  ViewType
+} from 'angular2/src/core/linker/view';
 import {ElementBinder} from 'angular2/src/core/linker/element_binder';
 import {
   DirectiveBinding,
@@ -37,7 +42,7 @@ import {
 import {DirectiveResolver} from 'angular2/src/core/linker/directive_resolver';
 import {Component} from 'angular2/src/core/metadata';
 import {AppViewManagerUtils} from 'angular2/src/core/linker/view_manager_utils';
-import {ViewType, RenderViewWithFragments} from 'angular2/src/core/render/render';
+import {RenderViewWithFragments} from 'angular2/src/core/render/render';
 
 export function main() {
   // TODO(tbosch): add more tests here!
diff --git a/modules_dart/transform/lib/src/transform/common/directive_metadata_reader.dart b/modules_dart/transform/lib/src/transform/common/directive_metadata_reader.dart
index 305754de50..9ef3440769 100644
--- a/modules_dart/transform/lib/src/transform/common/directive_metadata_reader.dart
+++ b/modules_dart/transform/lib/src/transform/common/directive_metadata_reader.dart
@@ -7,7 +7,7 @@ import 'package:angular2/src/core/compiler/directive_metadata.dart';
 import 'package:angular2/src/core/compiler/template_compiler.dart';
 import 'package:angular2/src/core/change_detection/change_detection.dart';
 import 'package:angular2/src/core/linker/interfaces.dart' show LifecycleHooks;
-import 'package:angular2/src/core/render/api.dart' show ViewEncapsulation;
+import 'package:angular2/src/core/metadata/view.dart' show ViewEncapsulation;
 import 'package:angular2/src/transform/common/annotation_matcher.dart';
 import 'package:angular2/src/transform/common/interface_matcher.dart';
 import 'package:barback/barback.dart' show AssetId;