Breaking changes in 'analyzer' package.

pkg/code_transformers/lib/src/dart_sdk.dart

Index: pkg/code_transformers/lib/src/dart_sdk.dart
diff --git a/pkg/code_transformers/lib/src/dart_sdk.dart b/pkg/code_transformers/lib/src/dart_sdk.dart
index 3438218d514ec87f7fc3656f941ace7b4899b71d..e78057ff5da5c84b16af51fea134f4bfb4788b26 100644
--- a/pkg/code_transformers/lib/src/dart_sdk.dart
+++ b/pkg/code_transformers/lib/src/dart_sdk.dart
@@ -108,10 +108,10 @@ class DartSourceProxy implements UriAnnotatedSource {
     return new DartSourceProxy(source, uri);
   }
 
-  Source resolveRelative(Uri relativeUri) {
+  Uri resolveRelative(Uri relativeUri) {
     // Assume that the type can be accessed via this URI, since these
     // should only be parts for dart core files.
-    return wrap(_proxy.resolveRelative(relativeUri), uri);
+    return _proxy.uri.resolveUri(relativeUri);

[blois, 2014/08/05 21:33:26]

The DartSourceProxy wraps 'dart:' sources, and I'm seeing that _proxy.uri is
'dart:core', which is incorrectly resolving 'annotation.dart' to be:
'dart:annotation.dart'.

I think that the 'uri' property is incredibly misleading, in this case (and in
others).

The problem is that we have URIs which can be resolved relative to something,
but not absolutely. This is one example, but another one is files in packages
not in the lib/ folder. These are only accessible via relative URIs by other
files within that package.

Because Source.uri is not something that can be used as the import URI, I do not
think it should use the 'uri' name.

[Brian Wilkerson, 2014/08/05 21:44:21]

That's a bug. It should be resolved to dart:core/annotation.dart. There is a
test for this, at least on the Java side.
 
The uri field should always be the absolute URI resulting from resolving the URI
in the directive against the absolute URI of the file containing the directive.
It won't always be the same as the original (possibly relative) URI in the
directive, but it should be a valid absolute URI that references the same file
(and hence could replace the import URI).

   }
 
   bool exists() => _proxy.exists();
@@ -181,6 +181,11 @@ class MockDartSdk implements DartSdk {
     }
     return src;
   }
+
+  @override
+  Source fromFileUri(Uri uri) {
+    throw new UnsupportedError('MockDartSdk.fromFileUri');
+  }
 }
 
 class _MockSdkSource implements UriAnnotatedSource {
@@ -209,6 +214,6 @@ class _MockSdkSource implements UriAnnotatedSource {
 
   bool get isInSystemLibrary => true;
 
-  Source resolveRelative(Uri relativeUri) =>
+  Uri resolveRelative(Uri relativeUri) =>
       throw new UnsupportedError('not expecting relative urls in dart: mocks');
 }