Debugger no longer causes top level variable initialization

runtime/vm/debugger.cc

Index: runtime/vm/debugger.cc
===================================================================
--- runtime/vm/debugger.cc	(revision 31556)
+++ runtime/vm/debugger.cc	(working copy)
@@ -1806,21 +1806,26 @@
   DictionaryIterator it(lib);
   Object& entry = Object::Handle(isolate_);
   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();
       if ((field_name.CharAt(0) == '_') && !include_private_fields) {
         // Skip library-private field.
         continue;
       }
-      field_value = GetStaticField(cls, field_name);
+      // If the field is not initialized yet, just report the value

[Jacob, 2014/01/09 17:51:12]

At what times will this occur?
It seems confusing for a user to see a static field with a value of null even
though it is initialized to a non-null value

E.g. 
class Pen {
  static DASHED = new LineStyle("dashed");
  static DOTTED = new LineStyle("dotted");
}

+      // to be null. We don't want to execute the implicit getter
+      // since it may have side effects.
+      if (field.value() == Object::sentinel().raw()) {
+        field_value = Object::null();

[srdjan, 2014/01/09 18:15:32]

Can we use a value like 'optimized out', or in this case 'uninitialized'.

+      } else {
+        field_value = field.value();
+      }
       if (!prefix.IsNull()) {
         field_name = String::Concat(prefix, field_name);
       }