Also use transition types when constructing Events that raise exceptions.

Also use transition types when constructing Events that raise exceptions.

Follow up the switch to making Events be garbage collected in
http://codereview.chromium.org/182763002 and also use a transition
type for the result if the event constructor raises exceptions.

R=haraken@chromium.org
BUG=340522
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=168149

[sof, 2014-02-28 16:26:59]

Please take a look.

(Four fast/events tests assert & fail on trunk, which is how i noticed.)

[haraken, 2014-02-28 16:34:10]

On 2014/02/28 16:26:59, sof wrote:
> Please take a look.
> 
> (Four fast/events tests assert & fail on trunk, which is how i noticed.)

Thanks for catching this. LGTM.

I want to make this a compile error instead of an assertion. I come up with one
idea, so let me try on Monday.

[sof, 2014-02-28 16:37:32]

On 2014/02/28 16:34:10, haraken wrote:
> On 2014/02/28 16:26:59, sof wrote:
> > Please take a look.
> > 
> > (Four fast/events tests assert & fail on trunk, which is how i noticed.)
> 
> Thanks for catching this. LGTM.
> 
> I want to make this a compile error instead of an assertion. I come up with
one
> idea, so let me try on Monday.

Sounds good, the more errors we can generate at compile-time, the better. :)

[sof, 2014-02-28 16:38:09]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-02-28 16:38:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sigbjornf@opera.com/183973007/1

[commit-bot: I haz the power, 2014-02-28 18:23:09]

Change committed as 168149