Get rid of library proto information for debugger libraries

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';
@@ skipping 491 matching lines @@
     }
 
     return properties.toList();
   }
 }
 
 /// Formatter for module Dart Library objects.
 class LibraryModuleFormatter implements Formatter {
   String libraryName;
 
-  accept(object, config) {
-    libraryName = dart.getDartLibraryName(object);
-    return libraryName != null;
-  }
+  accept(object, config) => dart.getDartLibraryName(object) != null;
 
   bool hasChildren(object) => true;
 
   String preview(object) {
     var libraryNames = libraryName.split('/');

[Jacob, 2016/08/02 23:57:43]

This seems broken. Should we lookup the library name here?
 I agree we should never have been caching the libraryName on the formatter.

[bmilligan, 2016/08/03 00:03:55]

I think this was leftover from before I was using symbols...It works now without
caching.

     // Library names are received with a repeat directory name, so strip the
     // last directory entry here to make the path cleaner. For example, the
     // library "third_party/dart/utf/utf" shoud display as
     // "third_party/dart/utf/".
     if (libraryNames.length > 1) {
       libraryNames[libraryNames.length - 1] = '';
     }
     return 'Library Module: ${libraryNames.join('/')}';
   }
 
@@ skipping 312 matching lines @@
     return children;
   }
 }
 
 /// This entry point is automatically invoked by the code generated by
 /// Dart Dev Compiler
 registerDevtoolsFormatter() {
   var formatters = [_devtoolsFormatter];
   JS('', 'dart.global.devtoolsFormatters = #', formatters);
 }