Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(77)

Unified Diff: lib/compiler/implementation/lib/native_helper.dart

Issue 11307009: Revert "Minifying renamer for classes, methods and instance variables." (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « lib/compiler/implementation/lib/js_helper.dart ('k') | utils/compiler/build_helper.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/compiler/implementation/lib/native_helper.dart
diff --git a/lib/compiler/implementation/lib/native_helper.dart b/lib/compiler/implementation/lib/native_helper.dart
index 4e0a96aadae532643bd2aab80358ca3728de94b3..1ff2e1c098d8dba355251d3427425d78844c55b7 100644
--- a/lib/compiler/implementation/lib/native_helper.dart
+++ b/lib/compiler/implementation/lib/native_helper.dart
@@ -189,31 +189,21 @@ dynamicBind(var obj,
String name,
var methods,
List arguments) {
- // The tag is related to the class name. E.g. the dart:html class
- // '_ButtonElement' has the tag 'HTMLButtonElement'. TODO(erikcorry): rename
- // getTypeNameOf to getTypeTag.
String tag = getTypeNameOf(obj);
- var hasOwnProperty = JS('var', 'Object.prototype.hasOwnProperty');
- var method = lookupDynamicClass(hasOwnProperty, methods, tag);
+ var method = JS('var', '#[#]', methods, tag);
if (method == null && _dynamicMetadata != null) {
- // Look at the inheritance data, getting the class tags and using them
- // to check the methods table for this method name.
for (int i = 0; i < arrayLength(_dynamicMetadata); i++) {
MetaInfo entry = arrayGet(_dynamicMetadata, i);
- if (JS('bool', '#.call(#, #)', hasOwnProperty, entry._set, tag)) {
- method = lookupDynamicClass(hasOwnProperty, methods, entry._tag);
- // Stop if we found it in the methods array.
+ if (JS('bool', '#', propertyGet(entry._set, tag))) {
+ method = propertyGet(methods, entry._tag);
if (method != null) break;
}
}
}
- // If we didn't find the method then look up in the Dart Object class, using
- // getTypeNameOf in case the minifier has renamed Object.
if (method == null) {
- String nameOfObjectClass = getTypeNameOf(const Object());
- method = lookupDynamicClass(hasOwnProperty, methods, nameOfObjectClass);
+ method = propertyGet(methods, 'Object');
}
var proto = JS('var', 'Object.getPrototypeOf(#)', obj);
@@ -233,24 +223,13 @@ dynamicBind(var obj,
proto, name, name);
}
- if (JS('bool', '!#.call(#, #)', hasOwnProperty, proto, name)) {
+ if (JS('bool', '!#.hasOwnProperty(#)', proto, name)) {
defineProperty(proto, name, method);
}
return JS('var', '#.apply(#, #)', method, obj, arguments);
}
-// For each method name and class inheritance subtree, we use an ordinary JS
-// object as a hash map to store the method for each class. Entries are added
-// in native_emitter.dart (see dynamicName). In order to avoid the class names
-// clashing with the method names on Object.prototype (needed for native
-// objects) we must always use hasOwnProperty.
-var lookupDynamicClass(var hasOwnProperty, var methods, String className) {
- return JS('bool', '#.call(#, #)', hasOwnProperty, methods, className) ?
- JS('var', '#[#]', methods, className) :
- null;
-}
-
/**
* Code for doing the dynamic dispatch on JavaScript prototypes that are not
* available at compile-time. Each property of a native Dart class
@@ -278,13 +257,8 @@ dynamicFunction(name) {
var methods = JS('var', '{}');
// If there is a method attached to the Dart Object class, use it as
// the method to call in case no method is registered for that type.
- var dartMethod =
- JS('var', 'Object.getPrototypeOf(#)[#]', const Object(), name);
- // Take the method from the Dart Object class if we didn't find it yet and it
- // is there.
- if (dartMethod != null) {
- JS('void', '#["Object"] = #', methods, dartMethod);
- }
+ var dartMethod = JS('var', 'Object.getPrototypeOf(#)[#]', const Object(), name);
+ if (dartMethod != null) propertySet(methods, 'Object', dartMethod);
var bind = JS('var',
'function() {'
« no previous file with comments | « lib/compiler/implementation/lib/js_helper.dart ('k') | utils/compiler/build_helper.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698