Observatory: handle the case where package:foo imports are canonicalized as packages/foo.

Observatory: handle the case where package:foo imports are canonicalized as packages/foo.

BUG=http://dartbug.com/24925
R=floitsch@google.com

Committed: https://github.com/dart-lang/sdk/commit/f1391c7841f350bea81200c8f5a66790201b4af9

[rmacnak, 2015-11-17 01:39:50]

rmacnak@google.com changed reviewers:
+ floitsch@google.com

[floitsch, 2015-11-17 01:42:31]

LGTM.

https://codereview.chromium.org/1447383002/diff/1/runtime/observatory/lib/src...
File runtime/observatory/lib/src/elements/script_inset.dart (right):

https://codereview.chromium.org/1447383002/diff/1/runtime/observatory/lib/src...
runtime/observatory/lib/src/elements/script_inset.dart:675: if (targetUri.scheme
== 'package') {
I guess this will get worse when we have package-maps.

[rmacnak, 2015-11-17 01:53:07]

https://codereview.chromium.org/1447383002/diff/1/runtime/observatory/lib/src...
File runtime/observatory/lib/src/elements/script_inset.dart (right):

https://codereview.chromium.org/1447383002/diff/1/runtime/observatory/lib/src...
runtime/observatory/lib/src/elements/script_inset.dart:675: if (targetUri.scheme
== 'package') {
On 2015/11/17 01:42:31, floitsch wrote:
> I guess this will get worse when we have package-maps.

Yeah. Maybe the flutter embedder could canonicalize package: as package: and
only do the resolution to a file in a loading request like the standalone
embedder, but I don't know if there's some other reason to keep it the current
way.

[rmacnak, 2015-11-17 01:55:46]

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