Move EventManager to AppViewUtils.

Move listen calls out of renderer for Renderer deprecation.

*BREAKING API CHANGE*
Renderer marked deprecated.
To listen to events use element.onEventName or element.on['customevent'] instead of injecting Renderer to call listen.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=138762470

Author: ferhatb

lib/platform/common_dom.dart

@@ -5,7 +5,6 @@ library angular2.platform.common_dom;
 export "package:angular2/src/core/render/api.dart" show sharedStylesHost;
 export "package:angular2/src/platform/dom/dom_adapter.dart"
     show DOM, setRootDomAdapter, DomAdapter;
-export "package:angular2/src/platform/dom/dom_renderer.dart" show DomRenderer;
 export "package:angular2/src/platform/dom/dom_tokens.dart" show DOCUMENT;
 export "package:angular2/src/platform/dom/events/dom_events.dart"
     show DomEventsPlugin;

lib/src/compiler/view_compiler/constants.dart

@@ -77,7 +77,6 @@ class ViewConstructorVars {
 }
 
 class ViewProperties {
-  static var renderer = new o.ReadClassMemberExpr('renderer');
   static var projectableNodes = new o.ReadClassMemberExpr('projectableNodes');
 }
 

lib/src/compiler/view_compiler/event_binder.dart

@@ -4,7 +4,7 @@ import '../template_ast.dart' show BoundEventAst, DirectiveAst;
 import 'compile_binding.dart' show CompileBinding;
 import 'compile_element.dart' show CompileElement;
 import 'compile_method.dart' show CompileMethod;
-import 'constants.dart' show EventHandlerVars, ViewProperties;
+import 'constants.dart' show EventHandlerVars;
 import 'expression_converter.dart' show convertCdStatementToIr;
 
 class CompileEventListener {
@@ -93,11 +93,13 @@ class CompileEventListener {
       new o.ReadClassMemberExpr(_methodName)
           .callMethod(o.BuiltinMethod.bind, [o.THIS_EXPR])
     ]);
