Option support for `enableSuperMixins` (and more).

Option support for `enableSuperMixins` (and more).

* Adds language configuration support to `.analysis_options`:

  analyzer:
    language:
      enableSuperMixins: true

  (with validation).

* Extracts context configuration out of `server` context_manager and into `analyzer` where it can be re-used by `analyzer_cli`.
* Some general cleanup of option validation (with more to come!)

R=brianwilkerson@google.com

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

[pquitslund, 2015-10-29 19:56:29]

pquitslund@google.com changed reviewers:
+ brianwilkerson@google.com

[Brian Wilkerson, 2015-10-29 20:24:04]

LGTM

https://codereview.chromium.org/1423333002/diff/20001/pkg/analysis_server/pub...
File pkg/analysis_server/pubspec.yaml (right):

https://codereview.chromium.org/1423333002/diff/20001/pkg/analysis_server/pub...
pkg/analysis_server/pubspec.yaml:9: analyzer: '>=0.26.1+17 <0.27.0'
Technically, we shouldn't increase the lower bound until we've actually
published a new version.

https://codereview.chromium.org/1423333002/diff/20001/pkg/analyzer/lib/src/ge...
File pkg/analyzer/lib/src/generated/utilities_general.dart (right):

https://codereview.chromium.org/1423333002/diff/20001/pkg/analyzer/lib/src/ge...
pkg/analyzer/lib/src/generated/utilities_general.dart:13: bool isTrue(dynamic
value) =>
"dynamic" --> "Object"

https://codereview.chromium.org/1423333002/diff/20001/pkg/analyzer/lib/src/ta...
File pkg/analyzer/lib/src/task/options.dart (right):

https://codereview.chromium.org/1423333002/diff/20001/pkg/analyzer/lib/src/ta...
pkg/analyzer/lib/src/task/options.dart:38: static const String
enable_super_mixins = 'enableSuperMixins';
Shouldn't we be using camel case for variable names?

[pquitslund, 2015-10-29 21:06:19]

Thanks!

https://codereview.chromium.org/1423333002/diff/20001/pkg/analysis_server/pub...
File pkg/analysis_server/pubspec.yaml (right):

https://codereview.chromium.org/1423333002/diff/20001/pkg/analysis_server/pub...
pkg/analysis_server/pubspec.yaml:9: analyzer: '>=0.26.1+17 <0.27.0'
On 2015/10/29 20:24:03, Brian Wilkerson wrote:
> Technically, we shouldn't increase the lower bound until we've actually
> published a new version.

Right.  I'll publish at the same time as I land this CL. :)  (nearly atomic!)

https://codereview.chromium.org/1423333002/diff/20001/pkg/analyzer/lib/src/ge...
File pkg/analyzer/lib/src/generated/utilities_general.dart (right):

https://codereview.chromium.org/1423333002/diff/20001/pkg/analyzer/lib/src/ge...
pkg/analyzer/lib/src/generated/utilities_general.dart:13: bool isTrue(dynamic
value) =>
On 2015/10/29 20:24:03, Brian Wilkerson wrote:
> "dynamic" --> "Object"

Done.

https://codereview.chromium.org/1423333002/diff/20001/pkg/analyzer/lib/src/ta...
File pkg/analyzer/lib/src/task/options.dart (right):

https://codereview.chromium.org/1423333002/diff/20001/pkg/analyzer/lib/src/ta...
pkg/analyzer/lib/src/task/options.dart:38: static const String
enable_super_mixins = 'enableSuperMixins';
On 2015/10/29 20:24:03, Brian Wilkerson wrote:
> Shouldn't we be using camel case for variable names?

Yes!  Fixed.

[pquitslund, 2015-10-29 21:11:30]

Committed patchset #4 (id:60001) manually as
af7017e585f87ddc8dd0c149dcfbc586729eeb85 (presubmit successful).