Remove the ability in dart2js to turn off syntax-only generic methods.

pkg/compiler/lib/src/options.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.
 
 library dart2js.src.options;
 
 import '../compiler.dart' show PackagesDiscoveryProvider;
 import 'commandline_options.dart' show Flags;
 
-/// Options used for parsing.
-///
-/// Use this to conditionally support certain constructs, e.g.,
-/// experimental ones.
-abstract class ParserOptions {
-  const ParserOptions();
-
-  /// Support parsing of generic method declarations, and invocations of
-  /// methods where type arguments are passed.
-  bool get enableGenericMethodSyntax;
-}
-
 /// Options used for controlling diagnostic messages.
 abstract class DiagnosticOptions {
   const DiagnosticOptions();
 
   /// If `true`, warnings cause the compilation to fail.
   bool get fatalWarnings;
 
   /// Emit terse diagnostics without howToFix.
   bool get terseDiagnostics;
 
   /// If `true`, warnings are not reported.
   bool get suppressWarnings;
 
   /// If `true`, hints are not reported.
   bool get suppressHints;
 
   /// Returns `true` if warnings and hints are shown for all packages.
   bool get showAllPackageWarnings;
 
   /// Returns `true` if warnings and hints are hidden for all packages.
   bool get hidePackageWarnings;
 
   /// Returns `true` if warnings should be should for [uri].
   bool showPackageWarningsFor(Uri uri);
 }
 
 /// Object for passing options to the compiler. Superclasses are used to select
 /// subsets of these options, enabling each part of the compiler to depend on
 /// as few as possible.
