update elements and fix some bugs

test/iron_selector_exclude_local_names_test.dart

Index: test/iron_selector_exclude_local_names_test.dart
diff --git a/test/iron_selector_exclude_local_names_test.dart b/test/iron_selector_exclude_local_names_test.dart
index 3a1541671aa0476efb267051e97477826d897593..a433df11efe1c9f0a5176a302251354ec9620ffa 100644
--- a/test/iron_selector_exclude_local_names_test.dart
+++ b/test/iron_selector_exclude_local_names_test.dart
@@ -22,19 +22,19 @@ main() async {
     });
 
     test('default `excludedLocalNames`', () {
-      expect(test1.excludedLocalNames['template'], isNotNull);
-      expect(test2.excludedLocalNames['template'], isNotNull);
+      expect(test1.jsElement['_excludedLocalNames']['template'], isNotNull);

[Siggi Cherem (dart-lang), 2015/10/28 22:35:16]

is this just private internal state or something users were expected to
configure externally?

[jakemac, 2015/10/29 16:07:29]

This got moved to be a private variable. I guess technically that's a breaking
change, but I doubt this was being used and since we are in RC still I think its
fine not to bump the version.

+      expect(test2.jsElement['_excludedLocalNames']['template'], isNotNull);
     });
 
     test('custom `excludedLocalNames`', () {
-      test1.excludedLocalNames['foo'] = 1;
-      expect(test1.excludedLocalNames['foo'], isNotNull);
-      expect(test2.excludedLocalNames['foo'], isNull);
+      test1.jsElement['_excludedLocalNames']['foo'] = 1;
+      expect(test1.jsElement['_excludedLocalNames']['foo'], isNotNull);
+      expect(test2.jsElement['_excludedLocalNames']['foo'], isNull);
     });
 
     test('items', () {
-      test1.excludedLocalNames['span'] = 1;
-      test2.excludedLocalNames['div'] = 1;
+      test1.jsElement['_excludedLocalNames']['span'] = 1;
+      test2.jsElement['_excludedLocalNames']['div'] = 1;
 
       var NOT_FOUND = -1;
       var items1 = test1.items.map((el) => el.localName);