Debugger support to display global variables

runtime/vm/debugger.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/debugger.h"
 
 #include "vm/code_generator.h"
 #include "vm/code_patcher.h"
 #include "vm/compiler.h"
 #include "vm/dart_entry.h"
@@ skipping 155 matching lines @@
 }
 
 
 RawScript* ActivationFrame::SourceScript() {
   const Function& func = DartFunction();
   const Class& cls = Class::Handle(func.owner());
   return cls.script();
 }
 
 
+RawLibrary* ActivationFrame::Library() {
+  const Function& func = DartFunction();
+  const Class& cls = Class::Handle(func.owner());
+  return cls.library();
+}
+
+
 void ActivationFrame::GetPcDescriptors() {
   if (pc_desc_.IsNull()) {
     const Function& func = DartFunction();
     ASSERT(!func.HasOptimizedCode());
     Code& code = Code::Handle(func.unoptimized_code());
     ASSERT(!code.IsNull());
     pc_desc_ = code.pc_descriptors();
     ASSERT(!pc_desc_.IsNull());
   }
 }
@@ skipping 908 matching lines @@
       field_name = field.name();
       field_value = GetStaticField(cls, field_name);
       field_list.Add(field_name);
       field_list.Add(field_value);
     }
   }
   return Array::MakeArray(field_list);
 }
 
 
-RawArray* Debugger::GetLibraryFields(const Library& lib) {
-  const GrowableObjectArray& field_list =
-      GrowableObjectArray::Handle(GrowableObjectArray::New(8));
+void Debugger::CollectLibraryFields(const GrowableObjectArray& field_list,
+                                    const Library& lib,
+                                    const String& prefix) {
   DictionaryIterator it(lib);
-  Object& entry = Object::Handle();
-  Field& field = Field::Handle();
-  Class& cls = Class::Handle();
-  String& field_name = String::Handle();
-  Object& field_value = Object::Handle();
+  Object& entry = Object::Handle(isolate_);

[srdjan, 2012/06/26 17:08:26]

Why all this isolates_ passing? Is there a performance problem here?

[hausner, 2012/06/26 17:32:12]

As discussed offline, it has a tiny impact on performance. Each parameterless
call to Handle() results in a call to get the current thread, which is not
entirely free. We all agree that this only makes sense when a pointer to the
isolate happens to be easily accessible.

+  Field& field = Field::Handle(isolate_);
+  Class& cls = Class::Handle(isolate_);
+  String& field_name = String::Handle(isolate_);
+  Object& field_value = Object::Handle(isolate_);
   while (it.HasNext()) {
     entry = it.GetNext();
     if (entry.IsField()) {
       field ^= entry.raw();
       cls = field.owner();
       ASSERT(field.is_static());
       field_name = field.name();
       field_value = GetStaticField(cls, field_name);
+      if (!prefix.IsNull()) {
+        field_name = String::Concat(prefix, field_name);
+      }
       field_list.Add(field_name);
       field_list.Add(field_value);
     }
   }
+}
+
+
+RawArray* Debugger::GetLibraryFields(const Library& lib) {
+  const GrowableObjectArray& field_list =
+      GrowableObjectArray::Handle(GrowableObjectArray::New(8));
+  CollectLibraryFields(field_list, lib, String::Handle(isolate_));
+  return Array::MakeArray(field_list);
+}
+
+
+RawArray* Debugger::GetGlobalFields(const Library& lib) {
+  const GrowableObjectArray& field_list =
+      GrowableObjectArray::Handle(GrowableObjectArray::New(8));

[srdjan, 2012/06/26 17:08:26]

Why 8? Maybe better to not pass the length if you do not know it. 

[hausner, 2012/06/26 17:32:12]

No particular reason. It just eliminates the first two or so reallocate and copy
steps. If I don't pass an initial size, then GrowableObjectArray will, and it
doesn't know any better either... :)

+  String& prefix_name = String::Handle(isolate_);
+  CollectLibraryFields(field_list, lib, prefix_name);
+  Library& imported = Library::Handle(isolate_);
+  intptr_t num_imports = lib.num_imports();
+  for (int i = 0; i < num_imports; i++) {
+    imported = lib.ImportAt(i);
+    ASSERT(!imported.IsNull());
+    CollectLibraryFields(field_list, imported, prefix_name);
+  }
+  LibraryPrefix& prefix = LibraryPrefix::Handle(isolate_);
+  LibraryPrefixIterator it(lib);
+  while (it.HasNext()) {
+    prefix = it.GetNext();
+    prefix_name = prefix.name();
+    ASSERT(!prefix_name.IsNull());
+    prefix_name = String::Concat(prefix_name,
+                                 String::Handle(isolate_, String::New(".")));
+    for (int i = 0; i < prefix.num_libs(); i++) {
+      imported = prefix.GetLibrary(i);
+      CollectLibraryFields(field_list, imported, prefix_name);
+    }
+  }
   return Array::MakeArray(field_list);
 }
 
 
 void Debugger::VisitObjectPointers(ObjectPointerVisitor* visitor) {
   ASSERT(visitor != NULL);
   SourceBreakpoint* bpt = src_breakpoints_;
   while (bpt != NULL) {
     bpt->VisitObjectPointers(visitor);
     bpt = bpt->next();
@@ skipping 336 matching lines @@
 }
 
 
 void Debugger::RegisterCodeBreakpoint(CodeBreakpoint* bpt) {
   ASSERT(bpt->next() == NULL);
   bpt->set_next(code_breakpoints_);
   code_breakpoints_ = bpt;
 }
 
 }  // namespace dart