Cleanup inconsistent dart:html method signatures

sdk/lib/html/dartium/html_dartium.dart

Index: sdk/lib/html/dartium/html_dartium.dart
diff --git a/sdk/lib/html/dartium/html_dartium.dart b/sdk/lib/html/dartium/html_dartium.dart
index 173e527236904996c2cbb082a0a9cc3e15cb0ed2..4c9a376068fe114a99333414eceb03732c6e6259 100644
--- a/sdk/lib/html/dartium/html_dartium.dart
+++ b/sdk/lib/html/dartium/html_dartium.dart
@@ -11089,7 +11089,12 @@ abstract class ElementList<T extends Element> extends ListBase<T> {
 
 }
 
-// Wrapper over an immutable NodeList to make it implement ElementList<Element>.
+// Wrapper over an immutable NodeList to make it implement ElementList.
+//
+// Clients are {`Document`, `DocumentFragment`}.`querySelectorAll` which are
+// declared to return `ElementList`.  This provides all the static analysis
+// benefit so there is no need for this class have a constrained type parameter.
+//
 class _FrozenElementList extends ListBase
     implements ElementList, NodeListWrapper {
   final List<Node> _nodeList;
@@ -25179,7 +25184,7 @@ abstract class ParentNode extends NativeFieldWrapperClass2 {
   @DomName('ParentNode.querySelectorAll')
   @DocsEditable()
   @Experimental() // untriaged
-  List<Node> querySelectorAll(String selectors);
+  List<Node> _querySelectorAll(String selectors);
 
 }
 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
@@ -31419,10 +31424,10 @@ class Url extends NativeFieldWrapperClass2 implements UrlUtils {
     if ((blob_OR_source_OR_stream is Blob || blob_OR_source_OR_stream == null)) {
       return _blink.BlinkURL.instance.createObjectURL_Callback_1_(blob_OR_source_OR_stream);
     }
-    if ((blob_OR_source_OR_stream is MediaSource)) {

[Alan Knight, 2015/01/20 22:53:01]

I think this is fine, because it seems like it does the same thing in either
case, but why reverse the order?

In fact, the whole method seems strange. It has three separate tests, all of
which do the same thing. I hope it's just generated code and we don't care.

[Jacob, 2015/01/20 23:00:20]

yep. generated code, the generator is unstable, and we don't care.

+    if ((blob_OR_source_OR_stream is MediaStream)) {
       return _blink.BlinkURL.instance.createObjectURL_Callback_1_(blob_OR_source_OR_stream);
     }
-    if ((blob_OR_source_OR_stream is MediaStream)) {
+    if ((blob_OR_source_OR_stream is MediaSource)) {
       return _blink.BlinkURL.instance.createObjectURL_Callback_1_(blob_OR_source_OR_stream);
     }
     throw new ArgumentError("Incorrect number or type of arguments");