989212 - Rename nsEventStates to mozilla::EventStates

Status: RESOLVED FIXED

(Core :: DOM: Events, defect)

Comment 1

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Patch

It might be better to define NS_EVENT_STATE_* as constant in mozilla:: namespace. But it should be worked in another bug if it's necessary.

Comment 2

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

And the patch makes all cpp files which use EventState class or NS_EVENT_STATE_* include the header.

Comment 3

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

Comment on attachment 8399865 [details] [diff] [review]
Patch

Update IID of Element and nsIDocument and Link

+EventStateManager::SetContentState(nsIContent *aContent, EventStates aState)
While you're here, want to move * to be next to nsIContent, not aContent.


+        EventStates eventState =
+          GetContentState(isXULCheckboxRadio ? aFrame->GetParent() : aFrame,
                                              aWidgetType);

Indentation for aWidgetType is odd

Comment 4

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b29352bc495f

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b29352bc495f