-class CompilerOptions implements DiagnosticOptions, ParserOptions {
+class CompilerOptions implements DiagnosticOptions {
   /// The entry point of the application that is being compiled.
   final Uri entryPoint;
 
   /// Root location where SDK libraries are found.
   final Uri libraryRoot;
 
   /// Package root location.
   ///
   /// If not null then [packageConfig] should be null.
   final Uri packageRoot;
@@ skipping 76 matching lines @@
   /// Whether to emit a .json file with a summary of the information used by the
   /// compiler during optimization. This includes resolution details,
   /// dependencies between elements, results of type inference, and the output
   /// code for each function.
   final bool dumpInfo;
 
   /// Whether we allow passing an extra argument to `assert`, containing a
   /// reason for why an assertion fails. (experimental)
   final bool enableAssertMessage;
 
-  /// Support parsing of generic method declarations, and invocations of
-  /// methods where type arguments are passed.
-  final bool enableGenericMethodSyntax;
-
   /// Support access to initializing formal constructor arguments, e.g., the
   /// use of `x` to initialize `y` in `C(this.x) : y = x`.
   final bool enableInitializingFormalAccess;
 
   /// Whether the user specified a flag to allow the use of dart:mirrors. This
   /// silences a warning produced by the compiler.
   final bool enableExperimentalMirrors;
 
   /// Whether to enable minification
   // TODO(sigmund): rename to minify
@@ skipping 119 matching lines @@
         fatalWarnings: _hasOption(options, Flags.fatalWarnings),
         terseDiagnostics: _hasOption(options, Flags.terse),
         suppressWarnings: _hasOption(options, Flags.suppressWarnings),
         suppressHints: _hasOption(options, Flags.suppressHints),
         shownPackageWarnings:
             _extractOptionalCsvOption(options, Flags.showPackageWarnings),
         disableInlining: _hasOption(options, Flags.disableInlining),
         disableTypeInference: _hasOption(options, Flags.disableTypeInference),
         dumpInfo: _hasOption(options, Flags.dumpInfo),
         enableAssertMessage: _hasOption(options, Flags.enableAssertMessage),
-        enableGenericMethodSyntax:
-            true, // No check for `Flags.genericMethodSyntax`: always enabled.
         enableInitializingFormalAccess:
             _hasOption(options, Flags.initializingFormalAccess),
         enableExperimentalMirrors:
             _hasOption(options, Flags.enableExperimentalMirrors),
         enableMinification: _hasOption(options, Flags.minify),
         enableNativeLiveTypeAnalysis:
             !_hasOption(options, Flags.disableNativeLiveTypeAnalysis),
         enableTypeAssertions: _hasOption(options, Flags.enableCheckedMode),
         enableUserAssertions: _hasOption(options, Flags.enableCheckedMode),
         generateCodeWithCompileTimeErrors:
@@ skipping 48 matching lines @@
       Uri deferredMapUri: null,
       bool fatalWarnings: false,
       bool terseDiagnostics: false,
       bool suppressWarnings: false,
       bool suppressHints: false,
       List<String> shownPackageWarnings: null,
       bool disableInlining: false,
       bool disableTypeInference: false,
       bool dumpInfo: false,
       bool enableAssertMessage: false,
-      bool enableGenericMethodSyntax: false,
       bool enableInitializingFormalAccess: false,
       bool enableExperimentalMirrors: false,
       bool enableMinification: false,
       bool enableNativeLiveTypeAnalysis: true,
       bool enableTypeAssertions: false,
       bool enableUserAssertions: false,
       bool generateCodeWithCompileTimeErrors: false,
       bool generateSourceMap: true,
       bool hasIncrementalSupport: false,
       Uri outputUri: null,
@@ skipping 51 matching lines @@
         deferredMapUri: deferredMapUri,
         fatalWarnings: fatalWarnings,
         terseDiagnostics: terseDiagnostics,
         suppressWarnings: suppressWarnings,
         suppressHints: suppressHints,
         shownPackageWarnings: shownPackageWarnings,
         disableInlining: disableInlining || hasIncrementalSupport,
         disableTypeInference: disableTypeInference,
         dumpInfo: dumpInfo,
         enableAssertMessage: enableAssertMessage,
-        enableGenericMethodSyntax: enableGenericMethodSyntax,
         enableInitializingFormalAccess: enableInitializingFormalAccess,
         enableExperimentalMirrors: enableExperimentalMirrors,
         enableMinification: enableMinification,
         enableNativeLiveTypeAnalysis: enableNativeLiveTypeAnalysis,
         enableTypeAssertions: enableTypeAssertions,
         enableUserAssertions: enableUserAssertions,
         generateCodeWithCompileTimeErrors: generateCodeWithCompileTimeErrors,
         generateSourceMap: generateSourceMap,
         hasIncrementalSupport: hasIncrementalSupport,
         outputUri: outputUri,
@@ skipping 30 matching lines @@
       this.deferredMapUri: null,
       this.fatalWarnings: false,
       this.terseDiagnostics: false,
       this.suppressWarnings: false,
       this.suppressHints: false,
       List<String> shownPackageWarnings: null,
       this.disableInlining: false,
       this.disableTypeInference: false,
       this.dumpInfo: false,
       this.enableAssertMessage: false,
-      this.enableGenericMethodSyntax: false,
       this.enableInitializingFormalAccess: false,
       this.enableExperimentalMirrors: false,
       this.enableMinification: false,
       this.enableNativeLiveTypeAnalysis: false,
       this.enableTypeAssertions: false,
       this.enableUserAssertions: false,
       this.generateCodeWithCompileTimeErrors: false,
       this.generateSourceMap: true,
       this.hasIncrementalSupport: false,
       this.outputUri: null,
@@ skipping 37 matching lines @@
       deferredMapUri,
       fatalWarnings,
       terseDiagnostics,
       suppressWarnings,
       suppressHints,
       List<String> shownPackageWarnings,
       disableInlining,
       disableTypeInference,
       dumpInfo,
       enableAssertMessage,
-      enableGenericMethodSyntax,
       enableInitializingFormalAccess,
       enableExperimentalMirrors,
       enableMinification,
       enableNativeLiveTypeAnalysis,
       enableTypeAssertions,
       enableUserAssertions,
       generateCodeWithCompileTimeErrors,
       generateSourceMap,
       hasIncrementalSupport,
       outputUri,
@@ skipping 38 matching lines @@
         terseDiagnostics: terseDiagnostics ?? options.terseDiagnostics,
         suppressWarnings: suppressWarnings ?? options.suppressWarnings,
         suppressHints: suppressHints ?? options.suppressHints,
         shownPackageWarnings:
             shownPackageWarnings ?? options._shownPackageWarnings,
         disableInlining: disableInlining ?? options.disableInlining,
         disableTypeInference:
             disableTypeInference ?? options.disableTypeInference,
         dumpInfo: dumpInfo ?? options.dumpInfo,
         enableAssertMessage: enableAssertMessage ?? options.enableAssertMessage,
-        enableGenericMethodSyntax:
-            enableGenericMethodSyntax ?? options.enableGenericMethodSyntax,
         enableInitializingFormalAccess: enableInitializingFormalAccess ??
             options.enableInitializingFormalAccess,
         enableExperimentalMirrors:
             enableExperimentalMirrors ?? options.enableExperimentalMirrors,
         enableMinification: enableMinification ?? options.enableMinification,
         enableNativeLiveTypeAnalysis: enableNativeLiveTypeAnalysis ??
             options.enableNativeLiveTypeAnalysis,
         enableTypeAssertions:
             enableTypeAssertions ?? options.enableTypeAssertions,
         enableUserAssertions:
@@ skipping 122 matching lines @@
 }
 
 /// Locations of the platform descriptor files relative to the library root.
 const String _clientPlatform = "lib/dart_client.platform";
 const String _serverPlatform = "lib/dart_server.platform";
 const String _sharedPlatform = "lib/dart_shared.platform";
 
 const String _UNDETERMINED_BUILD_ID = "build number could not be determined";
 const bool _forceIncrementalSupport =
     const bool.fromEnvironment('DART2JS_EXPERIMENTAL_INCREMENTAL_SUPPORT');