fix(di): removed default visibility

BREAKING CHANGE:
    Directives will use the Unbounded visibility by default, whereas before the change they used Self

modules/angular2/src/core/annotations_impl/annotations.ts

@@ -1,6 +1,6 @@
 import {CONST, CONST_EXPR} from 'angular2/src/facade/lang';
 import {List} from 'angular2/src/facade/collection';
-import {InjectableMetadata, self} from 'angular2/src/di/metadata';
+import {InjectableMetadata} from 'angular2/src/di/metadata';
 import {DEFAULT} from 'angular2/change_detection';
 
 /**
@@ -792,7 +792,7 @@ export class Directive extends InjectableMetadata {
     exportAs?: string,
     compileChildren?: boolean,
   } = {}) {
-    super(self);
+    super();
     this.selector = selector;
     this.properties = properties;
     this.events = events;

modules/angular2/src/core/compiler/element_injector.ts

@@ -26,8 +26,7 @@ import {
   CyclicDependencyError,
   resolveForwardRef,
   VisibilityMetadata,
-  DependencyProvider,
-  self
+  DependencyProvider
 } from 'angular2/di';
 import {
   InjectorInlineStrategy,

modules/angular2/src/di/binding.ts

@@ -16,7 +16,7 @@ import {
   InjectableMetadata,
   VisibilityMetadata,
   OptionalMetadata,
-  unbounded,
+  DEFAULT_VISIBILITY,
   DependencyMetadata
 } from './metadata';
 import {NoAnnotationError} from './exceptions';
@@ -30,7 +30,7 @@ export class Dependency {
               public properties: List<any>) {}
 
   static fromKey(key: Key): Dependency {
-    return new Dependency(key, false, _defaulVisiblity(key.token), []);
+    return new Dependency(key, false, DEFAULT_VISIBILITY, []);
   }
 }
 
@@ -397,18 +397,16 @@ function _extractToken(typeOrFunc, annotations /*List<any> | any*/, params: List
   var optional = false;
 
   if (!isArray(annotations)) {
-    return _createDependency(annotations, optional, _defaulVisiblity(annotations), depProps);
+    return _createDependency(annotations, optional, DEFAULT_VISIBILITY, depProps);
   }
 
-  var visibility = null;
-  var defaultVisibility = unbounded;
+  var visibility = DEFAULT_VISIBILITY;
 
   for (var i = 0; i < annotations.length; ++i) {
     var paramAnnotation = annotations[i];
 
     if (paramAnnotation instanceof Type) {
       token = paramAnnotation;
-      defaultVisibility = _defaulVisiblity(token);
 
     } else if (paramAnnotation instanceof InjectMetadata) {
       token = paramAnnotation.token;
@@ -427,10 +425,6 @@ function _extractToken(typeOrFunc, annotations /*List<any> | any*/, params: List
     }
   }
 
-  if (isBlank(visibility)) {
-    visibility = defaultVisibility;
-  }
-
   token = resolveForwardRef(token);
 
   if (isPresent(token)) {
@@ -439,16 +433,6 @@ function _extractToken(typeOrFunc, annotations /*List<any> | any*/, params: List
     throw new NoAnnotationError(typeOrFunc, params);
   }
 }
-function _defaulVisiblity(typeOrFunc) {
-  if (!(typeOrFunc instanceof Type)) return unbounded;
-
-  // TODO: vsavkin revisit this after clarifying lookup rules
-  if (!reflector.isReflectionEnabled()) return unbounded;
-
-  var f =
-      ListWrapper.filter(reflector.annotations(typeOrFunc), s => s instanceof InjectableMetadata);
-  return f.length === 0 ? unbounded : f[0].visibility;
-}
 
 function _createDependency(token, optional, visibility, depProps): Dependency {
   return new Dependency(Key.get(token), optional, visibility, depProps);

modules/angular2/src/di/decorators.dart

@@ -21,7 +21,7 @@ class Optional extends OptionalMetadata {
  * {@link InjectableMetadata}.
  */
 class Injectable extends InjectableMetadata {
-	const Injectable([VisibilityMetadata visibility = unbounded]): super(visibility);
+	const Injectable(): super();
 }
 
 /**

modules/angular2/src/di/decorators.ts

@@ -30,8 +30,8 @@ export interface OptionalFactory {
  * Factory for creating {@link InjectableMetadata}.
  */
 export interface InjectableFactory {
-  (visibility?: VisibilityMetadata): any;
-  new (visibility?: VisibilityMetadata): InjectableMetadata;
+  (): any;
+  new (): InjectableMetadata;
 }
 
 /**

modules/angular2/src/di/injector.ts

@@ -14,7 +14,7 @@ import {
 import {FunctionWrapper, Type, isPresent, isBlank, CONST_EXPR} from 'angular2/src/facade/lang';
 import {Key} from './key';
 import {resolveForwardRef} from './forward_ref';
-import {VisibilityMetadata, unbounded} from './metadata';
+import {VisibilityMetadata, DEFAULT_VISIBILITY} from './metadata';
 
 const _constructing = CONST_EXPR(new Object());
 const _notFound = CONST_EXPR(new Object());
@@ -521,7 +521,9 @@ export class Injector {
    *binding).
    * @returns an instance represented by the token. Throws if not found.
    */
-  get(token): any { return this._getByKey(Key.get(token), unbounded, false, PUBLIC_AND_PRIVATE); }
+  get(token): any {
+    return this._getByKey(Key.get(token), DEFAULT_VISIBILITY, false, PUBLIC_AND_PRIVATE);
+  }
 
   /**
    * Retrieves an instance from the injector.
@@ -530,7 +532,7 @@ export class Injector {
    * @returns an instance represented by the token. Returns `null` if not found.
    */
   getOptional(token): any {
-    return this._getByKey(Key.get(token), unbounded, true, PUBLIC_AND_PRIVATE);
+    return this._getByKey(Key.get(token), DEFAULT_VISIBILITY, true, PUBLIC_AND_PRIVATE);
   }
 
   /**

modules/angular2/src/di/metadata.ts

@@ -79,7 +79,7 @@ export class DependencyMetadata {
  */
 @CONST()
 export class InjectableMetadata {
-  constructor(public visibility: VisibilityMetadata = unbounded) {}
+  constructor() {}
 }
 
 /**
@@ -123,8 +123,6 @@ export class SelfMetadata extends VisibilityMetadata {
   toString(): string { return `@Self()`; }
 }
 
-export const self = CONST_EXPR(new SelfMetadata());
-
 /**
  * Specifies that an injector should retrieve a dependency from the direct parent.
  *
@@ -235,4 +233,4 @@ export class UnboundedMetadata extends VisibilityMetadata {
   toString(): string { return `@Unbounded(self: ${this.includeSelf}})`; }
 }
 
-export const unbounded = CONST_EXPR(new UnboundedMetadata({self: true}));
+export const DEFAULT_VISIBILITY = CONST_EXPR(new UnboundedMetadata({self: true}));