AS strong-mode .analysis_options awareness.

pkg/analysis_server/lib/src/context_manager.dart

Index: pkg/analysis_server/lib/src/context_manager.dart
diff --git a/pkg/analysis_server/lib/src/context_manager.dart b/pkg/analysis_server/lib/src/context_manager.dart
index 05a3fa108e7565c80e5091b328b664a66a1d35b2..2720aa79eec3be7aca8355063f87df6c146ab9b2 100644
--- a/pkg/analysis_server/lib/src/context_manager.dart
+++ b/pkg/analysis_server/lib/src/context_manager.dart
@@ -484,6 +484,16 @@ class ContextManagerImpl implements ContextManager {
       return;
     }
 
+    // Set strong mode.
+    var strongMode = analyzer['strong-mode'];
+    if (strongMode != null && strongMode == true) {

[Brian Wilkerson, 2015/10/13 20:12:02]

The expression "strongMode != null" isn't necessary.

+      AnalysisContext context = info.context;
+      AnalysisOptionsImpl options =
+          new AnalysisOptionsImpl.from(context.analysisOptions);
+      options.strongMode = true;

[Jennifer Messerly, 2015/10/13 20:00:28]

we had an issue at some point where we also needed to set the SDK context:
https://github.com/dart-lang/dev_compiler/blob/6fe74a269f7877b6aae3d03bfd9be9...

Not sure if that's still the case or not... Maybe Leaf/Brian remember what the
issue was?

[pquitslund, 2015/10/13 20:10:58]

Oh!  Interesting.  Leaf/Brian?

[Brian Wilkerson, 2015/10/13 20:12:02]

I don't remember. But eventually, to support this with the semantics we talked
about yesterday, we'll probably need two SDK's: one with strong mode enabled and
one with strong mode disabled. We should be able to create them lazily.

[Leaf, 2015/10/13 20:20:38]

Will we be analyzing the SDK in strong mode with this CL, or only user code?  It
shouldn't make much difference now, but once we start adding generic method
annotations that might change.

[Jennifer Messerly, 2015/10/13 20:20:54]

yeah. I guess this isn't a big issue yet, until we try using generic methods in
the SDK. At that point, we'd also likely want to be able to point at an
experimental SDK sources to analyze.

I guess we can try it out and see how it's working now, and cross that other
bridge when we come to it.

+      context.analysisOptions = options;
+    }
+
     // Set ignore patterns.
     YamlList exclude = analyzer['exclude'];
     if (exclude != null) {