Run DOM callbacks in the zone they are coming from.

Run DOM callbacks in the zone they are coming from.

This CL is an adapted version of: https://chromiumcodereview.appspot.com/23054006/

R=blois@google.com

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

[floitsch, 2013-09-25 14:28:05]

@Pete: could you please verify that the test completes successfully with
Dartium?

[blois, 2013-09-25 16:51:48]

Seeing an error in html/events_test:
5	FAIL	Expectation: DOM event callbacks are associated with the correct zone.
Caught type 'Null' is not a subtype of type 'bool' of 'boolean expression'.

[blois, 2013-09-25 17:57:33]

On 2013/09/25 16:51:48, blois wrote:
> Seeing an error in html/events_test:
> 5	FAIL	Expectation: DOM event callbacks are associated with the correct zone.
> Caught type 'Null' is not a subtype of type 'bool' of 'boolean expression'.

Specifically, I'm seeing this when running under DRT, but not when running
content_shell w/ UI. Can investigate further if needed.

[floitsch, 2013-09-26 14:17:36]

Learned that the test was synchronous...
Moved code in the test, to actually make sure we end up in the correct zone.

I have now compiled Dartium locally and I can't make the test fail. But I don't
even know where to get DRT.

If you have a failing test, please investigate. Maybe the test is written in a
bad way?

[floitsch, 2013-09-26 16:36:36]

PTAL.
I patched it into my local Dartium and the test passed.

[blois, 2013-09-26 16:41:27]

lgtm

https://codereview.chromium.org/24509004/diff/3001/tests/html/events_test.dart
File tests/html/events_test.dart (right):

https://codereview.chromium.org/24509004/diff/3001/tests/html/events_test.dar...
tests/html/events_test.dart:106: sub =
provider.forTarget(element).listen(expectAsync1(handler));
can also do:
element.on['test'].listen(...)

[floitsch, 2013-09-26 16:49:46]

https://codereview.chromium.org/24509004/diff/3001/tests/html/events_test.dart
File tests/html/events_test.dart (right):

https://codereview.chromium.org/24509004/diff/3001/tests/html/events_test.dar...
tests/html/events_test.dart:106: sub =
provider.forTarget(element).listen(expectAsync1(handler));
On 2013/09/26 16:41:27, blois wrote:
> can also do:
> element.on['test'].listen(...)

Done.

[floitsch, 2013-09-26 16:50:35]

Committed patchset #8 manually as r27961 (presubmit successful).