Issue 23133. Fix for restoring 'package:' uris on Windows.

Issue 23133. Fix for restoring 'package:' uris on Windows.

R=paulberry@google.com
BUG= https://code.google.com/p/dart/issues/detail?id=23133

Committed: https://code.google.com/p/dart/source/detail?r=45023

[Paul Berry, 2015-04-09 18:16:39]

Fix lgtm.  Can you please add a unit test to make sure this case doesn't get
broken in the future?  So many of us develop on macs that I worry if we don't
have a unit test, it would take us a long time to notice a regression.

[scheglov, 2015-04-09 18:21:52]

On 2015/04/09 18:16:39, Paul Berry wrote:
> Fix lgtm.  Can you please add a unit test to make sure this case doesn't get
> broken in the future?  So many of us develop on macs that I worry if we don't
> have a unit test, it would take us a long time to notice a regression.

I wish there were an easy way to test it.
We create package folders in a memory FS, which is of the Posix style.

Maybe in a future CL.

[scheglov, 2015-04-09 18:23:48]

Committed patchset #1 (id:1) manually as 45023 (presubmit successful).

[Siggi Cherem (dart-lang), 2015-04-09 23:55:44]

sigmund@google.com changed reviewers:
+ sigmund@google.com

[Siggi Cherem (dart-lang), 2015-04-09 23:55:45]

A couple tests in the bots started failing around the time this CL was
submitted. See:

http://chromegw.corp.google.com/i/client.dart/builders/pub-win-debug-be/build...

and:
http://chromegw.corp.google.com/i/client.dart/builders/pub-win-be/builds/9611...

The failures sound like it is related, but I'm not sure if this is the change
that caused that or some other change.

[scheglov, 2015-04-10 04:19:23]

On 2015/04/09 23:55:45, Siggi Cherem (dart-lang) wrote:
> A couple tests in the bots started failing around the time this CL was
> submitted. See:
> 
>
http://chromegw.corp.google.com/i/client.dart/builders/pub-win-debug-be/build...
> 
> and:
>
http://chromegw.corp.google.com/i/client.dart/builders/pub-win-be/builds/9611...
> 
> The failures sound like it is related, but I'm not sure if this is the change
> that caused that or some other change.

Yes, you are right, there is a failing test and it related to this CL.
I will fix it tomorrow.