Fixes a couple problems with GC of unoptimized code.

runtime/vm/raw_object.cc

 // Copyright (c) 2012, 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.
 
 #include "vm/raw_object.h"
 
 #include "vm/class_table.h"
 #include "vm/dart.h"
 #include "vm/freelist.h"
 #include "vm/isolate.h"
@@ skipping 348 matching lines @@
 }
 
 
 intptr_t RawRedirectionData::VisitRedirectionDataPointers(
     RawRedirectionData* raw_obj, ObjectPointerVisitor* visitor) {
   visitor->VisitPointers(raw_obj->from(), raw_obj->to());
   return RedirectionData::InstanceSize();
 }
 
 
+bool RawFunction::MayTrySkippingCode(RawFunction* raw_fun) {
+  Function fn;
+  fn = raw_fun;
+
+  Code code;
+  code = fn.CurrentCode();
+
+  // TODO(zra): If we would return true here, instead halve the usage counter,
+  //            and return true only if the counter is then zero. Otherwise
+  //            return false. For now, while testing, always skip the function's
+  //            code pointer when usage_counter > 0.
+
+  return fn.HasCode() &&  // Not already detached.
+         !code.is_optimized() &&
+         !fn.HasBreakpoint() &&
+         (fn.usage_counter() > 0);
+}
+
+
 intptr_t RawFunction::VisitFunctionPointers(RawFunction* raw_obj,
                                             ObjectPointerVisitor* visitor) {
-  visitor->VisitPointers(raw_obj->from(), raw_obj->to());
+  if (visitor->visit_function_code() ||
+      !RawFunction::MayTrySkippingCode(raw_obj)) {
+    visitor->VisitPointers(raw_obj->from(), raw_obj->to());
+  } else {
+    GrowableArray<RawFunction*>* sfga = visitor->skipped_code_functions();
+    ASSERT(sfga != NULL);
+    sfga->Add(raw_obj);
+    visitor->VisitPointers(raw_obj->from(), raw_obj->to_no_code());
+  }
   return Function::InstanceSize();
 }
 
 
 intptr_t RawField::VisitFieldPointers(RawField* raw_obj,
                                       ObjectPointerVisitor* visitor) {
   visitor->VisitPointers(raw_obj->from(), raw_obj->to());
   return Field::InstanceSize();
 }
 
@@ skipping 419 matching lines @@
 
 intptr_t RawMirrorReference::VisitMirrorReferencePointers(
     RawMirrorReference* raw_obj, ObjectPointerVisitor* visitor) {
   // Make sure that we got here with the tagged pointer as this.
   ASSERT(raw_obj->IsHeapObject());
   visitor->VisitPointers(raw_obj->from(), raw_obj->to());
   return MirrorReference::InstanceSize();
 }
 
 }  // namespace dart