perf(AppView): Move host events from call sites to component implementation.

Move largetemplates codesize benchmark to codegen, fix new errors caught by codegen.
Remove unused removeListener closure from EventsPlugin.

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

Author: ferhatb

lib/compiler.dart

@@ -30,3 +30,4 @@ export "src/compiler/compiler.dart"
         CompileDirectiveMetadata,
         CompilePipeMetadata;
 export "src/compiler/template_ast.dart";
+export "src/compiler/view_compiler/parse_utils.dart";

lib/src/compiler/expression_parser/ast.dart

@@ -241,7 +241,7 @@ class ASTWithSource extends AST {
   }
 
   @override
-  String toString() => '''${ this . source} in ${ this . location}''';
+  String toString() => '$source in $location';
 }
 
 class TemplateBinding {

lib/src/compiler/expression_parser/parser.dart

@@ -73,11 +73,13 @@ class TemplateBindingParseResult {
 
 @Injectable()
 class Parser {
-  Lexer _lexer;
+  final Lexer _lexer;
+
   Parser(this._lexer);
+
   ASTWithSource parseAction(String input, dynamic location) {
     this._checkNoInterpolation(input, location);
-    var tokens = this._lexer.tokenize(this._stripComments(input));
+    var tokens = _lexer.tokenize(this._stripComments(input));
     var ast = new _ParseAST(input, location, tokens, true).parseChain();
     return new ASTWithSource(ast, input, location);
   }

lib/src/compiler/template_ast.dart

@@ -6,9 +6,9 @@ import 'compile_metadata.dart'
         CompileDirectiveMetadata,
         CompileTokenMetadata,
         CompileProviderMetadata;
-import 'expression_parser/ast.dart'
-    show AST, ASTWithSource, ImplicitReceiver, MethodCall, PropertyRead;
-import 'view_compiler/constants.dart';
+import 'expression_parser/ast.dart' show AST;
+import 'package:angular2/src/compiler/view_compiler/parse_utils.dart'
+    show handlerTypeFromExpression, HandlerType;
 
 /// An Abstract Syntax Tree node representing part of a parsed Angular template.
 abstract class TemplateAst {
@@ -74,8 +74,6 @@ abstract class ElementProviderUsage {
   bool hasNonLocalRequest(ProviderAst providerAst);
 }
 
-enum HandlerType { simpleNoArgs, simpleOneArg, notSimple }
-
 /// A binding for an element event (e.g. (event)='handler()').
 class BoundEventAst implements TemplateAst {
   String name;
@@ -86,48 +84,7 @@ class BoundEventAst implements TemplateAst {
     return visitor.visitEvent(this, context);
   }
 
-  /// Classifies this event binding by it's form.
-  ///
-  /// The simple form looks like
-  ///
-  ///     (event)="handler($event)"
-  ///
-  /// or
-  ///
-  ///     (event)="handler()"
-  ///
-  /// Since these types of handlers are so common, we can optimize the code
-  /// generated for them.
-  HandlerType get handlerType {
-    var eventHandler = handler;
-    if (eventHandler is ASTWithSource) {
-      eventHandler = (eventHandler as ASTWithSource).ast;
-    }
-    if (eventHandler is! MethodCall) {
-      return HandlerType.notSimple;
-    }
-    var call = eventHandler as MethodCall;
-    if (call.receiver is! ImplicitReceiver) {
-      return HandlerType.notSimple;
-    }
-    if (call.args.isEmpty) {
-      return HandlerType.simpleNoArgs;
-    }
-    if (call.args.length != 1) {
-      return HandlerType.notSimple;
-    }
-    var singleArg = call.args.single;
-    if (singleArg is! PropertyRead) {
-      return HandlerType.notSimple;
-    }
-    var property = singleArg as PropertyRead;
-    if (property.name == EventHandlerVars.event.name &&
-        property.receiver is ImplicitReceiver) {
-      return HandlerType.simpleOneArg;
-    } else {
-      return HandlerType.notSimple;
-    }
-  }
+  HandlerType get handlerType => handlerTypeFromExpression(handler);
 }
 
 /// A reference declaration on an element (e.g. let someName='expression').

lib/src/compiler/view_compiler/event_binder.dart

@@ -1,11 +1,12 @@
 import '../compile_metadata.dart' show CompileDirectiveMetadata;
 import '../output/output_ast.dart' as o;
-import '../template_ast.dart' show BoundEventAst, DirectiveAst, HandlerType;
+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;
 import 'expression_converter.dart' show convertCdStatementToIr;
+import 'parse_utils.dart';
 
 /// Generates code to listen to a single eventName on a [CompileElement].
 ///
@@ -164,7 +165,8 @@ class CompileEventListener {
 }
 
 List<CompileEventListener> collectEventListeners(List<BoundEventAst> hostEvents,
-    List<DirectiveAst> dirs, CompileElement compileElement) {
+    List<DirectiveAst> dirs, CompileElement compileElement,
+    {bool includeComponentOutputs: true}) {
   List<CompileEventListener> eventListeners = [];
   for (var hostEvent in hostEvents) {
     compileElement.view.bindings
@@ -177,6 +179,10 @@ List<CompileEventListener> collectEventListeners(List<BoundEventAst> hostEvents,
   for (var directiveAst in dirs) {
     i++;
     var directiveInstance = compileElement.directiveInstances[i];
+    if (includeComponentOutputs == false &&
+        directiveAst.directive.isComponent) {
+      continue;
+    }
     for (var hostEvent in directiveAst.hostEvents) {
       compileElement.view.bindings
           .add(new CompileBinding(compileElement, hostEvent));
@@ -218,12 +224,6 @@ o.Expression convertStmtIntoExpression(o.Statement stmt) {
   return null;
 }
 
-final RegExp _eventNameRegExp = new RegExp(r'[^a-zA-Z_]');
-
-String sanitizeEventName(String name) {
-  return name.replaceAll(_eventNameRegExp, '_');
-}
-
 o.Expression _extractFunction(o.Expression returnExpr) {
   assert(returnExpr is o.InvokeMethodExpr);
   var callExpr = returnExpr as o.InvokeMethodExpr;

lib/src/compiler/view_compiler/parse_utils.dart

@@ -0,0 +1,56 @@
+import '../expression_parser/ast.dart'
+    show AST, ASTWithSource, ImplicitReceiver, MethodCall, PropertyRead;
+import 'constants.dart';
+
+enum HandlerType { simpleNoArgs, simpleOneArg, notSimple }
+
+/// Classifies this event binding by it's form.
+///
+/// The simple form looks like
+///
+///     (event)="handler($event)"
+///
+/// or
+///
+///     (event)="handler()"
+///
+/// Since these types of handlers are so common, we can optimize the code
+/// generated for them.
+HandlerType handlerTypeFromExpression(AST handler) {
+  var eventHandler = handler;
+  if (eventHandler is ASTWithSource) {
+    eventHandler = (eventHandler as ASTWithSource).ast;
+  }
+  if (eventHandler is! MethodCall) {
+    return HandlerType.notSimple;
+  }
+  var call = eventHandler as MethodCall;
+  if (call.receiver is! ImplicitReceiver) {
+    return HandlerType.notSimple;
+  }
+  if (call.args.isEmpty) {
+    return HandlerType.simpleNoArgs;
+  }
+  if (call.args.length != 1) {
+    return HandlerType.notSimple;
+  }
+  var singleArg = call.args.single;
+  if (singleArg is! PropertyRead) {
+    return HandlerType.notSimple;
+  }
+  var property = singleArg as PropertyRead;
+  if (property.name == EventHandlerVars.event.name &&
+      property.receiver is ImplicitReceiver) {
+    return HandlerType.simpleOneArg;
+  } else {
+    return HandlerType.notSimple;
+  }
+}
+
+final RegExp _eventNameRegExp = new RegExp(r'[^a-zA-Z0-9_]');
+
+/// Sanitizes event name so it can be used to construct a class member
+/// handler method name.
+String sanitizeEventName(String name) {
+  return name.replaceAll(_eventNameRegExp, '_');
+}

lib/src/compiler/view_compiler/view_binder.dart

@@ -63,8 +63,10 @@ class ViewBinderVisitor implements TemplateAstVisitor {
 
   dynamic visitElement(ElementAst ast, dynamic context) {
     var compileElement = (view.nodes[_nodeIndex++] as CompileElement);
-    var eventListeners =
-        collectEventListeners(ast.outputs, ast.directives, compileElement);
+    var eventListeners = collectEventListeners(
+        ast.outputs, ast.directives, compileElement,
+        includeComponentOutputs: false);
+
     bindRenderInputs(ast.inputs, compileElement);
     bindRenderOutputs(eventListeners);
     var index = -1;

lib/src/compiler/view_compiler/view_builder.dart

@@ -28,6 +28,7 @@ import "../template_ast.dart"
         DirectiveAst,
         BoundDirectivePropertyAst,
         templateVisitAll;
+import '../expression_parser/parser.dart' show Parser;
 import "compile_element.dart" show CompileElement, CompileNode;
 import "compile_method.dart";
 import "compile_view.dart" show CompileView;
@@ -36,11 +37,15 @@ import "constants.dart"
         ChangeDetectionStrategyEnum,
         ChangeDetectorStateEnum,
         DetectChangesVars,
+        EventHandlerVars,
         InjectMethodVars,
         ViewConstructorVars,
         ViewEncapsulationEnum,
         ViewProperties,
         ViewTypeEnum;
+import 'expression_converter.dart';
+import 'event_binder.dart' show convertStmtIntoExpression;
+import 'parse_utils.dart';
 import 'property_binder.dart';
 import "view_compiler_utils.dart"
     show
@@ -69,6 +74,7 @@ class ViewCompileDependency {
 
 class ViewBuilderVisitor implements TemplateAstVisitor {
   final CompileView view;
+  final Parser parser;
   final List<ViewCompileDependency> targetDependencies;
   final StylesCompileResult stylesCompileResult;
   static Map<String, CompileIdentifierMetadata> tagNameToIdentifier;
@@ -79,8 +85,8 @@ class ViewBuilderVisitor implements TemplateAstVisitor {
   static final defaultDocVarName = 'doc';
   String docVarName;
 
-  ViewBuilderVisitor(
-      this.view, this.targetDependencies, this.stylesCompileResult) {
+  ViewBuilderVisitor(this.view, this.parser, this.targetDependencies,
+      this.stylesCompileResult) {
     tagNameToIdentifier ??= {
       'a': Identifiers.HTML_ANCHOR_ELEMENT,
       'area': Identifiers.HTML_AREA_ELEMENT,
@@ -495,7 +501,7 @@ class ViewBuilderVisitor implements TemplateAstVisitor {
 
     // Create a visitor for embedded view and visit all nodes.
     var embeddedViewVisitor = new ViewBuilderVisitor(
-        embeddedView, targetDependencies, stylesCompileResult);
+        embeddedView, parser, targetDependencies, stylesCompileResult);
     templateVisitAll(
         embeddedViewVisitor,
         ast.children,
@@ -725,10 +731,11 @@ o.Expression createStaticNodeDebugInfo(CompileNode node) {
 
 /// Generates output ast for a CompileView and returns a [ClassStmt] for the
 /// view of embedded template.
-o.ClassStmt createViewClass(CompileView view, o.Expression nodeDebugInfosVar) {
+o.ClassStmt createViewClass(
+    CompileView view, o.Expression nodeDebugInfosVar, Parser parser) {
   var viewConstructor = _createViewClassConstructor(view, nodeDebugInfosVar);
   var viewMethods = (new List.from([
-    new o.ClassMethod("build", [], generateCreateMethod(view),
+    new o.ClassMethod("build", [], generateCreateMethod(view, parser),
         o.importType(Identifiers.ComponentRef, null)),
     new o.ClassMethod(
         "injectorGetInternal",
@@ -922,7 +929,7 @@ o.Statement createViewFactory(CompileView view, o.ClassStmt viewClass) {
       .toDeclStmt(view.viewFactory.name, [o.StmtModifier.Final]);
 }
 
-List<o.Statement> generateCreateMethod(CompileView view) {
+List<o.Statement> generateCreateMethod(CompileView view, Parser parser) {
   o.Expression parentRenderNodeExpr = o.NULL_EXPR;
   var parentRenderNodeStmts = <o.Statement>[];
   bool isComponent = view.viewType == ViewType.COMPONENT;
@@ -954,8 +961,13 @@ List<o.Statement> generateCreateMethod(CompileView view) {
     o.literalArr(view.subscriptions)
   ]).toStmt());
 
-  if (isComponent &&
-      view.viewIndex == 0 &&
+  bool isComponentRoot = isComponent && view.viewIndex == 0;
+
+  if (isComponentRoot) {
+    _writeComponentHostEventListeners(view, parser, statements);
+  }
+
+  if (isComponentRoot &&
       view.component.changeDetection == ChangeDetectionStrategy.Stateful) {
     // Connect ComponentState callback to view.
     statements.add((new o.ReadClassMemberExpr('ctx')
@@ -980,6 +992,56 @@ List<o.Statement> generateCreateMethod(CompileView view) {
   return statements;
 }
 
+/// Writes shared event handler wiring for events that are directly defined
+/// on host property of @Component annotation.
+void _writeComponentHostEventListeners(
+    CompileView view, Parser parser, List<o.Statement> statements) {
+  CompileDirectiveMetadata component = view.component;
+  for (String eventName in component.hostListeners.keys) {
+    String handlerSource = component.hostListeners[eventName];
+    var handlerAst = parser.parseAction(handlerSource, '');
+    HandlerType handlerType = handlerTypeFromExpression(handlerAst);
+
+    var context = new o.ReadClassMemberExpr('ctx');
+    var actionExpr = convertStmtIntoExpression(
+        convertCdStatementToIr(view, context, handlerAst, false).last);
+    if (handlerType == HandlerType.notSimple) {
+      List<o.Statement> stmts = <o.Statement>[
+        new o.InvokeMemberMethodExpr('markPathToRootAsCheckOnce', []).toStmt(),
+        new o.ReturnStatement(actionExpr)
+      ];
+      String methodName = '_handle_${sanitizeEventName(eventName)}__';
+      view.eventHandlerMethods.add(new o.ClassMethod(
+          methodName,
+          [new o.FnParam(EventHandlerVars.event.name, o.importType(null))],
+          stmts,
+          o.BOOL_TYPE,
+          [o.StmtModifier.Private]));
+
+      o.Expression handlerExpr = new o.InvokeMemberMethodExpr(
+          'evt', [new o.ReadClassMemberExpr(methodName)]);
+      o.Expression listenExpr = new o.InvokeMemberMethodExpr('listen', [
+        new o.ReadClassMemberExpr(appViewRootElementName),
+        o.literal(eventName),
+        handlerExpr
+      ]);
+      statements.add(listenExpr.toStmt());
+    } else {
+      assert(actionExpr is o.InvokeMethodExpr);
+      var callExpr = actionExpr as o.InvokeMethodExpr;
+      var eventListener = new o.InvokeMemberMethodExpr(
+          'eventHandler${handlerType == HandlerType.simpleNoArgs ? 0 : 1}',
+          [new o.ReadPropExpr(callExpr.receiver, callExpr.name)]);
+      o.Expression listenExpr = new o.InvokeMemberMethodExpr('listen', [
+        new o.ReadClassMemberExpr(appViewRootElementName),
+        o.literal(eventName),
+        eventListener
+      ]);
+      statements.add(listenExpr.toStmt());
+    }
+  }
+}
+
 List<o.Statement> generateDetectChangesMethod(CompileView view) {
   var stmts = <o.Statement>[];
   if (view.detectChangesInInputsMethod.isEmpty &&