Fix Object method tearoffs

pkg/dev_compiler/lib/src/compiler/code_generator.dart

Index: pkg/dev_compiler/lib/src/compiler/code_generator.dart
diff --git a/pkg/dev_compiler/lib/src/compiler/code_generator.dart b/pkg/dev_compiler/lib/src/compiler/code_generator.dart
index 2896c7aab3125bd45c31303d678a205076ff3844..fc4af138944fd14f52600581d94202f810cb3e14 100644
--- a/pkg/dev_compiler/lib/src/compiler/code_generator.dart
+++ b/pkg/dev_compiler/lib/src/compiler/code_generator.dart
@@ -4802,7 +4802,8 @@ class CodeGenerator extends Object
         result = _callHelper('bind(this, #, #)',
             [safeName, _emitTargetAccess(jsTarget, name, accessor)]);
       } else if (_isObjectMemberCall(target, memberName)) {
-        result = _callHelper('bind(#, #, #.#)',
+        result = _callHelper(

[Jennifer Messerly, 2017/06/22 22:13:03]

since we have just 4 of these (see `isObjectMember`), we could avoid the rest
args/concat/apply?

        var fn = js.call(
            memberName == 'noSuchMethod'
                ? 'function(i) { return #.#(this, i); }'
                : 'function() { return #.#(this); }',
            [_runtimeModule, memberName]);
        result = _callHelper(
            'bind(#, #, #)', [jsTarget, _propertyName(memberName), fn]);

... an idea.

Another idea is we could have a custom bind method for Object method tearoffs,
but IDK if it's worth it.

[vsm, 2017/06/22 23:01:01]

I don't think tearing these off is that common.  Went with your suggested code.

+            'bind(#, #, function (...args) { return #.#.apply(null, [this].concat(args)); })',
             [jsTarget, _propertyName(memberName), _runtimeModule, memberName]);
       } else {
         result = _callHelper('bind(#, #)', [jsTarget, safeName]);