Return an absolute URI for Platform.script.

Return an absolute URI for Platform.script.

BUG=
R=ajohnsen@google.com, sgjesse@google.com

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

Reverted: https://code.google.com/p/dart/source/detail?r=29742

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

[Anders Johnsen, 2013-11-01 10:13:50]

lgtm

https://codereview.chromium.org/46163009/diff/1/sdk/lib/io/platform.dart
File sdk/lib/io/platform.dart (right):

https://codereview.chromium.org/46163009/diff/1/sdk/lib/io/platform.dart#newc...
sdk/lib/io/platform.dart:84: * isolate. If the script argument on the command
line is relative,
Newline*2 before `If`.

[Anders Johnsen, 2013-11-01 10:14:44]

One more comment, can you add a test to check it's absolute, when being spawned
with a relative Uri? I think this can go in, in one of the process_*_test files.

[Søren Gjesse, 2013-11-01 10:17:28]

lgtm

https://codereview.chromium.org/46163009/diff/1/sdk/lib/io/platform.dart
File sdk/lib/io/platform.dart (right):

https://codereview.chromium.org/46163009/diff/1/sdk/lib/io/platform.dart#newc...
sdk/lib/io/platform.dart:88: * This implies that a path including "link/.." in
the script name
Instead of trying to examplify here maybe just say something like URI resolving
is pure string operation and might not be the same as file system's path
resolution.

[Bill Hesse, 2013-11-01 10:37:42]

Committed patchset #1 manually as r29740.

[Bill Hesse, 2013-11-01 12:49:45]

Comments addressed, test added, apidoc fixed on Windows.

https://codereview.chromium.org/46163009/diff/1/sdk/lib/io/platform.dart
File sdk/lib/io/platform.dart (right):

https://codereview.chromium.org/46163009/diff/1/sdk/lib/io/platform.dart#newc...
sdk/lib/io/platform.dart:84: * isolate. If the script argument on the command
line is relative,
On 2013/11/01 10:13:51, Anders Johnsen wrote:
> Newline*2 before `If`.

Done.

https://codereview.chromium.org/46163009/diff/1/sdk/lib/io/platform.dart#newc...
sdk/lib/io/platform.dart:88: * This implies that a path including "link/.." in
the script name
On 2013/11/01 10:17:28, Søren Gjesse wrote:
> Instead of trying to examplify here maybe just say something like URI
resolving
> is pure string operation and might not be the same as file system's path
> resolution.

Done.  I think the specific case is worth including, because it is the most
important one.

[Anders Johnsen, 2013-11-01 12:51:42]

lgtm

[Søren Gjesse, 2013-11-01 12:53:12]

lgtm

[Bill Hesse, 2013-11-04 07:26:05]

Committed patchset #2 manually as r29792 (presubmit successful).