Bug 113676 – [Shadow DOM] Change the order of event dispatching at AT_TARGET phase.

• Component: UI Events
• Status: RESOLVED FIXED

[Hayato Ito]

The discussion on the spec side here:
https://www.w3.org/Bugs/Public/show_bug.cgi?id=21404


Given the event path, from Node A (top-most) to Node G (target, inner-most), where C and D are shadow hosts and G is the target, 
an event dispatching order in the current WebKit implementaion is:
(T: AT_TARGET, C: CAPTURING, B: BUBBLING)


For bubbling events:

  A          1 (C)  11 (B)
  B          2 (C)  10 (B)
  C (SH)     3 (T) 
  D          4 (C)   9 (B)
  E (SH)     5 (T)
  F          6 (C)   8 (B)
  G (Target) 7 (T)  


For non-bubbling events:

  A          1 (C)
  B          2 (C)
  C (SH)     3 (T) 
  D          4 (C)
  E (SH)     5 (T)
  F          6 (C)
  G (Target) 7 (T) 



We should change the order as follows:

For bubbling events:

  A          1 (C)  11 (B)
  B          2 (C)  10 (B)
  C (SH)             9 (T) 
  D          3 (C)   8 (B)
  E (SH)             7 (T)
  F          4 (C)   6 (B)
  G (Target)         5 (T)  


For non-bubbling events:

  A          1 (C)
  B          2 (C)
  C (SH)            7 (T) 
  D          3 (C)
  E (SH)            6 (T)
  F          4 (C)
  G (Target)        5 (T)

[Hayato Ito]

Created attachment 195925 [details]
Change the order

[Dimitri Glazkov (Google)]

Comment on attachment 195925 [details]
Change the order

This is great. Also, looks like a more straightforward code path.

[WebKit Review Bot]

Comment on attachment 195925 [details]
Change the order

Clearing flags on attachment: 195925

Committed r147371: <http://trac.webkit.org/changeset/147371>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.