1167607 - Move AsyncEventDispatcher from using DispatchChromeEvent

Status: RESOLVED FIXED

(Core :: DOM: Events, defect)

Description

[Xidorn Quan [:xidorn] UTC+11]

Currently, AsyncEventDispatcher uses nsContentUtils::DispatchChromeEvent for chrome-only event. It could cause undesired behavior if the target is in chrome. We should make it use normal DispatchEvent method with mOnlyChromeDispatch flag set.

As noted by :smaug, we should rename {a,e}DispatchChromeOnly to {a,e}OnlyChromeDispatch, and add comment to describe the behavior.

This changes the current behavior of that class, but it seems to be fine based on test result from try server: https://treeherder.mozilla.org/#/jobs?repo=try&revision=aa5369bee7a2

Comment 1

[Xidorn Quan [:xidorn] UTC+11]

Attachment: patch

Comment 2

[Xidorn Quan [:xidorn] UTC+11]

Attachment: patch

With the renaming, found some other place to change. The new try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=22327de3e9ff

Comment 3

[Xidorn Quan [:xidorn] UTC+11]

Attachment: patch

Add the fix for the failure on the bc1 test. New try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=84472593c13f

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8609941 [details] [diff] [review]
patch


>@@ -4237,17 +4237,17 @@ nsDocument::AddStyleSheetToStyleSets(nsI
>     init.memberName = argName;                                                \
>                                                                               \
>     nsRefPtr<className> event =                                               \
>       className::Constructor(this, NS_LITERAL_STRING(type), init);            \
>     event->SetTrusted(true);                                                  \
>     event->SetTarget(this);                                                   \
>     nsRefPtr<AsyncEventDispatcher> asyncDispatcher =                          \
>       new AsyncEventDispatcher(this, event);                                  \
>-    asyncDispatcher->mDispatchChromeOnly = true;                              \
>+    asyncDispatcher->mOnlyChromeDispatch = true;                              \


oh, tiny bit surprising code. I would have expected the event to have mOnlyChromeDispatch flag.
But no need to change, not about this bug.

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/51b6ac207b1a

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/51b6ac207b1a