Reland c9e1b88 and 56726fc with a DDC fix.

pkg/dev_compiler/lib/src/compiler/compiler.dart

 // Copyright (c) 2016, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 import 'dart:collection' show HashSet, Queue;
 import 'dart:convert' show BASE64, JSON, UTF8;
 import 'dart:io' show File;
-import 'package:analyzer/dart/element/element.dart' show LibraryElement;
+
 import 'package:analyzer/analyzer.dart'
     show AnalysisError, CompilationUnit, ErrorSeverity;
+import 'package:analyzer/dart/element/element.dart' show LibraryElement;
 import 'package:analyzer/file_system/file_system.dart' show ResourceProvider;
 import 'package:analyzer/file_system/physical_file_system.dart'
     show PhysicalResourceProvider;
 import 'package:analyzer/src/context/builder.dart' show ContextBuilder;
 import 'package:analyzer/src/context/context.dart' show AnalysisContextImpl;
+import 'package:analyzer/src/error/codes.dart' show StaticTypeWarningCode;
 import 'package:analyzer/src/generated/engine.dart'
     show AnalysisContext, AnalysisEngine;
 import 'package:analyzer/src/generated/sdk.dart' show DartSdkManager;
 import 'package:analyzer/src/generated/source.dart'
     show ContentCache, DartUriResolver;
 import 'package:analyzer/src/generated/source_io.dart'
     show Source, SourceKind, UriResolver;
 import 'package:analyzer/src/summary/package_bundle_reader.dart'
     show InSummarySource, InputPackagesResultProvider, SummaryDataStore;
-import 'package:analyzer/src/error/codes.dart' show StaticTypeWarningCode;
 import 'package:args/args.dart' show ArgParser, ArgResults;
 import 'package:args/src/usage_exception.dart' show UsageException;
 import 'package:func/func.dart' show Func1;
 import 'package:path/path.dart' as path;
 import 'package:source_maps/source_maps.dart';
 
 import '../analyzer/context.dart' show AnalyzerOptions, createSourceFactory;
 import '../js_ast/js_ast.dart' as JS;
 import 'code_generator.dart' show CodeGenerator;
 import 'error_helpers.dart' show errorSeverity, formatError, sortErrors;
@@ skipping 43 matching lines @@
         options: options.contextBuilderOptions);
 
     var analysisOptions = contextBuilder.getAnalysisOptions(analysisRoot);
     var sdk = contextBuilder.findSdk(null, analysisOptions);
 
     var sdkResolver = new DartUriResolver(sdk);
 
     // Read the summaries.
     var summaryData =
         new SummaryDataStore(options.summaryPaths, recordDependencyInfo: true);
+    var sdkSummaryBundle = sdk.getLinkedBundle();

[vsm, 2017/01/24 16:41:41]

This logic really needs to be folded in with the redundant bits in analyzer_cli
driver.  Is this not possible?

[scheglov, 2017/01/24 17:04:33]

It is possible.
But it does not seem very useful.

[vsm, 2017/01/24 17:10:35]

The current state means bugs like this need to be fixed in multiple places.

In general, from a pure abstraction perspective, almost none of the code in this
function belongs in dev_compiler.

[scheglov, 2017/01/24 17:13:07]

Exactly.
This code duplication should be solved for the whole function, not only for
these two tweaks.
But that's outside of the scope of this CL.

+    if (sdkSummaryBundle != null) {
+      summaryData.addBundle(null, sdkSummaryBundle);
+    }
 
     var srcFactory = createSourceFactory(options,
         sdkResolver: sdkResolver,
         fileResolvers: fileResolvers,
         summaryData: summaryData,
         resourceProvider: resourceProvider);
 
     var context =
         AnalysisEngine.instance.createAnalysisContext() as AnalysisContextImpl;
     context.analysisOptions = analysisOptions;
     context.sourceFactory = srcFactory;
-    context.typeProvider = sdkResolver.dartSdk.context.typeProvider;
-    context.resultProvider =
-        new InputPackagesResultProvider(context, summaryData);
+    if (sdkSummaryBundle != null) {
+      context.resultProvider =
+          new InputPackagesResultProvider(context, summaryData);
+    }
     options.declaredVariables.forEach(context.declaredVariables.define);
     context.declaredVariables.define('dart.isVM', 'false');
 
     // TODO(vsm): Should this be hardcoded?
     context.declaredVariables.define('dart.library.html', 'true');
     context.declaredVariables.define('dart.library.io', 'false');
 
     if (!context.analysisOptions.strongMode) {
       throw new ArgumentError('AnalysisContext must be strong mode');
     }
@@ skipping 458 matching lines @@
     // Fall back to a relative path.
     return path.toUri(path.relative(path.fromUri(uri), from: dir)).toString();
   }
 
   for (int i = 0; i < list.length; i++) {
     list[i] = transformUri(list[i]);
   }
   map['file'] = transformUri(map['file']);
   return map;
 }