add a class to read and parse libraries.dart file

add a class to read and parse libraries.dart file
Committed: https://code.google.com/p/dart/source/detail?r=11525

[danrubel, 2012-08-28 00:25:45]

https://chromiumcodereview.appspot.com/10875079/diff/1/compiler/java/com/goog...
File compiler/java/com/google/dart/compiler/SystemLibrariesReader.java (right):

https://chromiumcodereview.appspot.com/10875079/diff/1/compiler/java/com/goog...
compiler/java/com/google/dart/compiler/SystemLibrariesReader.java:123: }
Better would a a method that returns Map<String, DartLibrary> where DartLibrary
is a class that has methods such as
* getShortName()
* getPath()
* getPlatforms()
* getCategory()

https://chromiumcodereview.appspot.com/10875079/diff/1/compiler/javatests/com...
File compiler/javatests/com/google/dart/compiler/SystemLibrariesReaderTest.java
(right):

https://chromiumcodereview.appspot.com/10875079/diff/1/compiler/javatests/com...
compiler/javatests/com/google/dart/compiler/SystemLibrariesReaderTest.java:38:
continue;
Seems like we should fail if it cannot be resolved

https://chromiumcodereview.appspot.com/10875079/diff/1/compiler/javatests/com...
compiler/javatests/com/google/dart/compiler/SystemLibrariesReaderTest.java:43: 
Perhaps some tests to sanity check other aspects important to editor such as
platform and category.

[keertip, 2012-08-28 20:41:57]

PTAL

https://chromiumcodereview.appspot.com/10875079/diff/1/compiler/java/com/goog...
File compiler/java/com/google/dart/compiler/SystemLibrariesReader.java (right):

https://chromiumcodereview.appspot.com/10875079/diff/1/compiler/java/com/goog...
compiler/java/com/google/dart/compiler/SystemLibrariesReader.java:123: }
On 2012/08/28 00:25:45, danrubel wrote:
> Better would a a method that returns Map<String, DartLibrary> where
DartLibrary
> is a class that has methods such as
> * getShortName()
> * getPath()
> * getPlatforms()
> * getCategory()

Done.

https://chromiumcodereview.appspot.com/10875079/diff/1/compiler/javatests/com...
File compiler/javatests/com/google/dart/compiler/SystemLibrariesReaderTest.java
(right):

https://chromiumcodereview.appspot.com/10875079/diff/1/compiler/javatests/com...
compiler/javatests/com/google/dart/compiler/SystemLibrariesReaderTest.java:38:
continue;
On 2012/08/28 00:25:45, danrubel wrote:
> Seems like we should fail if it cannot be resolved

Done.

https://chromiumcodereview.appspot.com/10875079/diff/1/compiler/javatests/com...
compiler/javatests/com/google/dart/compiler/SystemLibrariesReaderTest.java:43: 
On 2012/08/28 00:25:45, danrubel wrote:
> Perhaps some tests to sanity check other aspects important to editor such as
> platform and category.

Done.

[danrubel, 2012-08-28 22:50:37]

lgtm