Implement closurization of regular methods in ObjectMirror.getField in the VM.

runtime/vm/resolver.cc

Index: runtime/vm/resolver.cc
diff --git a/runtime/vm/resolver.cc b/runtime/vm/resolver.cc
index 4021d7145243e8f8be5345bea24606b930fd6964..faad5dbd6a8053553245588cc1acd8d0a518df6d 100644
--- a/runtime/vm/resolver.cc
+++ b/runtime/vm/resolver.cc
@@ -138,6 +138,45 @@ RawFunction* Resolver::ResolveDynamicAnyArgs(
 }
 
 
+// TODO(13355): Remove.
+RawFunction* Resolver::ResolveDynamicAnyArgsAllowPrivate(
+    const Class& receiver_class,
+    const String& function_name) {
+  Class& cls = Class::Handle(receiver_class.raw());
+  if (FLAG_trace_resolving) {
+    OS::Print("ResolveDynamic '%s' for class %s\n",
+              function_name.ToCString(),
+              String::Handle(cls.Name()).ToCString());
+  }
+
+  const bool is_getter = Field::IsGetterName(function_name);
+  String& field_name = String::Handle();
+  if (is_getter) {
+    field_name ^= Field::NameFromGetter(function_name);
+  }
+
+  // Now look for an instance function whose name matches function_name
+  // in the class.
+  Function& function = Function::Handle();
+  while (function.IsNull() && !cls.IsNull()) {
+    function ^= cls.LookupDynamicFunctionAllowPrivate(function_name);
+
+    // Getter invocation might actually be a method extraction.
+    if (is_getter && function.IsNull()) {
+      function ^= cls.LookupDynamicFunctionAllowPrivate(field_name);
+      if (!function.IsNull()) {
+        // We were looking for the getter but found a method with the same name.
+        // Create a method extractor and return it.
+        function ^= CreateMethodExtractor(function_name, function);
+      }
+    }
+
+    cls = cls.SuperClass();

[siva, 2013/09/25 20:17:57]

Can we change the loop such that cls.cls.SuperClass() is not done once the
function is found i.e non null.

while (!cls.IsNull()) {
  function ^= ....;
  if (!function.IsNull()) {
    return function.raw();
  }
  if (is_getter) {
    function ^= .....
    if (!function.IsNull()) {
      function ^= ...
      return function.raw();
    }
    .....
  }
  cls = cls.SuperClass(); 
}
return Function::null();

[rmacnak, 2013/09/25 21:41:35]

Done, and likewise to ResolveDynamicAnyArgs.

+  }
+  return function.raw();
+}
+
+
 RawFunction* Resolver::ResolveStatic(const Library& library,
                                      const String& class_name,
                                      const String& function_name,