Implement Model-Driven-Views spec for Dart

tests/compiler/dart2js/analyze_api_test.dart

 // Copyright (c) 2013, 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 analyze_api;
 
 import "package:expect/expect.dart";
 import 'dart:uri';
 import 'dart:io';
 import '../../../sdk/lib/_internal/compiler/compiler.dart' as api;
 import '../../../sdk/lib/_internal/compiler/implementation/apiimpl.dart';
 import '../../../sdk/lib/_internal/compiler/implementation/dart2jslib.dart'
     hide Compiler;
 import '../../../sdk/lib/_internal/compiler/implementation/filenames.dart';
 import '../../../sdk/lib/_internal/compiler/implementation/source_file_provider.dart';
 import '../../../sdk/lib/_internal/libraries.dart';
 
 /**
  * Map of white-listed warnings and errors.
  *
  * Only add a white-listing together with a bug report to dartbug.com and add
  * the bug issue number as a comment on the white-listing.
  *
  * Use an identifiable suffix of the file uri as key. Use a fixed substring of
  * the error/warning message in the list of white-listings for each file.
  */
 // TODO(johnniwinther): Support canonical URIs as keys and message kinds as
 // values.
-const Map<String,List<String>> WHITE_LIST = const { };
+const Map<String,List<String>> WHITE_LIST = const {
+  'html_dart2js.dart': const ['Warning: Using "new Symbol"'], // Issue 10565.

[ahe, 2013/05/15 12:46:00]

This change is not cool.

By supressing this warning and using new Symbol in dart:html, you're making it
impossible for the dart2js team to implement mirrors and minification without
bloating *all* programs.

Also, why did you not consult with the dart2js team before submitting this CL?

[Jennifer Messerly, 2013/05/15 17:59:30]

This is not correct. "new Symbol" is only required for *some* of the dart:html
APIs. If you aren't using them, you shouldn't pay for it.

In any case, as I have explained elsewhere, Model-Driven-Views needs mirrors. So
we must figure out a way to make mirrors not all-or-nothing.

+};
 
 class CollectingDiagnosticHandler extends FormattingDiagnosticHandler {
   bool hasWarnings = false;
   bool hasErrors = false;
 
   Map<String,Map<String,int>> whiteListMap = new Map<String,Map<String,int>>();
 
   CollectingDiagnosticHandler(SourceFileProvider provider) : super(provider) {
     WHITE_LIST.forEach((String file, List<String> messageParts) {
       var useMap = new Map<String,int>();
@@ skipping 83 matching lines @@
       libraryRoot, libraryRoot,
       <String>['--analyze-only', '--analyze-all',
                '--categories=Client,Server']);
   compiler.librariesToAnalyzeWhenRun = uriList;
   compiler.run(null);
   Expect.isFalse(handler.hasWarnings);
   Expect.isFalse(handler.hasErrors);
   Expect.isTrue(handler.checkWhiteListUse());
   handler.reportWhiteListUse();
 }