Fixed type error in MapEntry and added keyToStringConfig to display the toString() of a key in a Ma…

tool/input_sdk/private/debugger.dart

 // Copyright (c) 2015, 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.
 
 library dart._debugger;
 
 import 'dart:_foreign_helper' show JS;
 import 'dart:_runtime' as dart;
 import 'dart:core';
 import 'dart:collection';
 import 'dart:html' as html;
 
-/// Config object to pass to devtools to signal that an object should not be
+/// Config String to pass to devtools to signal that an object should not be
 /// formatted by the Dart formatter. This is used to specify that an Object
 /// should just be displayed using the regular JavaScript view instead of a
 /// custom Dart view. For example, this is used to display the JavaScript view
 /// of a Dart Function as a child of the regular Function object.
-const skipDartConfig = const Object();
+const skipDartConfig = "skipDartConfig";
+
+/// Config String to pass to devtools to signal that a map key object should
+/// have its toString() displayed by the Dart formatter.
+const keyToStringConfig = "keyToStringConfig";

[Jacob, 2016/06/03 23:11:50]

A cleaner solution would be to have an 
enum of configs
as only one config can be set at a time.
https://www.dartlang.org/docs/dart-up-and-running/ch02.html#enums

Add a enum entry that is 
Default to indicate no config is set.

[priscillalee, 2016/06/07 22:58:01]

Used a class JsonMLConfig that simulated an enum instead.

+
 final int maxIterableChildrenToDisplay = 50;
 
 var _devtoolsFormatter = new JsonMLFormatter(new DartFormatter());
 
 String _typeof(object) => JS('String', 'typeof #', object);
 
 List<String> getOwnPropertyNames(object) => JS('List<String>',
     'dart.list(Object.getOwnPropertyNames(#), #)', object, String);
 
 List getOwnPropertySymbols(object) =>
@@ skipping 49 matching lines @@
 bool hasMethod(object, String name) {
   try {
     return dart.hasMethod(object, name);
   } catch (e) {
     return false;
   }
 }
 
 /// [JsonMLFormatter] consumes [NameValuePair] objects and
 class NameValuePair {
-  NameValuePair({this.name, this.value, bool skipDart})
-      : skipDart = skipDart == true;
+  NameValuePair({this.name, this.value, bool skipDart, bool keyToString})

[Jacob, 2016/06/03 23:11:50]

change this to just take an entry of the 
enum JsonMLConfig
instead of multiple bools.

[priscillalee, 2016/06/07 22:58:01]

Done.

+      : skipDart = skipDart == true,
+        keyToString = keyToString == true;
 
   // Define equality and hashCode so that NameValuePair can be used
   // in a Set to dedupe entries with duplicate names.
   operator ==(other) => other is NameValuePair && other.name == name;
   int get hashCode => name.hashCode;
 
   final String name;
   final Object value;
   final bool skipDart;
+  final bool keyToString;
 }
 
 class MapEntry {
   MapEntry({this.key, this.value});
 
-  final String key;
+  final Object key;
   final Object value;
 }
 
 class ClassMetadata {
   ClassMetadata(this.object);
 
   final Object object;
 }
 
 class HeritageClause {
@@ skipping 71 matching lines @@
   // TODO(jacobr): define a SimpleFormatter base class that DartFormatter
   // implements if we decide to use this class elsewhere. We specify that the
   // type is DartFormatter here purely to get type checking benefits not because
   // this class is really intended to only support instances of type
   // DartFormatter.
   DartFormatter _simpleFormatter;
 
   JsonMLFormatter(this._simpleFormatter);
 
   header(object, config) {
     if (identical(config, skipDartConfig) || isNativeJavaScriptObject(object)) {

[Jacob, 2016/06/03 23:11:50]

switch to using == once you switch to an enum

[priscillalee, 2016/06/07 22:58:01]

Done.

       return null;
     }
 
     var c = _simpleFormatter.preview(object);
     if (c == null) return null;
 
+    if (identical(config, keyToStringConfig)) {

[Jacob, 2016/06/03 23:11:50]

switch to using == instead of identical

[priscillalee, 2016/06/07 22:58:01]

Done.

+      c = object.toString();
+    }
+
     // Indicate this is a Dart Object by using a Dart background color.
     // This is stylistically a bit ugly but it eases distinguishing Dart and
     // JS objects.
     var element = new JsonMLElement('span')
       ..setStyle('background-color: #d9edf7')
       ..createTextChild(c);
     return element.toJsonML();
   }
 
   bool hasBody(object) => _simpleFormatter.hasChildren(object);
@@ skipping 13 matching lines @@
         ..setStyle('color: rgb(136, 19, 145);');
       if (_typeof(child.value) == 'object' ||
           _typeof(child.value) == 'function') {
         nameSpan.addStyle("padding-left: 13px;");
 
         li.appendChild(nameSpan);
         var objectTag = li.createObjectTag(child.value);
         if (child.skipDart) {
           objectTag.addAttribute('config', skipDartConfig);
         }
+        if (child.keyToString) {
+          objectTag.addAttribute('config', keyToStringConfig);
+        }
         if (!_simpleFormatter.hasChildren(child.value)) {
           li.setStyle("padding-left: 13px;");
         }
       } else {
         li.setStyle("padding-left: 13px;");
         li.createChild('span')
           ..appendChild(nameSpan)
           ..createTextChild(safePreview(child.value));
       }
     }
@@ skipping 308 matching lines @@
   accept(object) => object is MapEntry;
 
   String preview(object) {
     MapEntry entry = object;
     return '${safePreview(entry.key)} => ${safePreview(entry.value)}';
   }
 
   bool hasChildren(object) => true;
 
   List<NameValuePair> children(object) => <NameValuePair>[
-        new NameValuePair(name: 'key', value: object.key),
+        new NameValuePair(name: 'key', value: object.key, keyToString: true),
         new NameValuePair(name: 'value', value: object.value)
       ];
 }
 
 /// Formatter for Dart Iterable objects including List and Set.
 class HeritageClauseFormatter implements Formatter {
   bool accept(object) => object is HeritageClause;
 
   String preview(object) {
     HeritageClause clause = object;
@@ skipping 12 matching lines @@
     return ret;
   }
 }
 
 /// This entry point is automatically invoked by the code generated by
 /// Dart Dev Compiler
 registerDevtoolsFormatter() {
   var formatters = [_devtoolsFormatter];
   JS('', 'dart.global.devtoolsFormatters = #', formatters);
 }