dart2js: mark FunctionElements as dependencies always

pkg/compiler/lib/src/deferred_load.dart

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library deferred_load;
 
 import 'common.dart';
 import 'common/backend_api.dart' show
     Backend;
 import 'common/tasks.dart' show
@@ skipping 246 matching lines @@
       Set<ConstantValue> constants,
       isMirrorUsage) {
 
     if (element.isErroneous) {
       // Erroneous elements are ignored.
       return;
     }
 
     /// Recursively collects all the dependencies of [type].
     void collectTypeDependencies(DartType type) {
+      // TODO(het): we would like to separate out types that are only needed for
+      // rti from types that are needed for their members.

[sigurdm, 2015/10/13 07:39:07]

I think johnniwinther is working on this.

       if (type is GenericType) {
         type.typeArguments.forEach(collectTypeDependencies);
       }
       if (type is FunctionType) {
         for (DartType argumentType in type.parameterTypes) {
           collectTypeDependencies(argumentType);
         }
         for (DartType argumentType in type.optionalParameterTypes) {
           collectTypeDependencies(argumentType);
         }
@@ skipping 57 matching lines @@
 
     // TODO(sigurdm): How is metadata on a patch-class handled?
     for (MetadataAnnotation metadata in element.metadata) {
       ConstantValue constant =
           backend.constants.getConstantValueForMetadata(metadata);
       if (constant != null) {
         constants.add(constant);
       }
     }
 
-    if (element is FunctionElement &&
-        compiler.resolverWorld.closurizedMembers.contains(element)) {
+    if (element is FunctionElement) {
       collectTypeDependencies(element.type);
     }
 
     if (element.isClass) {
       // If we see a class, add everything its live instance members refer
       // to.  Static members are not relevant, unless we are processing
       // extra dependencies due to mirrors.
       void addLiveInstanceMember(Element element) {
         if (!compiler.enqueuer.resolution.hasBeenProcessed(element)) return;
         if (!isMirrorUsage && !element.isInstanceMember) return;
@@ skipping 579 matching lines @@
     return result;
   }
 
   bool operator ==(other) {
     if (other is! _DeclaredDeferredImport) return false;
     return declaration == other.declaration;
   }
 
   int get hashCode => declaration.hashCode * 17;
 }