Future-ify all the html methods and add named parameters

Future-ify All the methods!

(with a few exceptions we've planned). 
Also added some tests to test these apis.

BUG=

Committed: https://code.google.com/p/dart/source/detail?r=19586

[Emily Fortuna, 2013-03-06 00:04:44]

This removes callbacks from all spots with the exceptions of RAF and
setImmediate that we've discussed. 

I also added more auto-named parameter generation.

[blois, 2013-03-06 00:34:29]

https://codereview.chromium.org/12380046/diff/27001/sdk/lib/html/dart2js/html...
File sdk/lib/html/dart2js/html_dart2js.dart (right):

https://codereview.chromium.org/12380046/diff/27001/sdk/lib/html/dart2js/html...
sdk/lib/html/dart2js/html_dart2js.dart:26961: +  Future openDatabase(String
name, String version, String displayName, int estimatedSize) {
I don't think this should be a Future. Does websql_test pass?

https://codereview.chromium.org/12380046/diff/27001/sdk/lib/html/dart2js/html...
sdk/lib/html/dart2js/html_dart2js.dart:26974: +  SqlDatabaseSync
_openDatabaseSync(String name, String version, String displayName, int
estimatedSize, [DatabaseCallback creationCallback]) native;
This should not be a Future.

https://codereview.chromium.org/12380046/diff/27001/tests/html/fileapi_test.dart
File tests/html/fileapi_test.dart (right):

https://codereview.chromium.org/12380046/diff/27001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:27: }).catchError(
Should not need to do the catch error- it should propagate automatically.

https://codereview.chromium.org/12380046/diff/27001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:53: (AsyncError e) {
Should probably provide a test function as well and let all other errors
propagate though.

https://codereview.chromium.org/12380046/diff/27001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:65: (e) {
Can just ignore this error, test framework will get it.

https://codereview.chromium.org/12380046/diff/27001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:84: expect(e.error.code,
equals(FileError.NOT_FOUND_ERR));
Ditto for test function for catchError.

https://codereview.chromium.org/12380046/diff/27001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:96: return e.getMetadata().then((Metadata metadata)
{
sent mail with suggested style changes.

https://codereview.chromium.org/12380046/diff/27001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:104: fail('Got file error: ${e.error.code}');
Don't need catchError.

https://codereview.chromium.org/12380046/diff/27001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:112: Future doDirSetup(successCallback) {
Seems like this should have a class which holds the success args and just
returns that in the future. A bit odd to have the callback & future.

https://codereview.chromium.org/12380046/diff/27001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:115: options: {'create': true}).then(
ditto on style changes.

https://codereview.chromium.org/12380046/diff/27001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:136: (e) {
don't need catchError

[Emily Fortuna, 2013-03-06 21:29:18]

reformatted!
ptal

https://codereview.chromium.org/12380046/diff/27001/tests/html/fileapi_test.dart
File tests/html/fileapi_test.dart (right):

https://codereview.chromium.org/12380046/diff/27001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:27: }).catchError(
On 2013/03/06 00:34:29, blois wrote:
> Should not need to do the catch error- it should propagate automatically.

Done.

https://codereview.chromium.org/12380046/diff/27001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:65: (e) {
On 2013/03/06 00:34:29, blois wrote:
> Can just ignore this error, test framework will get it.

Done.

https://codereview.chromium.org/12380046/diff/27001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:104: fail('Got file error: ${e.error.code}');
On 2013/03/06 00:34:29, blois wrote:
> Don't need catchError.

Done.

https://codereview.chromium.org/12380046/diff/27001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:136: (e) {
On 2013/03/06 00:34:29, blois wrote:
> don't need catchError

Done.

[blois, 2013-03-06 21:43:56]

https://codereview.chromium.org/12380046/diff/29001/tests/html/fileapi_test.dart
File tests/html/fileapi_test.dart (right):

https://codereview.chromium.org/12380046/diff/29001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:48: .catchError((AsyncError e) {
Should still use the test filter

https://codereview.chromium.org/12380046/diff/29001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:72: .catchError((AsyncError e) {
Should still use the test filter

https://codereview.chromium.org/12380046/diff/29001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:105: .then((DirectoryEntry dir) {
formatting

https://codereview.chromium.org/12380046/diff/29001/tests/html/fileapi_test.d...
tests/html/fileapi_test.dart:192: .then((fileObj) {
formatting

[Emily Fortuna, 2013-03-06 21:56:06]

ptal

[blois, 2013-03-06 22:12:53]

lgtm

[Emily Fortuna, 2013-03-06 22:33:07]

Committed patchset #3 manually as r19586 (presubmit successful).