Speed up interop patch file creation

sdk/lib/js/dartium/js_dartium.dart

 // Copyright (c) 2013, 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.
 
 /**
  * Support for interoperating with JavaScript.
  *
  * This library provides access to JavaScript objects from Dart, allowing
  * Dart code to get and set properties, and call methods of JavaScript objects
  * and invoke JavaScript functions. The library takes care of converting
@@ skipping 77 matching lines @@
 library dart.js;
 
 import 'dart:collection' show ListMixin;
 import 'dart:nativewrappers';
 import 'dart:math' as math;
 import 'dart:mirrors' as mirrors;
 import 'dart:html' as html;
 import 'dart:html_common' as html_common;
 import 'dart:indexed_db' as indexed_db;
 import 'dart:typed_data';
+import 'dart:core';
+
+import 'cached_patches.dart';
 
 // Pretend we are always in checked mode as we aren't interested in users
 // running Dartium code outside of checked mode.
 @Deprecated("Internal Use Only")
 final bool CHECK_JS_INVOCATIONS = true;
 
 final String _DART_RESERVED_NAME_PREFIX = r'JS$';
 
 String _stripReservedNamePrefix(String name) =>
     name.startsWith(_DART_RESERVED_NAME_PREFIX)
@@ skipping 163 matching lines @@
               .putIfAbsent(symbol, () => new _DeclarationSet())
               .add(declaration);
         }
       }
     });
   }
 }
 
 _finalizeJsInterfaces() native "Js_finalizeJsInterfaces";
 
+// Create the files for the generated Dart files from IDLs.  These only change
+// when browser's Dart files change (dart:*).
+String createCachedPatchesFile() {

[Jacob, 2016/03/28 14:23:58]

can you keep this method out of dart:js which is a public library?
At least this method needs to prefixed with
@Deprecated

[terry, 2016/03/28 14:45:31]

Added @Deprecated("Internal Use Only")
Done.

+  var patches = _generateInteropPatchFiles(['dart:html',
+                                            'dart:indexed_db',
+                                            'dart:web_gl',
+                                            'dart:web_sql',
+                                            'dart:svg',
+                                            'dart:web_audio']);
+  var sb = new StringBuffer();
+
+  sb.write("""
+
+// START_OF_CACHED_PATCHES
+// Copyright (c) 2013, 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.
+
+// DO NOT EDIT GENERATED FILE.
+
+library cached_patches;
+
+var cached_patches = {""");
+
+  for (var baseIndex = 0; baseIndex < patches.length; baseIndex += 3) {
+    var uri = patches[baseIndex + 0];
+    var uri_js_interop = patches[baseIndex + 1];
+    var source = patches[baseIndex + 2];
+
+    if (uri != 'dart:js') {
+      sb.write('"$uri": ["$uri", "$uri_js_interop", """$source"""],');
+    }
+  }
+
+  sb.write("""};
+// END_OF_CACHED_PATCHES
+  """);
+
+  return "$sb";
+}
+
 String _getJsName(mirrors.DeclarationMirror mirror) {
-  for (var annotation in mirror.metadata) {
-    if (mirrors.MirrorSystem.getName(annotation.type.simpleName) == "JS") {
-      mirrors.LibraryMirror library = annotation.type.owner;
-      var uri = library.uri;
-      // make sure the annotation is from package://js
-      if (uri.scheme == 'package' && uri.path == 'js/js.dart') {
+  if (_atJsType != null) {
+    for (var annotation in mirror.metadata) {
+      if (annotation.type.reflectedType == _atJsType) {
         try {
           var name = annotation.reflectee.name;
           return name != null ? name : "";
         } catch (e) {}
       }
     }
   }
   return null;
 }
 
 bool _isAnonymousClass(mirrors.ClassMirror mirror) {
   for (var annotation in mirror.metadata) {
     if (mirrors.MirrorSystem.getName(annotation.type.simpleName) ==
         "_Anonymous") {
       mirrors.LibraryMirror library = annotation.type.owner;
       var uri = library.uri;
       // make sure the annotation is from package://js
       if (uri.scheme == 'package' && uri.path == 'js/js.dart') {
         return true;
       }
     }
   }
   return false;
 }
 
 bool _hasJsName(mirrors.DeclarationMirror mirror) => _getJsName(mirror) != null;
 
+var _domNameType;
+
 bool hasDomName(mirrors.DeclarationMirror mirror) {
   var location = mirror.location;
   if (location == null || location.sourceUri.scheme != 'dart') return false;
   for (var annotation in mirror.metadata) {
     if (mirrors.MirrorSystem.getName(annotation.type.simpleName) == "DomName") {
       // We can't make sure the annotation is in dart: as Dartium believes it
       // is file://dart/sdk/lib/html/html_common/metadata.dart
       // instead of a proper dart: location.
       return true;
     }
@@ skipping 113 matching lines @@
 }
 
 bool _isExternal(mirrors.MethodMirror mirror) {
   // This try-catch block is a workaround for BUG:24834.
   try {
     return mirror.isExternal;
   } catch (e) {}
   return false;
 }
 
-List<String> _generateExternalMethods() {
+List<String> _generateExternalMethods(List<String> libraryPaths) {
   var staticCodegen = <String>[];
-  mirrors.currentMirrorSystem().libraries.forEach((uri, library) {
+
+  if (libraryPaths.length == 0) {
+    mirrors.currentMirrorSystem().libraries.forEach((uri, library) {
+      var library_name = "${uri.scheme}:${uri.path}";
+      if (cached_patches.containsKey(library_name)) {
+        var patch = cached_patches[library_name];
+        staticCodegen.addAll(patch);
+      } else {
+        _generateLibraryCodegen(uri, library, staticCodegen);
+      }
+    });    // End of library foreach
+  } else {
+    // Used to generate cached_patches.dart file for all IDL generated dart:
+    // files to the WebKit DOM.
+    for (var library_name in libraryPaths) {
+      var parts = library_name.split(':');
+      var uri = new Uri(scheme: parts[0], path: parts[1]);
+      var library = mirrors.currentMirrorSystem().libraries[uri];
+      _generateLibraryCodegen(uri, library, staticCodegen);
+    }
+  }
+
+  return staticCodegen;
+}
+
+_generateLibraryCodegen(uri, library, staticCodegen) {
+    // Is it a dart generated library?
+    var dartLibrary = uri.scheme == 'dart';  
+
     var sb = new StringBuffer();
     String jsLibraryName = _getJsName(library);
     library.declarations.forEach((name, declaration) {
       if (declaration is mirrors.MethodMirror) {
         if ((_hasJsName(declaration) || jsLibraryName != null) &&
             _isExternal(declaration)) {
           addMemberHelper(declaration, jsLibraryName, sb);
         }
       } else if (declaration is mirrors.ClassMirror) {
         mirrors.ClassMirror clazz = declaration;
-        var isDom = hasDomName(clazz);
+        var isDom = dartLibrary ? hasDomName(clazz) : false;
         var isJsInterop = _hasJsName(clazz);
         if (isDom || isJsInterop) {
           // TODO(jacobr): verify class implements JavaScriptObject.
           var className = mirrors.MirrorSystem.getName(clazz.simpleName);
           var classNameImpl = '${className}Impl';
           var sbPatch = new StringBuffer();
           if (isJsInterop) {
             String jsClassName = _getJsMemberName(clazz);
 
             jsInterfaceTypes.add(clazz);
@@ skipping 40 matching lines @@
                 sbPatch.write("  ");
                 addMemberHelper(
                     declaration,
                     (jsLibraryName != null && jsLibraryName.isNotEmpty)
                         ? "${jsLibraryName}.${jsClassName}"
                         : jsClassName,
                     sbPatch,
                     isStatic: true,
                     memberName: className);
               }
-            });
+            });   // End of clazz.declarations.forEach 
 
             clazz.staticMembers.forEach((memberName, member) {
               if (_isExternal(member)) {
                 sbPatch.write("  ");
                 addMemberHelper(
                     member,
                     (jsLibraryName != null && jsLibraryName.isNotEmpty)
                         ? "${jsLibraryName}.${jsClassName}"
                         : jsClassName,
                     sbPatch,
@@ skipping 38 matching lines @@
 
 /**
  * Placeholder object for cases where we need to determine exactly how many
  * args were passed to a function.
  */
 const ${_UNDEFINED_VAR} = const Object();
 
 ${sb}
 """);
     }
-  });
+}
 
-  return staticCodegen;
-}
+// Remember the @JS type to compare annotation type.
+var _atJsType = -1;
 
 /**
  * Generates part files defining source code for JSObjectImpl, all DOM classes
  * classes. This codegen  is needed so that type checks for all registered
  * JavaScript interop classes pass.
  */
-List<String> _generateInteropPatchFiles() {
-  var ret = _generateExternalMethods();
+List<String> _generateInteropPatchFiles(List<String> libraryPaths) {
+  // Cache the @JS Type.
+  if (_atJsType == -1) {
+    var uri = new Uri(scheme: "package", path: "js/js.dart");
+    var jsLibrary = mirrors.currentMirrorSystem().libraries[uri];
+    if (jsLibrary != null) {
+      // @ JS used somewhere.
+      var jsDeclaration = jsLibrary.declarations[new Symbol("JS")];
+      _atJsType = jsDeclaration.reflectedType;
+    } else {
+      // @ JS not used in any library.
+      _atJsType = null;
+    }
+  }
+
+  var ret = _generateExternalMethods(libraryPaths);
   var libraryPrefixes = new Map<mirrors.LibraryMirror, String>();
   var prefixNames = new Set<String>();
   var sb = new StringBuffer();
 
   var implements = <String>[];
   var implementsArray = <String>[];
   var implementsDom = <String>[];
   var listMirror = mirrors.reflectType(List);
   var functionMirror = mirrors.reflectType(Function);
   var jsObjectMirror = mirrors.reflectType(JSObject);
@@ skipping 957 matching lines @@
     var ret = _interopCaptureThisExpando[f];
     if (ret == null) {
       // TODO(jacobr): we could optimize this.
       ret = new JSFunction.create(
           new JsFunction.withThis(new _CreateDartFunctionForInterop(f)));
       _interopCaptureThisExpando[f] = ret;
     }
     return ret;
   }
 }