-    o.Expression listenExpr = ViewProperties.renderer.callMethod('listen', [
+
+    o.Expression listenExpr = new o.InvokeMemberMethodExpr('listen', [
       this.compileElement.renderNode,
       o.literal(this.eventName),
       eventListener
     ]);
+
     this
         .compileElement
         .view

lib/src/compiler/view_compiler/property_binder.dart

@@ -137,9 +137,9 @@ void bindAndWriteToRenderer(List<BoundElementPropertyAst> boundProps,
           updateStmts.add(
               logBindingUpdateStmt(renderNode, boundProp.name, currValExpr));
         }
-        updateStmts.add(new o.ReadClassMemberExpr('renderer').callMethod(
-            renderMethod,
-            [renderNode, o.literal(boundProp.name), renderValue]).toStmt());
+        updateStmts.add(new o.InvokeMemberMethodExpr(
+                'setProp', [renderNode, o.literal(boundProp.name), renderValue])
+            .toStmt());
         break;
       case PropertyBindingType.Attribute:
         // For attributes convert value to a string.
@@ -408,7 +408,7 @@ void bindToUpdateMethod(
 
 o.Statement logBindingUpdateStmt(
     o.Expression renderNode, String propName, o.Expression value) {
-  return o.THIS_EXPR.prop('renderer').callMethod('setBindingDebugInfo', [
+  return new o.InvokeMemberMethodExpr('setBindingDebugInfo', [
     renderNode,
     o.literal('ng-reflect-${propName}'),
     value.isBlank().conditional(o.NULL_EXPR, value.callMethod('toString', []))

lib/src/compiler/view_compiler/view_compiler_utils.dart

@@ -18,8 +18,6 @@ const NAMESPACE_URIS = const {
 
 // Template Anchor comment.
 const TEMPLATE_COMMENT_TEXT = 'template bindings={}';
-// RegExp to match anchor comment when logging bindings for debugging.
-var TEMPLATE_BINDINGS_EXP = new RegExp(r'^template bindings=(.*)$');
 
 // Creates method parameters list for AppView set attribute calls.
 List<o.Expression> createSetAttributeParams(

lib/src/core/linker/app_view.dart

@@ -16,6 +16,7 @@ import 'element_injector.dart' show ElementInjector;
 import 'exceptions.dart' show ViewDestroyedException;
 import 'view_ref.dart' show ViewRefImpl;
 import 'view_type.dart' show ViewType;
+import 'dart:js_util' as js_util;
 
 export 'package:angular2/src/core/change_detection/component_state.dart';
 
@@ -463,6 +464,20 @@ abstract class AppView<T> {
     }
     DomRootRenderer.isDirty = true;
   }
+
+  Function listen(dynamic renderElement, String name, Function callback) {
+    return appViewUtils.eventManager.addEventListener(renderElement, name,
+        (Event event) {
+      var result = callback(event);
+      if (identical(result, false)) {
+        event.preventDefault();
+      }
+    });
+  }
+
+  void setProp(Element element, String name, Object value) {
+    js_util.setProperty(element, name, value);
+  }
 }
 
 dynamic _findLastRenderNode(dynamic node) {

lib/src/core/linker/app_view_utils.dart

@@ -9,7 +9,8 @@ import 'package:angular2/src/core/security.dart' show SafeValue;
 import 'package:angular2/src/core/security.dart';
 import 'package:angular2/src/facade/exceptions.dart' show BaseException;
 import 'package:angular2/src/facade/lang.dart' show looseIdentical;
-
+import 'package:angular2/src/platform/dom/events/event_manager.dart'
+    show EventManager;
 import 'exceptions.dart' show ExpressionChangedAfterItHasBeenCheckedException;
 
 /// Function called when a view is destroyed.
@@ -24,6 +25,7 @@ AppViewUtils appViewUtils;
 class AppViewUtils {
   RootRenderer _renderer;
   String _appId;
+  EventManager eventManager;
   static int _nextCompTypeId = 0;
 
   /// Whether change detection should throw an exception when a change is
@@ -34,7 +36,8 @@ class AppViewUtils {
   static int _throwOnChangesCounter = 0;
   SanitizationService sanitizer;
 
-  AppViewUtils(this._renderer, @Inject(APP_ID) this._appId, this.sanitizer);
+  AppViewUtils(this._renderer, @Inject(APP_ID) this._appId, this.sanitizer,
+      this.eventManager);
 
   /// Used by the generated code.
   RenderComponentType createRenderComponentType(

lib/src/core/render/api.dart

@@ -98,6 +98,7 @@ abstract class RenderDebugInfo {
   String get source;
 }
 
+@Deprecated('Use dart:html')
 abstract class Renderer {
   dynamic selectRootElement(
       dynamic /* String | dynamic */ selectorOrNode, RenderDebugInfo debugInfo);
@@ -106,17 +107,12 @@ abstract class Renderer {
   void attachViewAfter(dynamic node, List<dynamic> viewRootNodes);
   void detachView(List<dynamic> viewRootNodes);
   void destroyView(dynamic hostElement, List<dynamic> viewAllNodes);
-  Function listen(dynamic renderElement, String name, Function callback);
   void setElementProperty(
       dynamic renderElement, String propertyName, dynamic propertyValue);
   @Deprecated("Use dart:html Element attributes and setAttribute.")
   void setElementAttribute(
       dynamic renderElement, String attributeName, String attributeValue);
 
-  /// Used only in debug mode to serialize property changes to dom nodes as
-  /// attributes.
-  void setBindingDebugInfo(
-      dynamic renderElement, String propertyName, String propertyValue);
   void setElementClass(dynamic renderElement, String className, bool isAdd);
   @Deprecated("Use dart:html Element.style instead")
   void setElementStyle(

lib/src/debug/debug_app_view.dart

@@ -1,4 +1,5 @@
 import 'dart:html';
+import 'dart:convert';
 
 import 'package:angular2/src/core/change_detection/change_detection.dart'
     show ChangeDetectionStrategy, ChangeDetectorState;
@@ -14,7 +15,12 @@ import 'package:angular2/src/core/render/api.dart' show RenderComponentType;
 import 'package:angular2/src/debug/debug_context.dart'
     show StaticNodeDebugInfo, DebugContext;
 import 'package:angular2/src/debug/debug_node.dart'
-    show DebugElement, DebugNode, getDebugNode, indexDebugNode;
+    show
+        DebugElement,
+        DebugNode,
+        getDebugNode,
+        indexDebugNode,
+        DebugEventListener;
 import 'package:angular2/src/platform/dom/dom_renderer.dart'
     show DomRootRenderer;
 
@@ -24,6 +30,10 @@ export 'package:angular2/src/debug/debug_context.dart'
 
 WtfScopeFn _scope_check = wtfCreateScope('AppView#check(ascii id)');
 
+// RegExp to match anchor comment when logging bindings for debugging.
+RegExp _TEMPLATE_BINDINGS_EXP = new RegExp(r'^template bindings=(.*)$');
+const _TEMPLATE_COMMENT_TEXT = 'template bindings={}';
+
 class DebugAppView<T> extends AppView<T> {
   static bool profilingEnabled = false;
 
@@ -142,6 +152,31 @@ class DebugAppView<T> extends AppView<T> {
     };
   }
 
+  Function listen(dynamic renderElement, String name, Function callback) {
+    var debugEl = getDebugNode(renderElement);
+    if (debugEl != null) {
+      debugEl.listeners.add(new DebugEventListener(name, callback));
+    }
+    return super.listen(renderElement, name, callback);
+  }
+
+  /// Used only in debug mode to serialize property changes to dom nodes as
+  /// attributes.
+  void setBindingDebugInfo(
+      dynamic renderElement, String propertyName, String propertyValue) {
+    if (renderElement is Comment) {
+      var existingBindings = _TEMPLATE_BINDINGS_EXP
+          .firstMatch(renderElement.text.replaceAll(new RegExp(r'\n'), ''));
+      var parsedBindings = JSON.decode(existingBindings[1]);
+      parsedBindings[propertyName] = propertyValue;
+      String debugStr =
+          const JsonEncoder.withIndent('  ').convert(parsedBindings);
+      renderElement.text = _TEMPLATE_COMMENT_TEXT.replaceFirst('{}', debugStr);
+    } else {
+      setAttr(renderElement, propertyName, propertyValue);
+    }
+  }
+
   /// Sets up current debug context to node so that failures can be associated
   /// with template source location and DebugElement.
   DebugContext dbg(num nodeIndex, num rowNum, num colNum) =>

lib/src/debug/debug_node.dart

@@ -5,16 +5,16 @@ import "package:angular2/src/core/zone/ng_zone.dart" show NgZone;
 
 typedef bool Predicate<T>(T item);
 
-class EventListener {
+class DebugEventListener {
   String name;
   Function callback;
-  EventListener(this.name, this.callback);
+  DebugEventListener(this.name, this.callback);
 }
 
 class DebugNode {
   RenderDebugInfo _debugInfo;
   dynamic nativeNode;
-  List<EventListener> listeners;
+  List<DebugEventListener> listeners;
   DebugElement parent;
   DebugNode(dynamic nativeNode, DebugNode parent, this._debugInfo) {
     this.nativeNode = nativeNode;