Fixing a bunch of dartc errors in dart:html tests.

tests/html/htmlelement_test.dart

 library ElementTest;
 import '../../pkg/unittest/lib/unittest.dart';
 import '../../pkg/unittest/lib/html_config.dart';
 import 'dart:html';
 
 main() {
   useHtmlConfiguration();
   test('InnerHTML', () {
     Element element = new Element.tag('div');
     element.id = 'test';
@@ skipping 43 matching lines @@
     hasBeenInvoked = false;
     expect(div.dataAttributes.putIfAbsent('bar', f), 'bar-value');
     expect(hasBeenInvoked, isTrue);
 
     hasBeenInvoked = false;
     expect(div.dataAttributes.putIfAbsent('bar', f), 'bar-value');
     expect(hasBeenInvoked, isFalse);
 
     final keys = <String> [];
     final values = <String> [];
-    div.dataAttributes.forEach(void f(String key, String value) {

[Emily Fortuna, 2012/12/21 00:44:18]

I thought this was supported? no longer?

[blois, 2012/12/21 01:05:42]

It's supported, just needed to remove the return value and function name.

[Emily Fortuna, 2012/12/21 01:07:31]

Oh okay. I thought for some reason that the vm and dart2js supported actually 
having the return value and function name, just not dart2js.

[blois, 2012/12/21 01:13:17]

Dartc was saying it's deprecated, but I don't know other than that. 

Removing them allows it to pass, so seems like a good thing :)

+    div.dataAttributes.forEach((String key, String value) {
         keys.add(key);
         values.add(value);
     });
     expect(keys, unorderedEquals(['foo', 'bar']));
     expect(values, unorderedEquals(['foo-value', 'bar-value']));
 
     expect(new List<String>.from(div.dataAttributes.keys),
         unorderedEquals(['foo', 'bar']));
     expect(new List<String>.from(div.dataAttributes.values),
         unorderedEquals(['foo-value', 'bar-value']));
 
     expect(div.dataAttributes.length, 2);
     expect(div.dataAttributes.isEmpty, isFalse);
 
     expect(div.dataAttributes.remove('qux'), isNull);
     expect(div.dataAttributes.length, 2);
     expect(div.dataAttributes.isEmpty, isFalse);
 
     expect(div.dataAttributes.remove('foo'), 'foo-value');
     expect(div.dataAttributes.length, 1);
     expect(div.dataAttributes.isEmpty, isFalse);
 
     div.dataAttributes.clear();
     expect(div.dataAttributes.length, 0);
     expect(div.dataAttributes.isEmpty, isTrue);
   });
 }