Simplifying entrypoint.dart, and changed IO load.

Simplifying entrypoint.dart, and changed IO load.

Now *.load(PathRep) methods will return null on
file not existing instead of throwing an error.

R=tapted@chromium.org

Committed: https://github.com/tapted/bleeding_edge/commit/b526326

[pajamallama, 2014-01-10 00:48:06]

Fixed some bugs in this code (arising from the exception => return null change).

[pajamallama, 2014-01-10 01:03:38]

Added another (very tiny) patch.. Soz!

[tapted, 2014-01-10 02:20:55]

lgtm with a nit

https://codereview.appspot.com/50280043/diff/60001/dart/sdk/lib/_internal/pub...
File dart/sdk/lib/_internal/pub/lib/src/entrypoint.dart (right):

https://codereview.appspot.com/50280043/diff/60001/dart/sdk/lib/_internal/pub...
dart/sdk/lib/_internal/pub/lib/src/entrypoint.dart:86: log.fine("Deleting
package directory for ${id.name} before get.");
This statement isn't really right any more. Maybe ~"Ensuring package directory
for .. is non-existent before get."

[pajamallama, 2014-01-12 22:53:13]

https://codereview.appspot.com/50280043/diff/60001/dart/sdk/lib/_internal/pub...
File dart/sdk/lib/_internal/pub/lib/src/entrypoint.dart (right):

https://codereview.appspot.com/50280043/diff/60001/dart/sdk/lib/_internal/pub...
dart/sdk/lib/_internal/pub/lib/src/entrypoint.dart:86: log.fine("Deleting
package directory for ${id.name} before get.");
On 2014/01/10 02:20:56, tapted wrote:
> This statement isn't really right any more. Maybe ~"Ensuring package directory
> for .. is non-existent before get."

Done.

[pajamallama, 2014-01-13 00:24:04]

Committed patchset #5 manually as rb526326 (presubmit successful).