Closed Bug 895274 Opened 11 years ago Closed 9 years ago

Change GUI MESSAGES to enum

Categories

(Core :: DOM: Events, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla43
Tracking Status
firefox43 --- fixed

People

(Reporter: kanru, Assigned: masayuki)

References

Details

Attachments

(271 files, 2 obsolete files)

278.44 KB, patch
smaug
: review+
Details | Diff | Splinter Review
87.03 KB, patch
smaug
: review+
Details | Diff | Splinter Review
243.49 KB, patch
smaug
: review+
Details | Diff | Splinter Review
241.15 KB, patch
Details | Diff | Splinter Review
8.50 KB, patch
smaug
: review+
Details | Diff | Splinter Review
35.52 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.45 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.70 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.56 KB, patch
smaug
: review+
Details | Diff | Splinter Review
63.95 KB, patch
smaug
: review+
Details | Diff | Splinter Review
40.31 KB, patch
smaug
: review+
Details | Diff | Splinter Review
41.89 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.99 KB, patch
smaug
: review+
Details | Diff | Splinter Review
6.12 KB, patch
smaug
: review+
Details | Diff | Splinter Review
6.67 KB, patch
smaug
: review+
Details | Diff | Splinter Review
6.57 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.73 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.20 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.88 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.07 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.11 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.04 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.32 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.24 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.12 KB, patch
smaug
: review+
Details | Diff | Splinter Review
54.64 KB, patch
smaug
: review+
Details | Diff | Splinter Review
69.74 KB, patch
smaug
: review+
Details | Diff | Splinter Review
68.62 KB, patch
smaug
: review+
Details | Diff | Splinter Review
21.23 KB, patch
smaug
: review+
Details | Diff | Splinter Review
25.29 KB, patch
smaug
: review+
Details | Diff | Splinter Review
20.49 KB, patch
smaug
: review+
Details | Diff | Splinter Review
31.05 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.57 KB, patch
smaug
: review+
Details | Diff | Splinter Review
16.70 KB, patch
smaug
: review+
Details | Diff | Splinter Review
18.14 KB, patch
smaug
: review+
Details | Diff | Splinter Review
6.78 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.72 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.27 KB, patch
smaug
: review+
Details | Diff | Splinter Review
8.17 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.86 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.76 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.58 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.64 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.35 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.20 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.82 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.67 KB, patch
smaug
: review+
Details | Diff | Splinter Review
7.34 KB, patch
smaug
: review+
Details | Diff | Splinter Review
7.59 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.72 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.50 KB, patch
smaug
: review+
Details | Diff | Splinter Review
18.93 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.37 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.22 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.00 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.94 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.21 KB, patch
smaug
: review+
Details | Diff | Splinter Review
6.79 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.59 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.19 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.79 KB, patch
smaug
: review+
Details | Diff | Splinter Review
12.68 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.87 KB, patch
smaug
: review+
Details | Diff | Splinter Review
13.28 KB, patch
smaug
: review+
Details | Diff | Splinter Review
10.09 KB, patch
smaug
: review+
Details | Diff | Splinter Review
6.09 KB, patch
smaug
: review+
Details | Diff | Splinter Review
8.10 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.82 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.58 KB, patch
smaug
: review+
Details | Diff | Splinter Review
23.37 KB, patch
smaug
: review+
Details | Diff | Splinter Review
23.96 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.58 KB, patch
smaug
: review+
Details | Diff | Splinter Review
8.55 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.87 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.52 KB, patch
smaug
: review+
Details | Diff | Splinter Review
19.34 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.78 KB, patch
smaug
: review+
Details | Diff | Splinter Review
19.21 KB, patch
smaug
: review+
Details | Diff | Splinter Review
9.10 KB, patch
smaug
: review+
Details | Diff | Splinter Review
7.75 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.40 KB, patch
smaug
: review+
Details | Diff | Splinter Review
17.99 KB, patch
smaug
: review+
Details | Diff | Splinter Review
13.15 KB, patch
smaug
: review+
Details | Diff | Splinter Review
16.11 KB, patch
smaug
: review+
Details | Diff | Splinter Review
8.06 KB, patch
smaug
: review+
Details | Diff | Splinter Review
9.89 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.05 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.30 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.63 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.93 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.68 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.61 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.54 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.23 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.04 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.27 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.68 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.28 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.01 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.71 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.89 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.94 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.63 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.70 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.53 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.04 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.05 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.18 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.52 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.38 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.78 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.80 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.59 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.20 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.77 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.60 KB, patch
smaug
: review+
Details | Diff | Splinter Review
12.22 KB, patch
smaug
: review+
Details | Diff | Splinter Review
10.53 KB, patch
smaug
: review+
Details | Diff | Splinter Review
16.53 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.27 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.71 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.90 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.90 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.89 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.87 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.92 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.91 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.99 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.10 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.16 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.03 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.02 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.02 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.00 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.93 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.92 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.94 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.87 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.83 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.84 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.86 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.30 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.51 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.16 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.62 KB, patch
smaug
: review+
Details | Diff | Splinter Review
15.71 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.63 KB, patch
smaug
: review+
Details | Diff | Splinter Review
10.66 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.98 KB, patch
smaug
: review+
Details | Diff | Splinter Review
12.42 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.90 KB, patch
smaug
: review+
Details | Diff | Splinter Review
9.32 KB, patch
smaug
: review+
Details | Diff | Splinter Review
29.94 KB, patch
smaug
: review+
Details | Diff | Splinter Review
22.02 KB, patch
smaug
: review+
Details | Diff | Splinter Review
17.07 KB, patch
smaug
: review+
Details | Diff | Splinter Review
13.87 KB, patch
smaug
: review+
Details | Diff | Splinter Review
7.12 KB, patch
smaug
: review+
Details | Diff | Splinter Review
13.23 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.37 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.50 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.96 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.98 KB, patch
smaug
: review+
Details | Diff | Splinter Review
8.44 KB, patch
smaug
: review+
Details | Diff | Splinter Review
8.61 KB, patch
smaug
: review+
Details | Diff | Splinter Review
7.48 KB, patch
smaug
: review+
Details | Diff | Splinter Review
7.83 KB, patch
smaug
: review+
Details | Diff | Splinter Review
6.69 KB, patch
smaug
: review+
Details | Diff | Splinter Review
8.01 KB, patch
smaug
: review+
Details | Diff | Splinter Review
8.41 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.63 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.54 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.85 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.85 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.89 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.38 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.05 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.81 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.51 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.97 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.19 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.45 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.84 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.81 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.49 KB, patch
smaug
: review+
Details | Diff | Splinter Review
21.92 KB, patch
smaug
: review+
Details | Diff | Splinter Review
9.08 KB, patch
smaug
: review+
Details | Diff | Splinter Review
7.53 KB, patch
smaug
: review+
Details | Diff | Splinter Review
19.63 KB, patch
smaug
: review+
Details | Diff | Splinter Review
15.12 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.53 KB, patch
smaug
: review+
Details | Diff | Splinter Review
19.98 KB, patch
smaug
: review+
Details | Diff | Splinter Review
9.99 KB, patch
smaug
: review+
Details | Diff | Splinter Review
7.86 KB, patch
smaug
: review+
Details | Diff | Splinter Review
31.40 KB, patch
smaug
: review+
Details | Diff | Splinter Review
16.46 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.54 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.65 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.56 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.97 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.89 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.95 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.87 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.93 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.99 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.69 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.01 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.79 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.72 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.56 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.37 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.47 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.93 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.93 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.90 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.80 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.64 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.27 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.61 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.16 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.02 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.30 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.96 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.85 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.70 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.60 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.48 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.68 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.78 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.52 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.52 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.15 KB, patch
smaug
: review+
Details | Diff | Splinter Review
995 bytes, patch
smaug
: review+
Details | Diff | Splinter Review
5.25 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.16 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.08 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.35 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.50 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.96 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.83 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.82 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.54 KB, patch
smaug
: review+
Details | Diff | Splinter Review
28.22 KB, patch
smaug
: review+
Details | Diff | Splinter Review
18.23 KB, patch
smaug
: review+
Details | Diff | Splinter Review
25.36 KB, patch
smaug
: review+
Details | Diff | Splinter Review
18.30 KB, patch
smaug
: review+
Details | Diff | Splinter Review
2.34 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.98 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.21 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.39 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.81 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.77 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.24 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.14 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.49 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.47 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.50 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.32 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.39 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.61 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.33 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.37 KB, patch
smaug
: review+
Details | Diff | Splinter Review
6.30 KB, patch
smaug
: review+
Details | Diff | Splinter Review
6.11 KB, patch
smaug
: review+
Details | Diff | Splinter Review
6.01 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.46 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.34 KB, patch
smaug
: review+
Details | Diff | Splinter Review
5.14 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.99 KB, patch
smaug
: review+
Details | Diff | Splinter Review
4.38 KB, patch
smaug
: review+
Details | Diff | Splinter Review
3.47 KB, patch
smaug
: review+
Details | Diff | Splinter Review
1.58 KB, patch
smaug
: review+
Details | Diff | Splinter Review
Is there any reason not to?
Enum should probably work. In theory that limits certain kinds of binary addons which might add
new types of event structs, but I haven't heard of any such addon.
Attached patch WIP patch (obsolete) — Splinter Review
Several places still use static_cast. One is the mId in EventNameMapping. I'm not sure how is it used. The others are in the interfaces defined in XPIDL, they accept uint32_t instead of the enum.

The name nsGUIMessage should be improved since in many places the variable are named aEventType but message.
Attachment #778361 - Flags: feedback?(bugs)
Comment on attachment 778361 [details] [diff] [review]
WIP patch

The name of the enum should not be nsGUIMessage.
Perhaps mozilla::EventMessageType or mozilla::EventMessage.

Couldn't you just change mapping's mId to be type of this new thing?
We really shouldn't be static_casting.

The naming should change. All thos NS_* event messages should
become eFooBar or eEventType_FooBar.

what is the sizeof a member variable which has type of this new enum?
nsEventListenerManager has an optimization and the sizeof mNoListenerForEvent
must not be more than 23 bits.
Attachment #778361 - Flags: feedback?(bugs) → feedback-
Many of the ns*Event types are not in the mozilla namespace. Should we fix this? Otherwise we have to use mozilla::EventMessageType in those declarations :(
Is there something wrong with using mozilla:: in those ns*Events?
Looks like that nobody works on this. Can I take this when I have time to fix this? Making event messages an enum help debug on Visual Studio.
(In reply to Masayuki Nakano (:masayuki) (Mozilla Japan) from comment #7)
> Looks like that nobody works on this. Can I take this when I have time to
> fix this? Making event messages an enum help debug on Visual Studio.

Please take it :)
I thought this as part of the Events refactoring effort but didn't find time to finish it.
Okay. Taking.
Assignee: nobody → masayuki
Status: NEW → ASSIGNED
Renaming WidgetEvent::message -> WidgetEvent::mMessage.

# I'd like to land patches separately because there will be a lot of patches to rename NS_* to eEventMessage_*.
Attachment #778361 - Attachment is obsolete: true
Attachment #8650912 - Flags: review?(bugs)
Attachment #8650912 - Flags: review?(bugs) → review+
I'll make the enum named enum, and rename each message name in following patches (next week).
Attachment #8651003 - Flags: review?(bugs)
And the patch breaks the 80 column per line rule, but please ignore that because each line will be modified when I rename it.
Comment on attachment 8651003 [details] [diff] [review]
part.2 Define event messages as anonymous enum with EventMessageList.h

The patch is a bit odd looking. For some reason first tons of stuff is removed, then the same thing copied and modified... I guess hg isn't clever enough to create better looking patch.

>+enum
>+{
>+
>+#define NS_EVENT_MESSAGE(aMessage, aValue) aMessage = aValue,
>+
>+#include "mozilla/EventMessageList.h"
>+
>+#undef NS_EVENT_MESSAGE
>+
>+  // For preventing bustage due to "," after the last item.
>+  eEventMessage_MaxValue
>+};

It would be nice if we could assert here or in ELM that aValue is always 
<= ((1 << 22) - 1) so that mNoListenerForEvent can handle the value.
But that can be done in a separate patch.
Attachment #8651003 - Flags: review?(bugs) → review+
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/bb91cf98a42b08db96677dd8bc3ebe5218945318
changeset:  bb91cf98a42b08db96677dd8bc3ebe5218945318
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 22 10:34:51 2015 +0900
description:
Bug 895274 part.1 Rename WidgetEvent::message to WidgetEvent::mMessage r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/756c006944ab770d06841c63a5d0f00616d3d41e
changeset:  756c006944ab770d06841c63a5d0f00616d3d41e
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 22 10:34:51 2015 +0900
description:
Bug 895274 part.2 Define event messages as anonymous enum with EventMessageList.h r=smaug
Whiteboard: [leave open]
It seems that we should use enum class instead of named enum. However, for using it, we need to replace all items of the enum. It's difficult to do it for now. Even if you like enum class better, we should use named enum for now. After we will add unique prefix like "eEventMessage_" at renaming each item, we can replace all items easier. So, if you like enum class better, please let me know. I'll do that after renaming all event messages.

Therefore, this patch just makes the enum of event messages named enum, "mozilla::EventMessage". And the type of this enum can be uint16_t rather than uint32_t.

This patch also checks the size of EventListenerManager's size at build time with static_assert(). For dosing that, I separated the integer members to EventListenerManagerBase.

And also this patch names all variables of EventMessage as "EventMesage", "Message" or "Msg".

Additionally, this patch adds a lot of "default: break;" to the switch statements since they are necessary for avoiding warnings.

I'll post this patch with -w.
Attachment #8652771 - Flags: review?(bugs)
FYI: I don't change nsBaseDragService::FireDragEventAtSource()'s argument since it's a scriptable interface method. We should make it non-scriptable or expose the concrete class later.
Now, we shouldn't call the event message as ID.
Attachment #8652775 - Flags: review?(bugs)
Attachment #8652775 - Flags: review?(bugs) → review+
Attachment #8652771 - Flags: review?(bugs) → review+
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/cd06f5f5641c3dfb7f07db878584d389fb4df1b7
changeset:  cd06f5f5641c3dfb7f07db878584d389fb4df1b7
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Aug 26 21:56:59 2015 +0900
description:
Bug 895274 part.3 Make the enum of event messages a named enum IGNORE IDL r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/5dc1087883b0c3685006722dbee0995c815ac4c1
changeset:  5dc1087883b0c3685006722dbee0995c815ac4c1
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Aug 26 21:56:59 2015 +0900
description:
Bug 895274 part.4 Clean up the macro names of EventNameList.h r=smaug
What's name do you like for each event message? I think normal event messages should be:

eEventMessage_KeyDown,
eEventMessage_MouseClick,
eEventMessage_MouseEnterWidget,
eEventMessage_MouseEnter,

On the other hand, the "start" and "end" messages should be "first" and "last" like the messages of Windows. start and end could be used by explaining operations, e.g., "drag start". So, I think they should be:

eEventMessage_KeyFirst,
eEventMessage_KeyLast,
eEventMessage_MouseFirst,
eEventMessage_MouseLast,

Do you have some better ideas?
Flags: needinfo?(bugs)
Do we need eEventMessage_* ?
Why not just eKeyDown etc?

first and last sound good.
Flags: needinfo?(bugs)
(In reply to Olli Pettay [:smaug] (high review load) from comment #30)
> Do we need eEventMessage_* ?
> Why not just eKeyDown etc?

Oh, does it OK? You don't think that we should make them enum class for stricter type check? If we should do that, common prefix helps me to replace with EventMessage::eKeyDown or something. (But if we don't need to do that, shorter name is better!)
Flags: needinfo?(bugs)
Well, does the longer name really help with type check? I think not. So I prefer shorter names, since those are still very clear.
Flags: needinfo?(bugs)
(In reply to Olli Pettay [:smaug] (high review load) from comment #32)
> Well, does the longer name really help with type check? I think not. So I
> prefer shorter names, since those are still very clear.

Okay, then, we should keep using named enum and shorter item names. I'll create some patches tomorrow.
FYI: I realized that we don't have any specific rule of naming enum items...
https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style
(In reply to Olli Pettay [:smaug] (high review load) from comment #32)
> Well, does the longer name really help with type check?

Oops, I missed this point. If we use enum class, we have following merits:

We can reduce "e" prefix because each item name is valid only under the class name. So, e.g., EventMessage::Null won't conflict with anything.

enum class allows to use forward declaration. Therefore, header files don't need to include the defined header file only for a method which has EventMessage as a result type or an argument.

And enum class doesn't allow following code:

uint32_t foo = enumclass::bar; // error at build time

So, when we need to use to set to/get from integer, we need to cast explicitly.
Flags: needinfo?(bugs)
In other words, shorter names are used for us, but they may conflict with other enum items if we name some of them with too generic name.
ah, conflicts are possible, though with e-prefix hopefully unlikely.
Flags: needinfo?(bugs)
Hmm, I feel following names are odd:

NS_KEY_BEFORE_DOWN
NS_KEY_AFTER_DOWN 
NS_KEY_BEFORE_UP  
NS_KEY_AFTER_UP   

They are named as "mozbrowser(before|after)key(down|up). And the class name is InternalBeforeAfterKeyboardEvent. So, in the current idea, they should be e(Before|After)Key(Down|Up)...
I guess that if we change the enum to enum class, the longest name will be probably EventMessage::QuerySelectionAsTransferable. It's 8 characters longer than NS_QUERY_SELECTION_AS_TRANSFERABLE.
Hmm, I was hoping e-prefixed names, and without EventMessage:: everywhere.

And yes, those before/after names could be changed.
Okay, then, I'll keep yesterday's rules.
Er, *was* hoping?
I think that message indicates the meaning of message. Then, void is better than null.
Attachment #8653881 - Flags: review?(bugs)
Looks like that this is an event fired at closing window (not DOM window). Therefore, I don't think "XUL" is necessary keyword for this.
Attachment #8653884 - Flags: review?(bugs)
I think "window" resize event is better.
Attachment #8653893 - Flags: review?(bugs)
The name may be too generic...
Attachment #8653897 - Flags: review?(bugs)
I guess that somebody forgot to remove them.
Attachment #8653899 - Flags: review?(bugs)
I don't think this needs "Moz" keyword because this enum isn't exposed to the web.

And I'm not sure which is better of "HitTest" and "Hittest". According to the dictionary, a word, "hittest", doesn't exist. Therefore, I use "HitTest" in this patch.
Attachment #8654028 - Flags: review?(bugs)
This also shouldn't need "Moz" keyword.
Attachment #8654033 - Flags: review?(bugs)
Attachment #8653882 - Flags: review?(bugs) → review+
Attachment #8653883 - Flags: review?(bugs) → review+
Attachment #8653884 - Flags: review?(bugs) → review+
Attachment #8653888 - Flags: review?(bugs) → review+
Attachment #8653889 - Flags: review?(bugs) → review+
Attachment #8653890 - Flags: review?(bugs) → review+
Attachment #8653891 - Flags: review?(bugs) → review+
Comment on attachment 8653893 [details] [diff] [review]
part.16 Raneme NS_RESIZE to eWindowResize

I think this should be just eResize, since, as odd as it is, there is resize event dispatched also to media elements.
With that, r+
Attachment #8653893 - Flags: review?(bugs) → review+
Comment on attachment 8654033 [details] [diff] [review]
part.38 Raneme NS_MOUSE_MOZLONGTAP to eMouseLongTap

># HG changeset patch
># User Masayuki Nakano <masayuki@d-toybox.com>
># Parent  ba03386e5ff331d963e3c8de27ad24733078b28b
>Bug 895274 part.38 Raneme NS_MOUSE_MOZLONGTAP to eMouseLongTap r=
Rename
Attachment #8654033 - Flags: review?(bugs) → review+
Comment on attachment 8653894 [details] [diff] [review]
part.17 Raneme NS_SCROLL_EVENT to eScroll

># HG changeset patch
># User Masayuki Nakano <masayuki@d-toybox.com>
># Parent  6b624e1d45ede081c9de33b2af195d64a40df1a3
>Bug 895274 part.17 Raneme NS_SCROLL_EVENT to eScroll r=
Rename
Attachment #8653894 - Flags: review?(bugs) → review+
Comment on attachment 8653895 [details] [diff] [review]
part.18 Raneme NS_PLUGIN_ACTIVATE to ePluginActivate

># HG changeset patch
># User Masayuki Nakano <masayuki@d-toybox.com>
># Parent  c17930d465c38337eee420ef7b3afae2f69b00fc
>Bug 895274 part.18 Raneme NS_PLUGIN_ACTIVATE to ePluginActivate r=
Rename

I guess the same copy-past mistake in most of the commit messages ;)
So, not complaining that more, but please fix everywhere.
Attachment #8653895 - Flags: review?(bugs) → review+
Attachment #8653896 - Flags: review?(bugs) → review+
Attachment #8653897 - Flags: review?(bugs) → review+
Attachment #8653898 - Flags: review?(bugs) → review+
Comment on attachment 8653899 [details] [diff] [review]
part.22 Get rid of NS_MOZ_USER_* since nobody is using them

I would just remove the unused code and the comment about these.
With that, r+
Attachment #8653899 - Flags: review?(bugs) → review+
Attachment #8654025 - Flags: review?(bugs) → review+
Attachment #8654028 - Flags: review?(bugs) → review+
Attachment #8654029 - Flags: review?(bugs) → review+
Attachment #8654032 - Flags: review?(bugs) → review+
Attachment #8653900 - Flags: review?(bugs) → review+
Attachment #8654016 - Flags: review?(bugs) → review+
Attachment #8653881 - Flags: review?(bugs) → review+
Attachment #8654026 - Flags: review?(bugs) → review+
Attachment #8654027 - Flags: review?(bugs) → review+
Attachment #8654022 - Flags: review?(bugs) → review+
Attachment #8654023 - Flags: review?(bugs) → review+
Attachment #8653886 - Flags: review?(bugs) → review+
Attachment #8653885 - Flags: review?(bugs) → review+
Attachment #8653887 - Flags: review?(bugs) → review+
Comment on attachment 8654020 [details] [diff] [review]
part.28 Raneme NS_MOUSE_ENTER_WIDGET to eMouseEnterIntoWidget

>-  // Do not take account NS_MOUSE_ENTER_WIDGET/EXIT_WIDGET so that loading a page
>+  // Do not take account eMouseEnterIntoWidget/FromWidget so that loading a page
eMouseEnterIntoWidget/ExitFromWidget
Attachment #8654020 - Flags: review?(bugs) → review+
Attachment #8654021 - Flags: review?(bugs) → review+
Attachment #8654017 - Flags: review?(bugs) → review+
Attachment #8654018 - Flags: review?(bugs) → review+
Attachment #8654019 - Flags: review?(bugs) → review+
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/3de33e39e9ce795331196508ea5d36f2547a4bcb
changeset:  3de33e39e9ce795331196508ea5d36f2547a4bcb
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:26 2015 +0900
description:
Bug 895274 part.5 Rename NS_EVENT_NULL to eVoidEvent r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/8efaeb0052b0a101a70ce32f71f72c825591833e
changeset:  8efaeb0052b0a101a70ce32f71f72c825591833e
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:26 2015 +0900
description:
Bug 895274 part.6 Rename NS_EVENT_ALL to eAllEvents r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/2a4abd0f4d111e3bce761afc1d78d624664eea0c
changeset:  2a4abd0f4d111e3bce761afc1d78d624664eea0c
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:26 2015 +0900
description:
Bug 895274 part.7 Rename NS_WINDOW_START to eWindowEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/ae177f1853f552127ca9216d487bd014bbd2d6df
changeset:  ae177f1853f552127ca9216d487bd014bbd2d6df
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:26 2015 +0900
description:
Bug 895274 part.8 Rename NS_XUL_CLOSE to eWindowClose r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/6e12f5164cb9f833ec9cbb958131aadb530e164c
changeset:  6e12f5164cb9f833ec9cbb958131aadb530e164c
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:27 2015 +0900
description:
Bug 895274 part.9 Rename NS_KEY_PRESS to eKeyPress r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/83b87aa188ab9c744697b87eb0c47c8a13d5babc
changeset:  83b87aa188ab9c744697b87eb0c47c8a13d5babc
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:27 2015 +0900
description:
Bug 895274 part.10 Rename NS_KEY_UP to eKeyUp r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/4a2797061277a5a2a3ba878144431a3b2f50392f
changeset:  4a2797061277a5a2a3ba878144431a3b2f50392f
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:27 2015 +0900
description:
Bug 895274 part.11 Rename NS_KEY_DOWN to eKeyDown r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/d826b30611d8941e7ff487ea8acd51cade119ce9
changeset:  d826b30611d8941e7ff487ea8acd51cade119ce9
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:27 2015 +0900
description:
Bug 895274 part.12 Rename NS_KEY_BEFORE_DOWN to eBeforeKeyDown r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/57679af15e80758688d4bc0f54b662a084ce0904
changeset:  57679af15e80758688d4bc0f54b662a084ce0904
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:28 2015 +0900
description:
Bug 895274 part.13 Rename NS_KEY_AFTERE_DOWN to eAfterKeyDown r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/15282e20479563dc050ec3015610d880001baed2
changeset:  15282e20479563dc050ec3015610d880001baed2
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:28 2015 +0900
description:
Bug 895274 part.14 Rename NS_KEY_BEFORE_UP to eBeforeKeyUp r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/ef04b1227c95e42b29d6bf0626c021b24871d767
changeset:  ef04b1227c95e42b29d6bf0626c021b24871d767
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:28 2015 +0900
description:
Bug 895274 part.15 Rename NS_KEY_AFTER_UP to eAfterKeyUp r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/90da8b9aaf2e777b0ed64773f75e4cd45a9b3e8a
changeset:  90da8b9aaf2e777b0ed64773f75e4cd45a9b3e8a
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:28 2015 +0900
description:
Bug 895274 part.16 Rename NS_RESIZE to eResize r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/862ef097c279ec53b682694351d11e61083e08d0
changeset:  862ef097c279ec53b682694351d11e61083e08d0
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:28 2015 +0900
description:
Bug 895274 part.17 Rename NS_SCROLL_EVENT to eScroll r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/da3eb3291c32042466fcc13813d85b6b758413d7
changeset:  da3eb3291c32042466fcc13813d85b6b758413d7
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:28 2015 +0900
description:
Bug 895274 part.18 Rename NS_PLUGIN_ACTIVATE to ePluginActivate r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/4a2099a81968ec3a5f5a60a682170ceb28ae8dd4
changeset:  4a2099a81968ec3a5f5a60a682170ceb28ae8dd4
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:28 2015 +0900
description:
Bug 895274 part.19 Rename NS_PLUGIN_FOCUS to ePluginFocus r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/b409cf40a9f3b785427eaf3b387cc0b240f9c619
changeset:  b409cf40a9f3b785427eaf3b387cc0b240f9c619
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:29 2015 +0900
description:
Bug 895274 part.20 Rename NS_OFFLINE to eOffline r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/55b1a8515716fb1d6af18392e78261d7ce737c52
changeset:  55b1a8515716fb1d6af18392e78261d7ce737c52
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:29 2015 +0900
description:
Bug 895274 part.21 Rename NS_ONLINE to eOnline r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/746fe566eb9f29590d4ea733db34bef46d62ed1d
changeset:  746fe566eb9f29590d4ea733db34bef46d62ed1d
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:29 2015 +0900
description:
Bug 895274 part.22 Get rid of NS_MOZ_USER_* since nobody is using them r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/31d9af1df30fe60d55b3078776d197e311b73ff6
changeset:  31d9af1df30fe60d55b3078776d197e311b73ff6
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:29 2015 +0900
description:
Bug 895274 part.23 Rename NS_LANGUAGECHANGE to eLanguageChange r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/ef0b9c30a9e7581939b413908f2f9b7699ef9974
changeset:  ef0b9c30a9e7581939b413908f2f9b7699ef9974
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:29 2015 +0900
description:
Bug 895274 part.24 Rename NS_MOUSE_MESSAGE_START to eMouseEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/cd4d30cf8ecfb8789d0e69c2adcdfe4c1de7ac8f
changeset:  cd4d30cf8ecfb8789d0e69c2adcdfe4c1de7ac8f
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:29 2015 +0900
description:
Bug 895274 part.25 Rename NS_MOUSE_MOVE to eMouseMove r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/ae625cd90875c0f939e83fc58c9e40fb3b6fd75f
changeset:  ae625cd90875c0f939e83fc58c9e40fb3b6fd75f
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:30 2015 +0900
description:
Bug 895274 part.26 Rename NS_MOUSE_BUTTON_UP to eMouseUp r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/e637f3ff6f3e3add6aa4ac4b7325e3983504cef7
changeset:  e637f3ff6f3e3add6aa4ac4b7325e3983504cef7
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:30 2015 +0900
description:
Bug 895274 part.27 Rename NS_MOUSE_BUTTON_DOWN to eMouseDown r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/389f60113c0c037c562eb46f4dfa23feab74a20e
changeset:  389f60113c0c037c562eb46f4dfa23feab74a20e
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:31 2015 +0900
description:
Bug 895274 part.28 Rename NS_MOUSE_ENTER_WIDGET to eMouseEnterIntoWidget r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/6949e818c397150c21e14c3d05edb55a86255eba
changeset:  6949e818c397150c21e14c3d05edb55a86255eba
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:31 2015 +0900
description:
Bug 895274 part.29 Rename NS_MOUSE_EXIT_WIDGET to eMouseExitFromWidget r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/881d177f345734dee4b9b6a82da9663357dd9da1
changeset:  881d177f345734dee4b9b6a82da9663357dd9da1
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:31 2015 +0900
description:
Bug 895274 part.30 Rename NS_MOUSE_DOUBLECLICK to eMouseDoubleClick r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/c9288d4f5e63369b8f98319ac029a759519ace62
changeset:  c9288d4f5e63369b8f98319ac029a759519ace62
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:32 2015 +0900
description:
Bug 895274 part.31 Rename NS_MOUSE_CLICK to eMouseClick r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/ce720f26f58cd3a808c4af314582d21a39e77a70
changeset:  ce720f26f58cd3a808c4af314582d21a39e77a70
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:32 2015 +0900
description:
Bug 895274 part.32 Rename NS_MOUSE_ACTIVATE to eMouseActivate r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/b8fe1dac71aeb337dbdb084297b25071d03cf1a0
changeset:  b8fe1dac71aeb337dbdb084297b25071d03cf1a0
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:32 2015 +0900
description:
Bug 895274 part.33 Rename NS_MOUSE_OVER to eMouseOver r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/4bcb03faecc6d51c09068243a4b73240b7af80a2
changeset:  4bcb03faecc6d51c09068243a4b73240b7af80a2
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:32 2015 +0900
description:
Bug 895274 part.34 Rename NS_MOUSE_OUT to eMouseOut r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/0d03f7de34c57ba40ca9a60d41f8ce5ca3a47aae
changeset:  0d03f7de34c57ba40ca9a60d41f8ce5ca3a47aae
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:32 2015 +0900
description:
Bug 895274 part.35 Rename NS_MOUSE_MOZHITTEST to eMouseHitTest r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/691fe61c6289fa27fd6b41a3bc210680992a5718
changeset:  691fe61c6289fa27fd6b41a3bc210680992a5718
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:33 2015 +0900
description:
Bug 895274 part.36 Rename NS_MOUSEENTER to eMouseEnter r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/be7520290d1273ee2c9245f1347f97933d50b216
changeset:  be7520290d1273ee2c9245f1347f97933d50b216
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:33 2015 +0900
description:
Bug 895274 part.37 Rename NS_MOUSELEAVE to eMouseLeave r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/5899c44ed9ce7f54da590ce89d8922d34e4b7ac9
changeset:  5899c44ed9ce7f54da590ce89d8922d34e4b7ac9
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sat Aug 29 08:58:33 2015 +0900
description:
Bug 895274 part.38 Rename NS_MOUSE_MOZLONGTAP to eMouseLongTap r=smaug
FYI: eResize conflicts with nsIPresShell::eResize. Therefore, I append scope to each eResize in nsPresShell.cpp.
https://hg.mozilla.org/integration/mozilla-inbound/diff/90da8b9aaf2e/layout/base/nsPresShell.cpp

If you think that this is bad, let me know.
Flags: needinfo?(bugs)
that is fine.
Flags: needinfo?(bugs)
https://hg.mozilla.org/mozilla-central/rev/3de33e39e9ce
https://hg.mozilla.org/mozilla-central/rev/8efaeb0052b0
https://hg.mozilla.org/mozilla-central/rev/2a4abd0f4d11
https://hg.mozilla.org/mozilla-central/rev/ae177f1853f5
https://hg.mozilla.org/mozilla-central/rev/6e12f5164cb9
https://hg.mozilla.org/mozilla-central/rev/83b87aa188ab
https://hg.mozilla.org/mozilla-central/rev/4a2797061277
https://hg.mozilla.org/mozilla-central/rev/d826b30611d8
https://hg.mozilla.org/mozilla-central/rev/57679af15e80
https://hg.mozilla.org/mozilla-central/rev/15282e204795
https://hg.mozilla.org/mozilla-central/rev/ef04b1227c95
https://hg.mozilla.org/mozilla-central/rev/90da8b9aaf2e
https://hg.mozilla.org/mozilla-central/rev/862ef097c279
https://hg.mozilla.org/mozilla-central/rev/da3eb3291c32
https://hg.mozilla.org/mozilla-central/rev/4a2099a81968
https://hg.mozilla.org/mozilla-central/rev/b409cf40a9f3
https://hg.mozilla.org/mozilla-central/rev/55b1a8515716
https://hg.mozilla.org/mozilla-central/rev/746fe566eb9f
https://hg.mozilla.org/mozilla-central/rev/31d9af1df30f
https://hg.mozilla.org/mozilla-central/rev/ef0b9c30a9e7
https://hg.mozilla.org/mozilla-central/rev/cd4d30cf8ecf
https://hg.mozilla.org/mozilla-central/rev/ae625cd90875
https://hg.mozilla.org/mozilla-central/rev/e637f3ff6f3e
https://hg.mozilla.org/mozilla-central/rev/389f60113c0c
https://hg.mozilla.org/mozilla-central/rev/6949e818c397
https://hg.mozilla.org/mozilla-central/rev/881d177f3457
https://hg.mozilla.org/mozilla-central/rev/c9288d4f5e63
https://hg.mozilla.org/mozilla-central/rev/ce720f26f58c
https://hg.mozilla.org/mozilla-central/rev/b8fe1dac71ae
https://hg.mozilla.org/mozilla-central/rev/4bcb03faecc6
https://hg.mozilla.org/mozilla-central/rev/0d03f7de34c5
https://hg.mozilla.org/mozilla-central/rev/691fe61c6289
https://hg.mozilla.org/mozilla-central/rev/be7520290d12
https://hg.mozilla.org/mozilla-central/rev/5899c44ed9ce
Hmm,

> 81 NS_EVENT_MESSAGE(NS_STREAM_EVENT_START, 1100)
> 82 NS_EVENT_MESSAGE(NS_LOAD,               NS_STREAM_EVENT_START)
> 83 NS_EVENT_MESSAGE(NS_PAGE_UNLOAD,        NS_STREAM_EVENT_START + 1)
> 84 NS_EVENT_MESSAGE(NS_HASHCHANGE,         NS_STREAM_EVENT_START + 2)
> 85 NS_EVENT_MESSAGE(NS_IMAGE_ABORT,        NS_STREAM_EVENT_START + 3)
> 86 NS_EVENT_MESSAGE(NS_LOAD_ERROR,         NS_STREAM_EVENT_START + 4)
> 87 NS_EVENT_MESSAGE(NS_POPSTATE,           NS_STREAM_EVENT_START + 5)
> 88 NS_EVENT_MESSAGE(NS_BEFORE_PAGE_UNLOAD, NS_STREAM_EVENT_START + 6)
> 89 NS_EVENT_MESSAGE(NS_PAGE_RESTORE,       NS_STREAM_EVENT_START + 7)
> 90 NS_EVENT_MESSAGE(NS_READYSTATECHANGE,   NS_STREAM_EVENT_START + 8)

Do you think "Page" is necessary for unload event?

eLoad, eUnload feel enough to me. I'm not sure NS_PAGE_RESTORE, though...
Flags: needinfo?(bugs)
Nice :/


Load certainly doesn't need 'Page', and I think unload could without it too.
Flags: needinfo?(bugs)
Comment on attachment 8654844 [details] [diff] [review]
part.39 Rename NS_POINTER_EVENT_START to ePointerEventFirst

>+NS_EVENT_MESSAGE(ePointerEventFirst,    4400)
I wonder if 4400 should be aligned with the ePointerEventFirst. But perhaps some other patch changes this stuff
Attachment #8654844 - Flags: review?(bugs) → review+
Attachment #8654845 - Flags: review?(bugs) → review+
Attachment #8654847 - Flags: review?(bugs) → review+
Attachment #8654850 - Flags: review?(bugs) → review+
Attachment #8654851 - Flags: review?(bugs) → review+
Attachment #8654852 - Flags: review?(bugs) → review+
Attachment #8654853 - Flags: review?(bugs) → review+
Attachment #8654854 - Flags: review?(bugs) → review+
Attachment #8654856 - Flags: review?(bugs) → review+
Attachment #8654857 - Flags: review?(bugs) → review+
Attachment #8654858 - Flags: review?(bugs) → review+
Attachment #8654859 - Flags: review?(bugs) → review+
Attachment #8654860 - Flags: review?(bugs) → review+
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/b3a11cf986072c52d4b00532febe1ec53fa4f22a
changeset:  b3a11cf986072c52d4b00532febe1ec53fa4f22a
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 01 00:20:05 2015 +0900
description:
Bug 895274 part.39 Rename NS_POINTER_EVENT_START to ePointerEventFirst and add ePointerEventLast r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/04ed6773489c51298603209fe7f3e0ff0c02368a
changeset:  04ed6773489c51298603209fe7f3e0ff0c02368a
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 01 00:20:05 2015 +0900
description:
Bug 895274 part.40 Rename NS_POINTER_LOST_CAPTURE to ePointerLostCapture r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/6fe4281b2f0bc35740e2399f01646b52ab3cd8f0
changeset:  6fe4281b2f0bc35740e2399f01646b52ab3cd8f0
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 01 00:20:05 2015 +0900
description:
Bug 895274 part.41 Rename NS_POINTER_GOT_CAPTURE to ePointerGotCapture r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/c0b1bff50484386fa3bcbb70e79f67f2748c69ee
changeset:  c0b1bff50484386fa3bcbb70e79f67f2748c69ee
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 01 00:20:05 2015 +0900
description:
Bug 895274 part.42 Rename NS_POINTER_CANCEL to ePointerCancel r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/81616c5e77d7893153138a33652694099827bc50
changeset:  81616c5e77d7893153138a33652694099827bc50
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 01 00:20:05 2015 +0900
description:
Bug 895274 part.43 Rename NS_POINTER_LEAVE to ePointerLeave r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/64973fea75722564451f73a524fc335d0f27d76e
changeset:  64973fea75722564451f73a524fc335d0f27d76e
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 01 00:20:05 2015 +0900
description:
Bug 895274 part.44 Rename NS_POINTER_ENTER to ePointerEnter r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/628fdef4a033fed83cd1bac4b1f1b6ea5b4017e1
changeset:  628fdef4a033fed83cd1bac4b1f1b6ea5b4017e1
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 01 00:20:05 2015 +0900
description:
Bug 895274 part.45 Rename NS_POINTER_OUT to ePointerOut r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/d7d3fe895b8dfb9176f560391b4934348fc1b630
changeset:  d7d3fe895b8dfb9176f560391b4934348fc1b630
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 01 00:20:05 2015 +0900
description:
Bug 895274 part.46 Rename NS_POINTER_OVER to ePointerOver r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/c66504b2b0b24846d0f9bd081ec6e36782237d7f
changeset:  c66504b2b0b24846d0f9bd081ec6e36782237d7f
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 01 00:20:06 2015 +0900
description:
Bug 895274 part.47 Rename NS_POINTER_DOWN to ePointerDown r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/b8957e880eba52047056b3e4d5f611fad08180e7
changeset:  b8957e880eba52047056b3e4d5f611fad08180e7
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 01 00:20:06 2015 +0900
description:
Bug 895274 part.48 Rename NS_POINTER_UP to ePointerUp r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/9a728655edf95ccfcd0c600eb381433bf620af9f
changeset:  9a728655edf95ccfcd0c600eb381433bf620af9f
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 01 00:20:06 2015 +0900
description:
Bug 895274 part.49 Rename NS_POINTER_MOVE to ePointerMove r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/44c0607b46208775584259b72146408b3ddc42ae
changeset:  44c0607b46208775584259b72146408b3ddc42ae
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 01 00:20:06 2015 +0900
description:
Bug 895274 part.50 Rename NS_CONTEXTMENU_MESSAGE_START to eContextMenuFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/3c16bb077e3173848498c17edb876ff2e8990e22
changeset:  3c16bb077e3173848498c17edb876ff2e8990e22
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 01 00:20:06 2015 +0900
description:
Bug 895274 part.51 Rename NS_CONTEXTMENU to eContextMenu r=smaug
(In reply to Olli Pettay [:smaug] (high review load, please consider other reviewers) from comment #109)
> Comment on attachment 8654844 [details] [diff] [review]
> part.39 Rename NS_POINTER_EVENT_START to ePointerEventFirst
> 
> >+NS_EVENT_MESSAGE(ePointerEventFirst,    4400)
> I wonder if 4400 should be aligned with the ePointerEventFirst. But perhaps
> some other patch changes this stuff

FYI: I'm checking every use case of the event message values. So, when we fix this bug, I think that we can remove specifying value to every event message.
Like this:

NS_EVENT_MESSAGE(ePointerMove)
NS_EVENT_MESSAGE(ePointerUp)
NS_EVENT_MESSAGE(ePointerDown)
NS_EVENT_MESSAGE(ePointerOver)
NS_EVENT_MESSAGE(ePointerOut)
NS_EVENT_MESSAGE(ePointerEnter)
NS_EVENT_MESSAGE(ePointerLeave)
NS_EVENT_MESSAGE(ePointerCancel)
NS_EVENT_MESSAGE(ePointerGotCapture)
NS_EVENT_MESSAGE(ePointerLostCapture)
NS_SPECIAL_EVENT_MESSAGE(ePointerEventLast,     ePointerMove)
NS_SPECIAL_EVENT_MESSAGE(ePointerEventLast,     ePointerLostCapture)
According to the DOM event name, this could be eError, but it's too generic. I prefer eLoadError instead.
Attachment #8654926 - Flags: review?(bugs)
According to the DOM event name, this could be eAbort. But it's too generic. Therefore, I prefer eImageAbort instead.
Attachment #8654928 - Flags: review?(bugs)
Could be eSubmit. But "Form" keyword is clearer. Therefore, I don't remove it from this event message. But it could be...
Attachment #8654935 - Flags: review?(bugs)
Same. And also "Selected" isn't good for consistency with other event message names. I like "Select" better.
Attachment #8654939 - Flags: review?(bugs)
Same, but eInvalid must be too generic.
Attachment #8654941 - Flags: review?(bugs)
I think that eFocus is enough rather than eFocusContent.
Attachment #8655393 - Flags: review?(bugs)
"UI" isn't clear to me. NS_UI_* define the event messages of legacy event which indicates an element state. So, I think that "Legacy" helps developers to understand what the events and how should they tread.

Therefore, I name this to eLegacyUIEventFirst. And each of the messages is:

* NS_UI_ACTIVATE -> eLegacyDOMActivate
* NS_UI_FOCUSIN ->  eLegacyDOMFocusIn
* NS_UI_FOCUSOUT -> eLegacyDOMFocusOut

So, the new names indicate which legacy DOM event explicitly.
Attachment #8655396 - Flags: review?(bugs)
I think that using DOM event name is better. So, I name this as eDragLeave, not eDragDropLeave.
Attachment #8655402 - Flags: review?(bugs)
Similar to NS_UI_*, let's use "Legacy" keyword for non-standard dnd events. This name must help developers what's different from the similar event, eDrop.
Attachment #8655413 - Flags: review?(bugs)
Same. This is a legacy event of eDragStart.
Attachment #8655414 - Flags: review?(bugs)
Attachment #8654917 - Flags: review?(bugs) → review+
Attachment #8654919 - Flags: review?(bugs) → review+
Attachment #8654921 - Flags: review?(bugs) → review+
Attachment #8654923 - Flags: review?(bugs) → review+
Attachment #8654925 - Flags: review?(bugs) → review+
Attachment #8654926 - Flags: review?(bugs) → review+
Attachment #8654928 - Flags: review?(bugs) → review+
Attachment #8654929 - Flags: review?(bugs) → review+
Attachment #8654930 - Flags: review?(bugs) → review+
Attachment #8654932 - Flags: review?(bugs) → review+
Attachment #8654934 - Flags: review?(bugs) → review+
Attachment #8654935 - Flags: review?(bugs) → review+
Attachment #8654936 - Flags: review?(bugs) → review+
Attachment #8654937 - Flags: review?(bugs) → review+
Attachment #8654939 - Flags: review?(bugs) → review+
Attachment #8654941 - Flags: review?(bugs) → review+
Attachment #8655392 - Flags: review?(bugs) → review+
Attachment #8655393 - Flags: review?(bugs) → review+
Attachment #8655395 - Flags: review?(bugs) → review+
Comment on attachment 8655396 [details] [diff] [review]
part.71 Rename NS_UI_EVENT_START to eLegacyUIEventFirst

Ah, I like this naming.
Attachment #8655396 - Flags: review?(bugs) → review+
Attachment #8655397 - Flags: review?(bugs) → review+
Attachment #8655398 - Flags: review?(bugs) → review+
Attachment #8655400 - Flags: review?(bugs) → review+
Attachment #8655401 - Flags: review?(bugs) → review+
Attachment #8655402 - Flags: review?(bugs) → review+
Attachment #8655403 - Flags: review?(bugs) → review+
Attachment #8655404 - Flags: review?(bugs) → review+
Attachment #8655405 - Flags: review?(bugs) → review+
Attachment #8655407 - Flags: review?(bugs) → review+
Attachment #8655409 - Flags: review?(bugs) → review+
Attachment #8655410 - Flags: review?(bugs) → review+
Attachment #8655412 - Flags: review?(bugs) → review+
Attachment #8655413 - Flags: review?(bugs) → review+
Attachment #8655414 - Flags: review?(bugs) → review+
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/196b7de875ef521f938904e0dceb62a6ba2fd897
changeset:  196b7de875ef521f938904e0dceb62a6ba2fd897
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:07:58 2015 +0900
description:
Bug 895274 part.52 Get rid of NS_PAGE_RESTORE due to unused r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/6966f9d6f70bfeaebdd4f08afb1f725a17949787
changeset:  6966f9d6f70bfeaebdd4f08afb1f725a17949787
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:07:58 2015 +0900
description:
Bug 895274 part.53 Rename NS_STREAM_EVENT_START to eStreamEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/305e40d37ee560305b95ba5e95a73d5b0b30a28b
changeset:  305e40d37ee560305b95ba5e95a73d5b0b30a28b
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:07:58 2015 +0900
description:
Bug 895274 part.54 Rename NS_READYSTATECHANGE to eReadyStateChange r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/3bfcdb7bfe8d137cd7cba3dcdbb98d46328dcaca
changeset:  3bfcdb7bfe8d137cd7cba3dcdbb98d46328dcaca
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:07:59 2015 +0900
description:
Bug 895274 part.55 Rename NS_BEFORE_PAGE_UNLOAD to eBeforeUnload r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/44d54a262d71f16b578bdee228b7854b77a75b11
changeset:  44d54a262d71f16b578bdee228b7854b77a75b11
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:07:59 2015 +0900
description:
Bug 895274 part.56 Rename NS_POPSTATE to ePopState r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/227bdcb2316da1bb496246ca4f8034d8e4d078e3
changeset:  227bdcb2316da1bb496246ca4f8034d8e4d078e3
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:07:59 2015 +0900
description:
Bug 895274 part.57 Rename NS_LOAD_ERROR to eLoadError r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/347f30d4b57b21604f9df04fe09f09c45f66e657
changeset:  347f30d4b57b21604f9df04fe09f09c45f66e657
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:07:59 2015 +0900
description:
Bug 895274 part.58 Rename NS_IMAGE_ABORT to eImageAbort r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/0a0c6edeaeab51cf022b2d023f52c53fda196892
changeset:  0a0c6edeaeab51cf022b2d023f52c53fda196892
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:07:59 2015 +0900
description:
Bug 895274 part.59 Rename NS_HASHCHANGE to eHashChange r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/280ce6c3a4eeb770861c5a8a7c6721a3d384e65f
changeset:  280ce6c3a4eeb770861c5a8a7c6721a3d384e65f
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:07:59 2015 +0900
description:
Bug 895274 part.60 Rename NS_PAGE_UNLOAD to eUnload r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/c3d179d89c2c14b77643b7911fd5600c8af61f41
changeset:  c3d179d89c2c14b77643b7911fd5600c8af61f41
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:07:59 2015 +0900
description:
Bug 895274 part.61 Rename NS_LOAD to eLoad r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/9afdaf8b032a01c5b9fc437dbde4db57f767bc34
changeset:  9afdaf8b032a01c5b9fc437dbde4db57f767bc34
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:07:59 2015 +0900
description:
Bug 895274 part.62 Rename NS_FORM_EVENT_START to eFormEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/5443202e95e681f179b5b0a86cc4aa00164452b1
changeset:  5443202e95e681f179b5b0a86cc4aa00164452b1
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:00 2015 +0900
description:
Bug 895274 part.63 Rename NS_FORM_SUBMIT to eFormSubmit r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/58ea57abc0e9736acc0e3c7fbf1a65d92b502edc
changeset:  58ea57abc0e9736acc0e3c7fbf1a65d92b502edc
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:00 2015 +0900
description:
Bug 895274 part.64 Rename NS_FORM_RESET to eFormReset r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/a4db59edaa15f10f93bbb372cf53224c1ecc6482
changeset:  a4db59edaa15f10f93bbb372cf53224c1ecc6482
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:00 2015 +0900
description:
Bug 895274 part.65 Rename NS_FORM_CHANGE to eFormChange r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/932f80808057e731b911c0a8cfd218541a03d324
changeset:  932f80808057e731b911c0a8cfd218541a03d324
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:00 2015 +0900
description:
Bug 895274 part.66 Rename NS_FORM_SELECTED to eFormSelect r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/5653edb7c0f71dfd7444ba15e5f09e4182076f9b
changeset:  5653edb7c0f71dfd7444ba15e5f09e4182076f9b
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:00 2015 +0900
description:
Bug 895274 part.67 Rename NS_FORM_INVALID to eFormInvalid r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/5f67d5de5adb1a9b31a945cbbe91db3e620a4e57
changeset:  5f67d5de5adb1a9b31a945cbbe91db3e620a4e57
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:00 2015 +0900
description:
Bug 895274 part.68 Rename NS_FOCUS_EVENT_START to eFocusEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/e9bb1fc9977aa1fb3df612fcd4954172bad8de41
changeset:  e9bb1fc9977aa1fb3df612fcd4954172bad8de41
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:00 2015 +0900
description:
Bug 895274 part.69 Rename NS_FOCUS_CONTENT to eFocus r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/89d8ce319bd0bff6a553ec382a0669abf79de8d9
changeset:  89d8ce319bd0bff6a553ec382a0669abf79de8d9
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:00 2015 +0900
description:
Bug 895274 part.70 Rename NS_BLUR_CONTENT to eBlur r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/4682f4db642d462d74c8afa0102373e191f47edd
changeset:  4682f4db642d462d74c8afa0102373e191f47edd
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:01 2015 +0900
description:
Bug 895274 part.71 Rename NS_UI_EVENT_START to eLegacyUIEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/cf4c3be6ea5689cf5cedf253b1bfd27290d2ba25
changeset:  cf4c3be6ea5689cf5cedf253b1bfd27290d2ba25
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:01 2015 +0900
description:
Bug 895274 part.72 Rename NS_UI_ACTIVATE to eLegacyDOMActivate r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/deb5c3173e6e250766cb28957dd919fd5e38dc3e
changeset:  deb5c3173e6e250766cb28957dd919fd5e38dc3e
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:01 2015 +0900
description:
Bug 895274 part.73 Rename NS_UI_FOCUSIN to eLegacyDOMFocusIn r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/be746a34f096038983737ea2f143bfc7daec1004
changeset:  be746a34f096038983737ea2f143bfc7daec1004
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:01 2015 +0900
description:
Bug 895274 part.74 Rename NS_UI_FOCUSOUT to eLegacyDOMFocusOut r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/38c50d1c496bcb1d3436ea34cfe7863a8e0004de
changeset:  38c50d1c496bcb1d3436ea34cfe7863a8e0004de
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:01 2015 +0900
description:
Bug 895274 part.75 Rename NS_DRAGDROP_EVENT_START to eDragDropEventFirst, define eDragDropEventLast and correct the type and name of event message in DataTransfer r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/76a8bd3c6ea34a2eccadf4a47d0eb6818e66aecc
changeset:  76a8bd3c6ea34a2eccadf4a47d0eb6818e66aecc
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:01 2015 +0900
description:
Bug 895274 part.76 Rename NS_DRAGDROP_LEAVE to eDragLeave r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/df96cb04b9dd41d97951a8cc6e68d7c332616c08
changeset:  df96cb04b9dd41d97951a8cc6e68d7c332616c08
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:01 2015 +0900
description:
Bug 895274 part.77 Rename NS_DRAGDROP_DROP to eDrop r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/2eb6dbb32ddc41d166123699cbc750d704cd120a
changeset:  2eb6dbb32ddc41d166123699cbc750d704cd120a
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:01 2015 +0900
description:
Bug 895274 part.78 Rename NS_DRAGDROP_START to eDragStart r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/99bfe44528f6dc6041a997710cdc49a5a53f13e7
changeset:  99bfe44528f6dc6041a997710cdc49a5a53f13e7
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:02 2015 +0900
description:
Bug 895274 part.79 Rename NS_DRAGDROP_END to eDragEnd r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/42d861b4a03568fa6bf3d4c45f6f089d04453f31
changeset:  42d861b4a03568fa6bf3d4c45f6f089d04453f31
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:02 2015 +0900
description:
Bug 895274 part.80 Rename NS_DRAGDROP_DRAG to eDrag r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/4f79fccb269836a8053a036c8a0734d618d92b41
changeset:  4f79fccb269836a8053a036c8a0734d618d92b41
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:02 2015 +0900
description:
Bug 895274 part.81 Rename NS_DRAGDROP_OVER to eDragOver r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/cad395df60d302e74ba82c7f46626180eb885d13
changeset:  cad395df60d302e74ba82c7f46626180eb885d13
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:02 2015 +0900
description:
Bug 895274 part.82 Rename NS_DRAGDROP_ENTER to eDragEnter r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/aaffbbb943e446e995c9f08c363bae24e0712a28
changeset:  aaffbbb943e446e995c9f08c363bae24e0712a28
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:02 2015 +0900
description:
Bug 895274 part.83 Rename NS_DRAGDROP_EXIT to eDragExit r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/3a188aef7f84eac26f6401ee337dd7c4d37b3f90
changeset:  3a188aef7f84eac26f6401ee337dd7c4d37b3f90
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:03 2015 +0900
description:
Bug 895274 part.84 Rename NS_DRAGDROP_DRAGDROP to eLegacyDragDrop r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/bd0d6c41b33ec106579e405ee660e150c0cac513
changeset:  bd0d6c41b33ec106579e405ee660e150c0cac513
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Wed Sep 02 15:08:03 2015 +0900
description:
Bug 895274 part.85 Rename NS_DRAGDROP_GESTURE to eLegacyDragGesture r=smaug
https://hg.mozilla.org/mozilla-central/rev/196b7de875ef
https://hg.mozilla.org/mozilla-central/rev/6966f9d6f70b
https://hg.mozilla.org/mozilla-central/rev/305e40d37ee5
https://hg.mozilla.org/mozilla-central/rev/3bfcdb7bfe8d
https://hg.mozilla.org/mozilla-central/rev/44d54a262d71
https://hg.mozilla.org/mozilla-central/rev/227bdcb2316d
https://hg.mozilla.org/mozilla-central/rev/347f30d4b57b
https://hg.mozilla.org/mozilla-central/rev/0a0c6edeaeab
https://hg.mozilla.org/mozilla-central/rev/280ce6c3a4ee
https://hg.mozilla.org/mozilla-central/rev/c3d179d89c2c
https://hg.mozilla.org/mozilla-central/rev/9afdaf8b032a
https://hg.mozilla.org/mozilla-central/rev/5443202e95e6
https://hg.mozilla.org/mozilla-central/rev/58ea57abc0e9
https://hg.mozilla.org/mozilla-central/rev/a4db59edaa15
https://hg.mozilla.org/mozilla-central/rev/932f80808057
https://hg.mozilla.org/mozilla-central/rev/5653edb7c0f7
https://hg.mozilla.org/mozilla-central/rev/5f67d5de5adb
https://hg.mozilla.org/mozilla-central/rev/e9bb1fc9977a
https://hg.mozilla.org/mozilla-central/rev/89d8ce319bd0
https://hg.mozilla.org/mozilla-central/rev/4682f4db642d
https://hg.mozilla.org/mozilla-central/rev/cf4c3be6ea56
https://hg.mozilla.org/mozilla-central/rev/deb5c3173e6e
https://hg.mozilla.org/mozilla-central/rev/be746a34f096
https://hg.mozilla.org/mozilla-central/rev/38c50d1c496b
https://hg.mozilla.org/mozilla-central/rev/76a8bd3c6ea3
https://hg.mozilla.org/mozilla-central/rev/df96cb04b9dd
https://hg.mozilla.org/mozilla-central/rev/2eb6dbb32ddc
https://hg.mozilla.org/mozilla-central/rev/99bfe44528f6
https://hg.mozilla.org/mozilla-central/rev/42d861b4a035
https://hg.mozilla.org/mozilla-central/rev/4f79fccb2698
https://hg.mozilla.org/mozilla-central/rev/cad395df60d3
https://hg.mozilla.org/mozilla-central/rev/aaffbbb943e4
https://hg.mozilla.org/mozilla-central/rev/3a188aef7f84
https://hg.mozilla.org/mozilla-central/rev/bd0d6c41b33e
When you have much time, could you check the patches? I don't like to ask review for the others since the new names should be decided by same person.
Flags: needinfo?(bugs)
Comment on attachment 8656604 [details] [diff] [review]
part.86 Rename NS_XUL_EVENT_START to eXULEventFirst

This is trivial stuff
Flags: needinfo?(bugs)
Attachment #8656604 - Flags: review+
I don't understand why this was separated from NS_XUL_*...
Comment on attachment 8656608 [details] [diff] [review]
part.90 Rename NS_XUL_POPUP_HIDDEN to eXULPopupHidden

Oh, I missed these.
Flags: needinfo?(bugs)
Attachment #8656608 - Flags: review+
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/e109ec7a6df6621a08bfe34e6e6c4e159bf2c8d4
changeset:  e109ec7a6df6621a08bfe34e6e6c4e159bf2c8d4
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 04 19:54:11 2015 +0900
description:
Bug 895274 part.86 Rename NS_XUL_EVENT_START to eXULEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/c24765571791120685b21e7f5b8bc7dc625a1eae
changeset:  c24765571791120685b21e7f5b8bc7dc625a1eae
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 04 19:54:11 2015 +0900
description:
Bug 895274 part.87 Rename NS_XUL_POPUP_SHOWING to eXULPopupShowing r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/97e0ba98dd52703a0040a4883c626fa5170f4785
changeset:  97e0ba98dd52703a0040a4883c626fa5170f4785
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 04 19:54:11 2015 +0900
description:
Bug 895274 part.88 Rename NS_XUL_POPUP_SHOWN to eXULPopupShown r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/7a48f1bb8b42a984c1ccf81dc2eb5b93583ed221
changeset:  7a48f1bb8b42a984c1ccf81dc2eb5b93583ed221
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 04 19:54:11 2015 +0900
description:
Bug 895274 part.89 Rename NS_XUL_POPUP_HIDING to eXULPopupHiding r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/decc683d234c431ffe003ccbbade26cf312563ff
changeset:  decc683d234c431ffe003ccbbade26cf312563ff
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 04 19:54:11 2015 +0900
description:
Bug 895274 part.90 Rename NS_XUL_POPUP_HIDDEN to eXULPopupHidden r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/8f9d85e978cac16a296c52cc1dd050e2fbdc3f86
changeset:  8f9d85e978cac16a296c52cc1dd050e2fbdc3f86
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 04 19:54:12 2015 +0900
description:
Bug 895274 part.91 Rename NS_XUL_BROADCAST to eXULBroadcast r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/7fd80515c0fecca6849e56ca1e392df06e8f0c7f
changeset:  7fd80515c0fecca6849e56ca1e392df06e8f0c7f
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 04 19:54:12 2015 +0900
description:
Bug 895274 part.92 Rename NS_XUL_COMMAND_UPDATE to eXULCommandUpdate r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/25c5f2f3ee8a1e011d985921bb2f4d60595892f4
changeset:  25c5f2f3ee8a1e011d985921bb2f4d60595892f4
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 04 19:54:12 2015 +0900
description:
Bug 895274 part.93 Rename NS_XULCOMMAND_EVENT_START to eXULCommandEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/0720ec8893664f4dd3a48f56f0b92e99ed3a4fdb
changeset:  0720ec8893664f4dd3a48f56f0b92e99ed3a4fdb
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 04 19:54:12 2015 +0900
description:
Bug 895274 part.94 Rename NS_XUL_COMMAND to eXULCommand r=smaug
I think that mutation events should have "legacy" keyword.
Attachment #8657744 - Flags: review?(bugs)
Comment on attachment 8657744 [details] [diff] [review]
part.95 Rename NS_MUTATION_NODEINSERTEDINTODOCUMENT to eLegacyNodeInsertedIntoDocument

(we should just get rid of this event message at some point)
Attachment #8657744 - Flags: review?(bugs) → review+
Attachment #8657745 - Flags: review?(bugs) → review+
Attachment #8657746 - Flags: review?(bugs) → review+
Attachment #8657747 - Flags: review?(bugs) → review+
Attachment #8657748 - Flags: review?(bugs) → review+
Attachment #8657749 - Flags: review?(bugs) → review+
Attachment #8657750 - Flags: review?(bugs) → review+
Attachment #8657751 - Flags: review?(bugs) → review+
Attachment #8657752 - Flags: review?(bugs) → review+
Attachment #8657824 - Flags: review?(bugs) → review+
Attachment #8657825 - Flags: review?(bugs) → review+
Attachment #8657826 - Flags: review?(bugs) → review+
Hmm, they are defined in the spec, but we don't register them in EventNameList.h.
Attachment #8657830 - Flags: review?(bugs)
A lot of SVG events are only registered in EventNameList.h. We don't fire the events? Or do we have a way to fire the simple events without creating Widget*Event in the stack?
Attachment #8657834 - Flags: review?(bugs)
Attachment #8657830 - Flags: review?(bugs) → review+
Attachment #8657831 - Flags: review?(bugs) → review+
Attachment #8657832 - Flags: review?(bugs) → review+
Attachment #8657834 - Flags: review?(bugs) → review+
Attachment #8657835 - Flags: review?(bugs) → review+
Attachment #8657836 - Flags: review?(bugs) → review+
Attachment #8657837 - Flags: review?(bugs) → review+
Attachment #8657838 - Flags: review?(bugs) → review+
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/460f5affc668d32c3227aca44a0b55e761dec6eb
changeset:  460f5affc668d32c3227aca44a0b55e761dec6eb
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:50 2015 +0900
description:
Bug 895274 part.95 Rename NS_MUTATION_NODEINSERTEDINTODOCUMENT to eLegacyNodeInsertedIntoDocument r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/53fbc3307221c5635d3a6c23631c0380645c203f
changeset:  53fbc3307221c5635d3a6c23631c0380645c203f
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:50 2015 +0900
description:
Bug 895274 part.96 Rename NS_MUTATION_START to eLegacyMutationEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/2f8c25b844e6acc92d446b6b0576f31f1efb0e8b
changeset:  2f8c25b844e6acc92d446b6b0576f31f1efb0e8b
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:50 2015 +0900
description:
Bug 895274 part.97 Rename NS_MUTATION_NODEREMOVEDFROMDOCUMENT to eLegacyNodeRemovedFromDocument r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/79e293defe0fa1211255c62cf5cada0cbcd27e5f
changeset:  79e293defe0fa1211255c62cf5cada0cbcd27e5f
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:50 2015 +0900
description:
Bug 895274 part.98 Rename NS_MUTATION_CHARACTERDATAMODIFIED to eLegacyCharacterDataModified r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/f6f918d677aabe854314d81d65d02d07216dddae
changeset:  f6f918d677aabe854314d81d65d02d07216dddae
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:51 2015 +0900
description:
Bug 895274 part.99 Rename NS_MUTATION_SUBTREEMODIFIED to eLegacySubtreeModified r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/d643e9bbecbcc0e9a71752e6bb24c90572872b73
changeset:  d643e9bbecbcc0e9a71752e6bb24c90572872b73
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:51 2015 +0900
description:
Bug 895274 part.100 Rename NS_MUTATION_NODEINSERTED to eLegacyNodeInserted r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/50a074c6f1e11e0ec19205f97f138927119de8dd
changeset:  50a074c6f1e11e0ec19205f97f138927119de8dd
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:51 2015 +0900
description:
Bug 895274 part.101 Rename NS_MUTATION_NODEREMOVED to eLegacyNodeRemoved r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/ec25eda8de151b000251faa14b478822dade1d93
changeset:  ec25eda8de151b000251faa14b478822dade1d93
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:51 2015 +0900
description:
Bug 895274 part.102 Rename NS_MUTATION_ATTRMODIFIED to eLegacyAttrModified r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/82aa99137da495953561777280da9d4e41554776
changeset:  82aa99137da495953561777280da9d4e41554776
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:51 2015 +0900
description:
Bug 895274 part.103 Rename NS_MUTATION_END to eLegacyMutationEventLast r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/ca2ed2eb12c063168835797e989d8a2bab39fe04
changeset:  ca2ed2eb12c063168835797e989d8a2bab39fe04
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:51 2015 +0900
description:
Bug 895274 part.104 Rename NS_PAGETRANSITION_START to ePageTransitionEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/396816b81be5c18f02691205e5babf70b35927d8
changeset:  396816b81be5c18f02691205e5babf70b35927d8
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:51 2015 +0900
description:
Bug 895274 part.105 Rename NS_PAGE_SHOW to ePageShow r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/182b27c4a45848afe761e7e2e8091abeb771be3e
changeset:  182b27c4a45848afe761e7e2e8091abeb771be3e
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:51 2015 +0900
description:
Bug 895274 part.106 Rename NS_PAGE_HIDE to ePageHide r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/573aca3165b2f6c7f2f168cc37c8b42198d6b422
changeset:  573aca3165b2f6c7f2f168cc37c8b42198d6b422
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:51 2015 +0900
description:
Bug 895274 part.107 Get rid of NS_SVG_ABORT and NS_SVG_ERROR due to unused r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/9472b6dbaf255c9116f6d4b37e932dac76c6f909
changeset:  9472b6dbaf255c9116f6d4b37e932dac76c6f909
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:51 2015 +0900
description:
Bug 895274 part.108 Rename NS_SVG_EVENT_START to eSVGEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/080f2acf959d6ae5a56db596ee0f69dbc94915cc
changeset:  080f2acf959d6ae5a56db596ee0f69dbc94915cc
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:51 2015 +0900
description:
Bug 895274 part.109 Rename NS_SVG_LOAD to eSVGLoad r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/549e7a38a199f096d2ca313e3417226dc92f7d9b
changeset:  549e7a38a199f096d2ca313e3417226dc92f7d9b
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:51 2015 +0900
description:
Bug 895274 part.110 Rename NS_SVG_UNLOAD to eSVGUnload r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/b33b114ec6d4d8e6b9c4557f45d50efe130d0f8d
changeset:  b33b114ec6d4d8e6b9c4557f45d50efe130d0f8d
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:52 2015 +0900
description:
Bug 895274 part.111 Rename NS_SVG_RESIZE to eSVGResize r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/9b40d2b9c5f1bb0b6ce536ebf56d7a2d9b9206fe
changeset:  9b40d2b9c5f1bb0b6ce536ebf56d7a2d9b9206fe
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:52 2015 +0900
description:
Bug 895274 part.112 Rename NS_SVG_SCROLL to eSVGScroll r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/92668c18d13158c4bc2d6e07a218dc556ea994f6
changeset:  92668c18d13158c4bc2d6e07a218dc556ea994f6
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:52 2015 +0900
description:
Bug 895274 part.113 Rename NS_SVGZOOM_EVENT_START to eSVGZoomEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/3f2f4df587af31ad56641e067e1864ad57c61d10
changeset:  3f2f4df587af31ad56641e067e1864ad57c61d10
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Mon Sep 07 23:55:52 2015 +0900
description:
Bug 895274 part.114 Rename NS_SVG_ZOOM to eSVGZoom r=smaug
Attachment #8658064 - Attachment description: part.144 NS_Rename NS_SELECTION_EVENT_START to eSelectionEventFirst → part.144 Rename NS_SELECTION_EVENT_START to eSelectionEventFirst
Attachment #8658031 - Flags: review?(bugs) → review+
Attachment #8658030 - Flags: review?(bugs) → review+
Attachment #8658027 - Flags: review?(bugs) → review+
Attachment #8658025 - Flags: review?(bugs) → review+
Attachment #8658024 - Flags: review?(bugs) → review+
Attachment #8658032 - Flags: review?(bugs) → review+
Attachment #8658033 - Flags: review?(bugs) → review+
Attachment #8658034 - Flags: review?(bugs) → review+
Attachment #8658035 - Flags: review?(bugs) → review+
Attachment #8658038 - Flags: review?(bugs) → review+
Attachment #8658039 - Flags: review?(bugs) → review+
Attachment #8658040 - Flags: review?(bugs) → review+
Attachment #8658041 - Flags: review?(bugs) → review+
Attachment #8658043 - Flags: review?(bugs) → review+
Attachment #8658044 - Flags: review?(bugs) → review+
Attachment #8658045 - Flags: review?(bugs) → review+
Attachment #8658048 - Flags: review?(bugs) → review+
Attachment #8658049 - Flags: review?(bugs) → review+
Attachment #8658050 - Flags: review?(bugs) → review+
Attachment #8658051 - Flags: review?(bugs) → review+
Attachment #8658052 - Flags: review?(bugs) → review+
Attachment #8658055 - Flags: review?(bugs) → review+
Attachment #8658056 - Flags: review?(bugs) → review+
Attachment #8658057 - Flags: review?(bugs) → review+
Attachment #8658058 - Flags: review?(bugs) → review+
Attachment #8658059 - Flags: review?(bugs) → review+
Attachment #8658060 - Flags: review?(bugs) → review+
Attachment #8658061 - Flags: review?(bugs) → review+
Attachment #8658063 - Flags: review?(bugs) → review+
Attachment #8658064 - Flags: review?(bugs) → review+
Comment on attachment 8658065 [details] [diff] [review]
part.145 Rename NS_SELECTION_SET to eSetSelection

Ah, this is about setSelection, not selectionSet
Attachment #8658065 - Flags: review?(bugs) → review+
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/cc988d9676a400f8a7cb0596e2738daf1f845bfc
changeset:  cc988d9676a400f8a7cb0596e2738daf1f845bfc
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:34 2015 +0900
description:
Bug 895274 part.115 Rename NS_CUTCOPYPASTE_EVENT to eClipboardEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/e248269468d485ddf48fe63e5e3ee955c74c03e0
changeset:  e248269468d485ddf48fe63e5e3ee955c74c03e0
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:34 2015 +0900
description:
Bug 895274 part.116 Rename NS_COPY to eCopy r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/a25fc31cd465e517a19870f311e7bfe733cae19a
changeset:  a25fc31cd465e517a19870f311e7bfe733cae19a
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:35 2015 +0900
description:
Bug 895274 part.117 Rename NS_CUT to eCut r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/2f15d2b271c76a22c8cb3a8696ded760caa23732
changeset:  2f15d2b271c76a22c8cb3a8696ded760caa23732
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:35 2015 +0900
description:
Bug 895274 part.118 Rename NS_PASTE to ePaste r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/096cf51d192007b304f27e8e8ceb6a1c3117371b
changeset:  096cf51d192007b304f27e8e8ceb6a1c3117371b
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:35 2015 +0900
description:
Bug 895274 part.119 Get rid of NS_NEED_KEY due to unused r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/8d2cc2f0fa0658bd285c8b3d582e98d906e3384b
changeset:  8d2cc2f0fa0658bd285c8b3d582e98d906e3384b
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:35 2015 +0900
description:
Bug 895274 part.120 Rename NS_MEDIA_EVENT_START to eMeditaEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/3ff229821a764ed555c8b66ffb1c2ba54055922b
changeset:  3ff229821a764ed555c8b66ffb1c2ba54055922b
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:35 2015 +0900
description:
Bug 895274 part.121 Rename NS_LOADSTART to eLoadStart r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/3f98fcec61b98041873521620a7232219339f46b
changeset:  3f98fcec61b98041873521620a7232219339f46b
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:35 2015 +0900
description:
Bug 895274 part.122 Rename NS_PROGRESS to eProgress r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/c510ffab1014c17f3173a3e9efcf15ba372265b3
changeset:  c510ffab1014c17f3173a3e9efcf15ba372265b3
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:35 2015 +0900
description:
Bug 895274 part.123 Rename NS_SUSPEND to eSuspend r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/94d303f30c15e9fa9c13cbc2a7cbfb58e70ace3d
changeset:  94d303f30c15e9fa9c13cbc2a7cbfb58e70ace3d
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:35 2015 +0900
description:
Bug 895274 part.124 Rename NS_EMPTIED to eEmptied r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/e208587b46fe983109ea1b82fe4f0bcd522f4ab1
changeset:  e208587b46fe983109ea1b82fe4f0bcd522f4ab1
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:36 2015 +0900
description:
Bug 895274 part.125 Rename NS_STALED to eStalled r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/7ce3a3e55500afaf19147e667735b8ea12f5af9e
changeset:  7ce3a3e55500afaf19147e667735b8ea12f5af9e
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:36 2015 +0900
description:
Bug 895274 part.126 Rename NS_PLAY to ePlay r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/7345eb48f962707973447323a729cc44d67365b0
changeset:  7345eb48f962707973447323a729cc44d67365b0
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:36 2015 +0900
description:
Bug 895274 part.127 Rename NS_PAUSE to ePause r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/1f6ce21b418e097d91d5e01662cf55979caf5f9a
changeset:  1f6ce21b418e097d91d5e01662cf55979caf5f9a
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:36 2015 +0900
description:
Bug 895274 part.128 Rename NS_LOADEDMETADATA to eLoadedMetaData r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/b33ca0b66d59dbd9e018d1e8b76745235b8750a0
changeset:  b33ca0b66d59dbd9e018d1e8b76745235b8750a0
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:36 2015 +0900
description:
Bug 895274 part.129 Rename NS_LOADEDDATA to eLoadedData r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/301e51d22aff888f330584974367ddae78cdb30b
changeset:  301e51d22aff888f330584974367ddae78cdb30b
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:36 2015 +0900
description:
Bug 895274 part.130 Rename NS_WAITING to eWaiting r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/834502fe7f2f948a4e8887f19bc809ebe6c7fa9b
changeset:  834502fe7f2f948a4e8887f19bc809ebe6c7fa9b
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:36 2015 +0900
description:
Bug 895274 part.131 Rename NS_PLAYING to ePlaying r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/b96778df945a73ac46c25870525e4e8889d1dae2
changeset:  b96778df945a73ac46c25870525e4e8889d1dae2
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:36 2015 +0900
description:
Bug 895274 part.132 Rename NS_CANPLAY to eCanPlay r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/78245a0ea853f873e33cd80644187aa0a9888878
changeset:  78245a0ea853f873e33cd80644187aa0a9888878
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:36 2015 +0900
description:
Bug 895274 part.133 Rename NS_CANPLAYTHROUGH to eCanPlayThrough r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/d80ad0e1eb053db6644d7dfa149eedc9cceaafde
changeset:  d80ad0e1eb053db6644d7dfa149eedc9cceaafde
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:37 2015 +0900
description:
Bug 895274 part.134 Rename NS_SEEKING to eSeeking r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/f08ab30607efb9257ccd5b12cd01606b8dd7c8c5
changeset:  f08ab30607efb9257ccd5b12cd01606b8dd7c8c5
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:37 2015 +0900
description:
Bug 895274 part.135 Rename NS_SEEKED to eSeeked r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/da9a63509cdc2555c4ff1c4f138feca1ee622086
changeset:  da9a63509cdc2555c4ff1c4f138feca1ee622086
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:37 2015 +0900
description:
Bug 895274 part.136 Rename NS_TIMEUPDATE to eTimeUpdate r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/7dfa381f18367183202ea12363562b10f0a451ff
changeset:  7dfa381f18367183202ea12363562b10f0a451ff
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:37 2015 +0900
description:
Bug 895274 part.137 Rename NS_ENDED to eEnded r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/9127d9bfb36a836198b9a78b433137214aa94549
changeset:  9127d9bfb36a836198b9a78b433137214aa94549
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:37 2015 +0900
description:
Bug 895274 part.138 Rename NS_RATECHANGE to eRateChange r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/1537722345b068ef919ee13cf545be233b4647e7
changeset:  1537722345b068ef919ee13cf545be233b4647e7
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:37 2015 +0900
description:
Bug 895274 part.139 Rename NS_DURATIONCHANGE to eDurationChange r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/a501c28cb98bccdb0a1b379a368e35278ffe0325
changeset:  a501c28cb98bccdb0a1b379a368e35278ffe0325
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:37 2015 +0900
description:
Bug 895274 part.140 Rename NS_VOLUMECHANGE to eVolumeChange r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/25e2d9a9d1f7e4fbc59dc7864aa91b13fd7499e3
changeset:  25e2d9a9d1f7e4fbc59dc7864aa91b13fd7499e3
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:37 2015 +0900
description:
Bug 895274 part.141 Get rid of NS_PLUGIN_FOCUS_EVENT due to unused r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/02d003c047ea5f9c0bd09cf36fe206d020b5949e
changeset:  02d003c047ea5f9c0bd09cf36fe206d020b5949e
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:38 2015 +0900
description:
Bug 895274 part.142 Rename NS_PLUGIN_EVENT_START to ePluginEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/8851fdf3cae405a9bf578898724454537a59a71d
changeset:  8851fdf3cae405a9bf578898724454537a59a71d
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:38 2015 +0900
description:
Bug 895274 part.143 Rename NS_PLUGIN_INPUT_EVENT to ePluginInputEvent r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/a6a0afa7ed7617c487c4c9d94ce7b9ddfd57fb3d
changeset:  a6a0afa7ed7617c487c4c9d94ce7b9ddfd57fb3d
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:38 2015 +0900
description:
Bug 895274 part.144 Rename NS_SELECTION_EVENT_START to eSelectionEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/59783997dbd58d4e46a819e544bf94bf42e1c93f
changeset:  59783997dbd58d4e46a819e544bf94bf42e1c93f
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 08 23:33:38 2015 +0900
description:
Bug 895274 part.145 Rename NS_SELECTION_SET to eSetSelection r=smaug
https://hg.mozilla.org/mozilla-central/rev/cc988d9676a4
https://hg.mozilla.org/mozilla-central/rev/e248269468d4
https://hg.mozilla.org/mozilla-central/rev/a25fc31cd465
https://hg.mozilla.org/mozilla-central/rev/2f15d2b271c7
https://hg.mozilla.org/mozilla-central/rev/096cf51d1920
https://hg.mozilla.org/mozilla-central/rev/8d2cc2f0fa06
https://hg.mozilla.org/mozilla-central/rev/3ff229821a76
https://hg.mozilla.org/mozilla-central/rev/3f98fcec61b9
https://hg.mozilla.org/mozilla-central/rev/c510ffab1014
https://hg.mozilla.org/mozilla-central/rev/94d303f30c15
https://hg.mozilla.org/mozilla-central/rev/e208587b46fe
https://hg.mozilla.org/mozilla-central/rev/7ce3a3e55500
https://hg.mozilla.org/mozilla-central/rev/7345eb48f962
https://hg.mozilla.org/mozilla-central/rev/1f6ce21b418e
https://hg.mozilla.org/mozilla-central/rev/b33ca0b66d59
https://hg.mozilla.org/mozilla-central/rev/301e51d22aff
https://hg.mozilla.org/mozilla-central/rev/834502fe7f2f
https://hg.mozilla.org/mozilla-central/rev/b96778df945a
https://hg.mozilla.org/mozilla-central/rev/78245a0ea853
https://hg.mozilla.org/mozilla-central/rev/d80ad0e1eb05
https://hg.mozilla.org/mozilla-central/rev/f08ab30607ef
https://hg.mozilla.org/mozilla-central/rev/da9a63509cdc
https://hg.mozilla.org/mozilla-central/rev/7dfa381f1836
https://hg.mozilla.org/mozilla-central/rev/9127d9bfb36a
https://hg.mozilla.org/mozilla-central/rev/1537722345b0
https://hg.mozilla.org/mozilla-central/rev/a501c28cb98b
https://hg.mozilla.org/mozilla-central/rev/25e2d9a9d1f7
https://hg.mozilla.org/mozilla-central/rev/02d003c047ea
https://hg.mozilla.org/mozilla-central/rev/8851fdf3cae4
https://hg.mozilla.org/mozilla-central/rev/a6a0afa7ed76
https://hg.mozilla.org/mozilla-central/rev/59783997dbd5
The new name, eLegacyMouseLineOrPageScroll, should be clearer what the event represents. "LineOrPage" is same as "WidgetWheelEvent::mLineOrPageDelta(X|Y)".
Attachment #8658724 - Flags: review?(bugs)
According to KeyboardEvent.key, "unidentified" is better word for DOM Events. I don't like "user defined" due to unclear what it means.
Attachment #8658728 - Flags: review?(bugs)
Attachment #8658721 - Flags: review?(bugs) → review+
Attachment #8658724 - Flags: review?(bugs) → review+
Comment on attachment 8658725 [details] [diff] [review]
part.148 Rename NS_MOUSE_PIXEL_SCROLL to eLegacyMousePixelScroll

s/eLegacyMousePiexelScroll/eLegacyMousePixelScroll/
everywhere in the patch.
With that, r+
Attachment #8658725 - Flags: review?(bugs) → review+
Comment on attachment 8658728 [details] [diff] [review]
part.149 Rename NS_USER_DEFINED_EVENT to eUnidentifiedEvent

I guess I can live with eUnidentifiedEvent.
Attachment #8658728 - Flags: review?(bugs) → review+
Attachment #8658729 - Flags: review?(bugs) → review+
Attachment #8658730 - Flags: review?(bugs) → review+
Comment on attachment 8658731 [details] [diff] [review]
part.152 Rename NS_QUERY_SELECTED_TEXT to eQuerySelectedText

> nsQueryContentEventResult::GetSucceeded(bool *aSucceeded)
> {
>-  NS_ENSURE_TRUE(mEventID != 0, NS_ERROR_NOT_INITIALIZED);
>+  NS_ENSURE_TRUE(mEventMessage != 0, NS_ERROR_NOT_INITIALIZED);
Don't you want to compare to eVoidEvent here.
Attachment #8658731 - Flags: review?(bugs) → review+
Attachment #8658732 - Flags: review?(bugs) → review+
Attachment #8658734 - Flags: review?(bugs) → review+
Attachment #8658736 - Flags: review?(bugs) → review+
(In reply to Olli Pettay [:smaug] from comment #246)
> Comment on attachment 8658725 [details] [diff] [review]
> part.148 Rename NS_MOUSE_PIXEL_SCROLL to eLegacyMousePixelScroll
> 
> s/eLegacyMousePiexelScroll/eLegacyMousePixelScroll/
> everywhere in the patch.
> With that, r+

Oops!

(In reply to Olli Pettay [:smaug] from comment #248)
> Comment on attachment 8658731 [details] [diff] [review]
> part.152 Rename NS_QUERY_SELECTED_TEXT to eQuerySelectedText
> 
> > nsQueryContentEventResult::GetSucceeded(bool *aSucceeded)
> > {
> >-  NS_ENSURE_TRUE(mEventID != 0, NS_ERROR_NOT_INITIALIZED);
> >+  NS_ENSURE_TRUE(mEventMessage != 0, NS_ERROR_NOT_INITIALIZED);
> Don't you want to compare to eVoidEvent here.

Right. Thank you for finding the mistake!
Attachment #8658737 - Flags: review?(bugs) → review+
Comment on attachment 8658738 [details] [diff] [review]
part.158 Rename NS_QUERY_DOM_WIDGET_HITTEST to eQueryDOMWidgetHittest

Not about this bug or patch, but I noticed
return "Unsupprted message"; 

Should be
return "Unsupported message";
Attachment #8658738 - Flags: review?(bugs) → review+
Attachment #8658739 - Flags: review?(bugs) → review+
Attachment #8658740 - Flags: review?(bugs) → review+
Attachment #8658735 - Flags: review?(bugs) → review+
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/6e4fc82f286ba247a31c7e9160aa73668e4efa36
changeset:  6e4fc82f286ba247a31c7e9160aa73668e4efa36
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Thu Sep 10 10:40:04 2015 +0900
description:
Bug 895274 part.146 Rename NS_MOUSE_SCROLL_START to eLegacyMouseScrollEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/c46079d8b6ed24f7533f1844873b8f18bdffc13c
changeset:  c46079d8b6ed24f7533f1844873b8f18bdffc13c
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Thu Sep 10 10:40:04 2015 +0900
description:
Bug 895274 part.147 Rename NS_MOUSE_SCROLL to eLegacyMouseLineOrPageScroll r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/7f6201ff997f763e3e6aff0689b742c043244a39
changeset:  7f6201ff997f763e3e6aff0689b742c043244a39
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Thu Sep 10 10:40:04 2015 +0900
description:
Bug 895274 part.148 Rename NS_MOUSE_PIXEL_SCROLL to eLegacyMousePixelScroll r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/4cdae41b9305f75ac82226a52f860ecd512c547c
changeset:  4cdae41b9305f75ac82226a52f860ecd512c547c
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Thu Sep 10 10:40:04 2015 +0900
description:
Bug 895274 part.149 Rename NS_USER_DEFINED_EVENT to eUnidentifiedEvent r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/4557cfecc20d65bc48fe43b6460038473fd4ed7d
changeset:  4557cfecc20d65bc48fe43b6460038473fd4ed7d
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Thu Sep 10 10:40:05 2015 +0900
description:
Bug 895274 part.150 Rename NS_QUERY_CONTENT_EVENT_START to eQueryContentEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/aaf1002beb949b424841d3fef3c731b25820f600
changeset:  aaf1002beb949b424841d3fef3c731b25820f600
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Thu Sep 10 10:40:05 2015 +0900
description:
Bug 895274 part.151 Rename NS_QUERY_SELECTION_AS_TRANSFERABLE to eQuerySelectionAsTransferable r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/ec614033e8225b089920f84d43d5930a2bef93c9
changeset:  ec614033e8225b089920f84d43d5930a2bef93c9
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Thu Sep 10 10:40:05 2015 +0900
description:
Bug 895274 part.152 Rename NS_QUERY_SELECTED_TEXT to eQuerySelectedText r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/25399075c9efe898cb2b2f137e4affeace30cd47
changeset:  25399075c9efe898cb2b2f137e4affeace30cd47
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Thu Sep 10 10:40:05 2015 +0900
description:
Bug 895274 part.153 Rename NS_QUERY_TEXT_CONTENT to eQueryTextContent r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/1a346d89cc2358675297e2014aa1e56146e71ae0
changeset:  1a346d89cc2358675297e2014aa1e56146e71ae0
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Thu Sep 10 10:40:06 2015 +0900
description:
Bug 895274 part.154 Rename NS_QUERY_CARET_RECT to eQueryCaretRect r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/a5c0e8cbf6073fe223f50ce2d8056678f69c88d5
changeset:  a5c0e8cbf6073fe223f50ce2d8056678f69c88d5
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Thu Sep 10 10:40:06 2015 +0900
description:
Bug 895274 part.155 Rename NS_QUERY_EDITOR_RECT to eQueryEditorRect r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/03965478482802c244ba8212177324cb7516d958
changeset:  03965478482802c244ba8212177324cb7516d958
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Thu Sep 10 10:40:06 2015 +0900
description:
Bug 895274 part.156 Rename NS_QUERY_CONTENT_STATE to eQueryContentState r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/2f4d6b2be0e628997a242f62b3fa93711333fba2
changeset:  2f4d6b2be0e628997a242f62b3fa93711333fba2
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Thu Sep 10 10:40:06 2015 +0900
description:
Bug 895274 part.157 Rename NS_QUERY_CHARACTER_AT_POINT to eQueryCharacterAtPoint r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/83512fe0f05403318cac6cdecf09e54bcd1077b8
changeset:  83512fe0f05403318cac6cdecf09e54bcd1077b8
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Thu Sep 10 10:40:06 2015 +0900
description:
Bug 895274 part.158 Rename NS_QUERY_DOM_WIDGET_HITTEST to eQueryDOMWidgetHittest r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/260d7fa5913db2f8f305f70a1df2bab24808cbed
changeset:  260d7fa5913db2f8f305f70a1df2bab24808cbed
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Thu Sep 10 10:40:06 2015 +0900
description:
Bug 895274 part.159 Rename NS_NOTIFYPAINT_START to ePaintEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/672a3f59ed55d59cf71dbfa2610f5d47adc8cf84
changeset:  672a3f59ed55d59cf71dbfa2610f5d47adc8cf84
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Thu Sep 10 10:40:06 2015 +0900
description:
Bug 895274 part.160 Rename NS_AFTERPAINT to eAfterPaint r=smaug
The name may not be good since NS_CLOSE has never been used since it's defined... However, let's use this name for now. We should remove unnecessary first and last values when we stop specifying the value of each event message.
Attachment #8659145 - Flags: review?(bugs)
It seems that this name is too generic, but I don't have better idea.
Attachment #8659147 - Flags: review?(bugs)
The spec uses "fullscreen", not "full screen". Therefore, I use "Fullscreen" in their names.
Attachment #8659151 - Flags: review?(bugs)
This event is fired on Mac when user touches the trackpad with two fingers. This won't cause DOM events therefore, I added "Operation" into the name for making what's started clearer.
Attachment #8659156 - Flags: review?(bugs)
Same. And "End" is better word for event staff (e.g., compositionend).
Attachment #8659157 - Flags: review?(bugs)
Attachment #8659123 - Flags: review?(bugs) → review+
Attachment #8659126 - Flags: review?(bugs) → review+
Attachment #8659128 - Flags: review?(bugs) → review+
Attachment #8659129 - Flags: review?(bugs) → review+
Attachment #8659130 - Flags: review?(bugs) → review+
Attachment #8659131 - Flags: review?(bugs) → review+
Attachment #8659132 - Flags: review?(bugs) → review+
Attachment #8659133 - Flags: review?(bugs) → review+
Attachment #8659134 - Flags: review?(bugs) → review+
Attachment #8659135 - Flags: review?(bugs) → review+
Attachment #8659137 - Flags: review?(bugs) → review+
Attachment #8659138 - Flags: review?(bugs) → review+
Attachment #8659139 - Flags: review?(bugs) → review+
Attachment #8659140 - Flags: review?(bugs) → review+
Attachment #8659141 - Flags: review?(bugs) → review+
Attachment #8659136 - Flags: review?(bugs) → review+
Attachment #8659145 - Flags: review?(bugs) → review+
Attachment #8659147 - Flags: review?(bugs) → review+
Attachment #8659149 - Flags: review?(bugs) → review+
Attachment #8659151 - Flags: review?(bugs) → review+
Attachment #8659152 - Flags: review?(bugs) → review+
Attachment #8659153 - Flags: review?(bugs) → review+
Attachment #8659154 - Flags: review?(bugs) → review+
Attachment #8659155 - Flags: review?(bugs) → review+
Attachment #8659156 - Flags: review?(bugs) → review+
Attachment #8659157 - Flags: review?(bugs) → review+
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/e59cc10fc94562a1859fc86f61b3c16523045450
changeset:  e59cc10fc94562a1859fc86f61b3c16523045450
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:51 2015 +0900
description:
Bug 895274 part.161 Rename NS_CONTENT_COMMAND_EVENT_START to eContentCommandEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/667d75050a9411dba70607df4437cba134753501
changeset:  667d75050a9411dba70607df4437cba134753501
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:52 2015 +0900
description:
Bug 895274 part.162 Rename NS_CONTENT_COMMAND_PASTE_TRANSFERABLE to eContentCommandPasteTransferable r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/54ac42eda76f6f66400846dd5a32e92b827ff389
changeset:  54ac42eda76f6f66400846dd5a32e92b827ff389
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:52 2015 +0900
description:
Bug 895274 part.163 Rename NS_CONTENT_COMMAND_CUT to eContentCommandCut r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/1c4f895797d85ea7853977809150c656ac9d807b
changeset:  1c4f895797d85ea7853977809150c656ac9d807b
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:52 2015 +0900
description:
Bug 895274 part.164 Rename NS_CONTENT_COMMAND_COPY to eContentCommandCopy r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/4bc7de51bb6ef20f52ac77b6d58d3df281e9e8db
changeset:  4bc7de51bb6ef20f52ac77b6d58d3df281e9e8db
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:52 2015 +0900
description:
Bug 895274 part.165 Rename NS_CONTENT_COMMAND_PASTE to eContentCommandPaste r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/fc17970a9de262e7408326cfc5ce8f26b936bbfc
changeset:  fc17970a9de262e7408326cfc5ce8f26b936bbfc
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:52 2015 +0900
description:
Bug 895274 part.166 Rename NS_CONTENT_COMMAND_DELETE to eContentCommandDelete r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/5863be1170c9bffe100db5bc150cba567a4f97e7
changeset:  5863be1170c9bffe100db5bc150cba567a4f97e7
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:53 2015 +0900
description:
Bug 895274 part.167 Rename NS_CONTENT_COMMAND_UNDO to eContentCommandUndo r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/29bac9c0a25ab19f1d46e0304d262604c815c4e8
changeset:  29bac9c0a25ab19f1d46e0304d262604c815c4e8
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:53 2015 +0900
description:
Bug 895274 part.168 Rename NS_CONTENT_COMMAND_REDO to eContentCommandRedo r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/6942dd64d65e89cecab991e36c82d15e2d9c83e7
changeset:  6942dd64d65e89cecab991e36c82d15e2d9c83e7
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:53 2015 +0900
description:
Bug 895274 part.169 Rename NS_CONTENT_COMMAND_SCROLL to eContentCommandScroll r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/22c8416176741836ab71461d76102eb1944c0083
changeset:  22c8416176741836ab71461d76102eb1944c0083
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:53 2015 +0900
description:
Bug 895274 part.170 Rename NS_ANIMATION_EVENT_START to eAnimationEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/67254ef03f091f2ca85a22e6d796f734834e8766
changeset:  67254ef03f091f2ca85a22e6d796f734834e8766
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:53 2015 +0900
description:
Bug 895274 part.171 Rename NS_ANIMATION_START to eAnimationStart r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/af23abc48df4250a24f4f4c8bbc82a0e9cb2321f
changeset:  af23abc48df4250a24f4f4c8bbc82a0e9cb2321f
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:53 2015 +0900
description:
Bug 895274 part.172 Rename NS_ANIMATION_END to eAnimationEnd r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/d040b1903b432cc75f153c5ec9e7aa04bf9aaa86
changeset:  d040b1903b432cc75f153c5ec9e7aa04bf9aaa86
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:53 2015 +0900
description:
Bug 895274 part.173 Rename NS_ANIMATION_ITERATION to eAnimationIteration r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/96ec3affa3d526bf405cfb33172142415e757da9
changeset:  96ec3affa3d526bf405cfb33172142415e757da9
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:53 2015 +0900
description:
Bug 895274 part.174 Rename NS_NOTIFYSCRIPT_START to eScriptEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/5c2c1630198490f1a30a87070cfca75e3d24de0f
changeset:  5c2c1630198490f1a30a87070cfca75e3d24de0f
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:53 2015 +0900
description:
Bug 895274 part.175 Rename NS_BEFORE_SCRIPT_EXECUTE to eBeforeScriptExecute r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/3d7a664c70d720cdda6723782f883478f9209f59
changeset:  3d7a664c70d720cdda6723782f883478f9209f59
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:54 2015 +0900
description:
Bug 895274 part.176 Rename NS_AFTER_SCRIPT_EXECUTE to eAfterScriptExecute r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/ec706e1895448231322775bc3fc79dc90ef3e8df
changeset:  ec706e1895448231322775bc3fc79dc90ef3e8df
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:54 2015 +0900
description:
Bug 895274 part.177 Rename NS_OPENCLOSE_EVENT_START to eOpenCloseEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/df160a23a95d4bdc8bcd22de1f4c8b456389cec6
changeset:  df160a23a95d4bdc8bcd22de1f4c8b456389cec6
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:54 2015 +0900
description:
Bug 895274 part.178 Rename NS_OPEN to eOpen r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/bceba97d9bf2f51f08b3502d1adb4e9ae501528e
changeset:  bceba97d9bf2f51f08b3502d1adb4e9ae501528e
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:54 2015 +0900
description:
Bug 895274 part.179 Remove NS_CLOSE due to unused r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/0ab51b7d2653b3d67e34eb1c24b7f46ca55f2a3a
changeset:  0ab51b7d2653b3d67e34eb1c24b7f46ca55f2a3a
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:54 2015 +0900
description:
Bug 895274 part.180 Rename NS_FULL_SCREEN_EVENT_START to eFullscreenEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/04f1150a54fd90d8ef7d07b8d25d6d976da41753
changeset:  04f1150a54fd90d8ef7d07b8d25d6d976da41753
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:54 2015 +0900
description:
Bug 895274 part.181 Rename NS_FULLSCREENCHANGE to eFullscreenChange r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/967c1272d20ab2c52e96b7e038711689497782f8
changeset:  967c1272d20ab2c52e96b7e038711689497782f8
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:54 2015 +0900
description:
Bug 895274 part.182 Rename NS_FULLSCREENERROR to eFullscreenError r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/7a008c1909daacb25f473b4a46f39b73dd556659
changeset:  7a008c1909daacb25f473b4a46f39b73dd556659
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:54 2015 +0900
description:
Bug 895274 part.183 Rename NS_WHEEL_EVENT_START to eWheelEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/e777173540407236c2b24357c854920757748f74
changeset:  e777173540407236c2b24357c854920757748f74
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:54 2015 +0900
description:
Bug 895274 part.184 Rename NS_WHEEL_WHEEL to eWheel r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/dd0a5b24d93ccd9dcec8cea76127a564ff9eaf02
changeset:  dd0a5b24d93ccd9dcec8cea76127a564ff9eaf02
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:55 2015 +0900
description:
Bug 895274 part.185 Rename NS_WHEEL_START to eWheelOperationStart r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/0f99a111de0a20e77d6de0a688860f0107f4e755
changeset:  0f99a111de0a20e77d6de0a688860f0107f4e755
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 01:59:55 2015 +0900
description:
Bug 895274 part.186 Rename NS_WHEEL_END to eWheelOperationEnd r=smaug
Hmm, I forgot to change this :-(
Attachment #8659657 - Flags: review?(bugs)
SMIL spec defines the events as beginEvent, endEvent and repeatEvent. The names don't have consistency with other DOM events, though. I guess that using these names might be better for SMIL implementer...
Attachment #8659668 - Flags: review?(bugs)
Attachment #8659657 - Flags: review?(bugs) → review+
Attachment #8659658 - Flags: review?(bugs) → review+
Attachment #8659659 - Flags: review?(bugs) → review+
Attachment #8659660 - Flags: review?(bugs) → review+
Attachment #8659662 - Flags: review?(bugs) → review+
Attachment #8659663 - Flags: review?(bugs) → review+
Attachment #8659665 - Flags: review?(bugs) → review+
Attachment #8659666 - Flags: review?(bugs) → review+
Attachment #8659667 - Flags: review?(bugs) → review+
Comment on attachment 8659668 [details] [diff] [review]
part.196 Rename NS_SMIL_BEGIN to eSMILBeginEvent

Yeah, SMIL and SVG events are nuts.
Attachment #8659668 - Flags: review?(bugs) → review+
Attachment #8659669 - Flags: review?(bugs) → review+
Attachment #8659670 - Flags: review?(bugs) → review+
Attachment #8659671 - Flags: review?(bugs) → review+
Attachment #8659672 - Flags: review?(bugs) → review+
Attachment #8659674 - Flags: review?(bugs) → review+
Attachment #8659675 - Flags: review?(bugs) → review+
Attachment #8659677 - Flags: review?(bugs) → review+
Attachment #8659678 - Flags: review?(bugs) → review+
Attachment #8659679 - Flags: review?(bugs) → review+
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/c8d53164ac16956a6443bfefc71eae50001453d5
changeset:  c8d53164ac16956a6443bfefc71eae50001453d5
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:26 2015 +0900
description:
Bug 895274 part.187 Rename NS_QUERY_TEXT_RECT to eQueryTextRect r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/6b29914363f3a7d104ec0d99c32e2b0abac0a8c9
changeset:  6b29914363f3a7d104ec0d99c32e2b0abac0a8c9
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:26 2015 +0900
description:
Bug 895274 part.188 Rename NS_COMPOSITION_COMMIT_AS_IS to eCompositionCommitAsIs r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/9e8b629344d1b4a51369a5b1cea45649bead14ba
changeset:  9e8b629344d1b4a51369a5b1cea45649bead14ba
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:26 2015 +0900
description:
Bug 895274 part.189 Rename NS_COMPOSITION_EVENT_START to eCompositionEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/094135c410a51de44a0080a904e01928021e3a3f
changeset:  094135c410a51de44a0080a904e01928021e3a3f
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:27 2015 +0900
description:
Bug 895274 part.190 Rename NS_COMPOSITION_START to eCompositionStart r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/e6420651e2c9ae7f21b03ac38c1424776c8f2179
changeset:  e6420651e2c9ae7f21b03ac38c1424776c8f2179
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:27 2015 +0900
description:
Bug 895274 part.191 Rename NS_COMPOSITION_END to eCompositionEnd r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/9f26179268e3b25496700f153b93365049dd2588
changeset:  9f26179268e3b25496700f153b93365049dd2588
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:27 2015 +0900
description:
Bug 895274 part.192 Rename NS_COMPOSITION_UPDATE to eCompositionUpdate r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/4802505f05ad9a02184bb5ffdd541057f51ebe28
changeset:  4802505f05ad9a02184bb5ffdd541057f51ebe28
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:27 2015 +0900
description:
Bug 895274 part.193 Rename NS_COMPOSITION_CHANGE to eCompositionChange r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/7cf9012735c6d2a6c7d814a33f1baa54af552c15
changeset:  7cf9012735c6d2a6c7d814a33f1baa54af552c15
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:27 2015 +0900
description:
Bug 895274 part.194 Rename NS_COMPOSITION_COMMIT to eCompositionCommit r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/07176b6eeedf1c0b4304a75795c0f1747b051365
changeset:  07176b6eeedf1c0b4304a75795c0f1747b051365
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:27 2015 +0900
description:
Bug 895274 part.195 Rename NS_SMIL_TIME_EVENT_START to eSMILEventFist r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/1ee79c909e1f3c3b49a1ad2345e75ce90adee31c
changeset:  1ee79c909e1f3c3b49a1ad2345e75ce90adee31c
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:27 2015 +0900
description:
Bug 895274 part.196 Rename NS_SMIL_BEGIN to eSMILBeginEvent r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/108cbd52aa7ecccf89aaaa23ec55a011fa7f9832
changeset:  108cbd52aa7ecccf89aaaa23ec55a011fa7f9832
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:27 2015 +0900
description:
Bug 895274 part.197 Rename NS_SMIL_END to eSMILEndEvent r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/9339cf2e33b38c6c4b50e8beff376fbd9a851a21
changeset:  9339cf2e33b38c6c4b50e8beff376fbd9a851a21
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:28 2015 +0900
description:
Bug 895274 part.198 Rename NS_SMIL_REPEAT to eSMILRepeatEvent r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/2fd2531aac12f0e334d6e1f0ab9501994c3529c3
changeset:  2fd2531aac12f0e334d6e1f0ab9501994c3529c3
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:28 2015 +0900
description:
Bug 895274 part.199 Rename NS_GAMEPAD_DISCONNECTED to eGamepadDisconnected r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/0cea2a2958ae8717491632e742059a4345f68398
changeset:  0cea2a2958ae8717491632e742059a4345f68398
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:28 2015 +0900
description:
Bug 895274 part.200 Rename NS_GAMEPAD_CONNECTED to eGamepadConnected r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/622806dd51c5660777de964847fadf5514827eea
changeset:  622806dd51c5660777de964847fadf5514827eea
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:28 2015 +0900
description:
Bug 895274 part.201 Rename NS_GAMEPAD_BUTTONDOWN to eGamepadButtonDown r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/4f954399289d7fca37fe5d466dcdc4e774ccbfde
changeset:  4f954399289d7fca37fe5d466dcdc4e774ccbfde
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:28 2015 +0900
description:
Bug 895274 part.202 Rename NS_GAMEPAD_BUTTONUP to eGamepadButtonUp r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/79473dc0e450ac3cfcb7ee139472d4a1e26ac067
changeset:  79473dc0e450ac3cfcb7ee139472d4a1e26ac067
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:28 2015 +0900
description:
Bug 895274 part.203 Rename NS_GAMEPAD_AXISMOVE to eGamepadAxisMove r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/848f9fe24f1cb34e3b933efd48c7bd2b435fd42e
changeset:  848f9fe24f1cb34e3b933efd48c7bd2b435fd42e
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:28 2015 +0900
description:
Bug 895274 part.204 Rename NS_GAMEPAD_START to eGamepadEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/f49b4cddbc850916b1d2ec85613c4a649d5eadae
changeset:  f49b4cddbc850916b1d2ec85613c4a649d5eadae
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Fri Sep 11 21:21:28 2015 +0900
description:
Bug 895274 part.205 Rename NS_GAMEPAD_END to eGamepadEventLast r=smaug
https://hg.mozilla.org/mozilla-central/rev/6e4fc82f286b
https://hg.mozilla.org/mozilla-central/rev/c46079d8b6ed
https://hg.mozilla.org/mozilla-central/rev/7f6201ff997f
https://hg.mozilla.org/mozilla-central/rev/4cdae41b9305
https://hg.mozilla.org/mozilla-central/rev/4557cfecc20d
https://hg.mozilla.org/mozilla-central/rev/aaf1002beb94
https://hg.mozilla.org/mozilla-central/rev/ec614033e822
https://hg.mozilla.org/mozilla-central/rev/25399075c9ef
https://hg.mozilla.org/mozilla-central/rev/1a346d89cc23
https://hg.mozilla.org/mozilla-central/rev/a5c0e8cbf607
https://hg.mozilla.org/mozilla-central/rev/039654784828
https://hg.mozilla.org/mozilla-central/rev/2f4d6b2be0e6
https://hg.mozilla.org/mozilla-central/rev/83512fe0f054
https://hg.mozilla.org/mozilla-central/rev/260d7fa5913d
https://hg.mozilla.org/mozilla-central/rev/672a3f59ed55
https://hg.mozilla.org/mozilla-central/rev/e59cc10fc945
https://hg.mozilla.org/mozilla-central/rev/667d75050a94
https://hg.mozilla.org/mozilla-central/rev/54ac42eda76f
https://hg.mozilla.org/mozilla-central/rev/1c4f895797d8
https://hg.mozilla.org/mozilla-central/rev/4bc7de51bb6e
https://hg.mozilla.org/mozilla-central/rev/fc17970a9de2
https://hg.mozilla.org/mozilla-central/rev/5863be1170c9
https://hg.mozilla.org/mozilla-central/rev/29bac9c0a25a
https://hg.mozilla.org/mozilla-central/rev/6942dd64d65e
https://hg.mozilla.org/mozilla-central/rev/22c841617674
https://hg.mozilla.org/mozilla-central/rev/67254ef03f09
https://hg.mozilla.org/mozilla-central/rev/af23abc48df4
https://hg.mozilla.org/mozilla-central/rev/d040b1903b43
https://hg.mozilla.org/mozilla-central/rev/96ec3affa3d5
https://hg.mozilla.org/mozilla-central/rev/5c2c16301984
https://hg.mozilla.org/mozilla-central/rev/3d7a664c70d7
https://hg.mozilla.org/mozilla-central/rev/ec706e189544
https://hg.mozilla.org/mozilla-central/rev/df160a23a95d
https://hg.mozilla.org/mozilla-central/rev/bceba97d9bf2
https://hg.mozilla.org/mozilla-central/rev/0ab51b7d2653
https://hg.mozilla.org/mozilla-central/rev/04f1150a54fd
https://hg.mozilla.org/mozilla-central/rev/967c1272d20a
https://hg.mozilla.org/mozilla-central/rev/7a008c1909da
https://hg.mozilla.org/mozilla-central/rev/e77717354040
https://hg.mozilla.org/mozilla-central/rev/dd0a5b24d93c
https://hg.mozilla.org/mozilla-central/rev/0f99a111de0a
According to the next event message, "scrolled area" is the right term, not "scroll area".
Attachment #8659945 - Flags: review?(bugs)
scrolledareachange should be the DOM event name. But it's defined as "MozScrolledAreaChanged". So, we should keep using "changed" for this message.
Attachment #8659946 - Flags: review?(bugs)
Although, the name is similar to eAfterPaint...
Attachment #8659955 - Flags: review?(bugs)
Probably, next set of the patches will be the last patches of this bug!
Attachment #8659962 - Flags: review?(bugs)
Attachment #8659930 - Flags: review?(bugs) → review+
Attachment #8659932 - Flags: review?(bugs) → review+
Attachment #8659935 - Flags: review?(bugs) → review+
Attachment #8659937 - Flags: review?(bugs) → review+
Attachment #8659938 - Flags: review?(bugs) → review+
Attachment #8659939 - Flags: review?(bugs) → review+
Attachment #8659940 - Flags: review?(bugs) → review+
Attachment #8659941 - Flags: review?(bugs) → review+
Attachment #8659943 - Flags: review?(bugs) → review+
Attachment #8659944 - Flags: review?(bugs) → review+
Attachment #8659945 - Flags: review?(bugs) → review+
Attachment #8659946 - Flags: review?(bugs) → review+
Attachment #8659947 - Flags: review?(bugs) → review+
Attachment #8659949 - Flags: review?(bugs) → review+
Attachment #8659951 - Flags: review?(bugs) → review+
Attachment #8659954 - Flags: review?(bugs) → review+
Attachment #8659955 - Flags: review?(bugs) → review+
Attachment #8659956 - Flags: review?(bugs) → review+
Attachment #8659957 - Flags: review?(bugs) → review+
Attachment #8659958 - Flags: review?(bugs) → review+
Attachment #8659959 - Flags: review?(bugs) → review+
Attachment #8659961 - Flags: review?(bugs) → review+
Attachment #8659962 - Flags: review?(bugs) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/4ac1ba0956eb
https://hg.mozilla.org/integration/mozilla-inbound/rev/8d9975a1d6f6
https://hg.mozilla.org/integration/mozilla-inbound/rev/fa148ad38740
https://hg.mozilla.org/integration/mozilla-inbound/rev/9f1540b140e5
https://hg.mozilla.org/integration/mozilla-inbound/rev/6953adc7ca1a
https://hg.mozilla.org/integration/mozilla-inbound/rev/909e0d60f7c3
https://hg.mozilla.org/integration/mozilla-inbound/rev/e14ea6453892
https://hg.mozilla.org/integration/mozilla-inbound/rev/c59957d6ca04
https://hg.mozilla.org/integration/mozilla-inbound/rev/9a17ec827f59
https://hg.mozilla.org/integration/mozilla-inbound/rev/cbf828cd48ae
https://hg.mozilla.org/integration/mozilla-inbound/rev/07c0e58b5f16
https://hg.mozilla.org/integration/mozilla-inbound/rev/62295fa19e50
https://hg.mozilla.org/integration/mozilla-inbound/rev/6f368a13d9ed
https://hg.mozilla.org/integration/mozilla-inbound/rev/5e8f4f902562
https://hg.mozilla.org/integration/mozilla-inbound/rev/062647162efb
https://hg.mozilla.org/integration/mozilla-inbound/rev/9a88876e7fa5
https://hg.mozilla.org/integration/mozilla-inbound/rev/9e042218bfc0
https://hg.mozilla.org/integration/mozilla-inbound/rev/39fcfeae16fa
https://hg.mozilla.org/integration/mozilla-inbound/rev/7a662a29591b
https://hg.mozilla.org/integration/mozilla-inbound/rev/d3a94d42ee8e
https://hg.mozilla.org/integration/mozilla-inbound/rev/ef4b9a26ef43
https://hg.mozilla.org/integration/mozilla-inbound/rev/0b13a7d48bc3
https://hg.mozilla.org/integration/mozilla-inbound/rev/28fe9b60f5d5
https://hg.mozilla.org/mozilla-central/rev/c8d53164ac16
https://hg.mozilla.org/mozilla-central/rev/6b29914363f3
https://hg.mozilla.org/mozilla-central/rev/9e8b629344d1
https://hg.mozilla.org/mozilla-central/rev/094135c410a5
https://hg.mozilla.org/mozilla-central/rev/e6420651e2c9
https://hg.mozilla.org/mozilla-central/rev/9f26179268e3
https://hg.mozilla.org/mozilla-central/rev/4802505f05ad
https://hg.mozilla.org/mozilla-central/rev/7cf9012735c6
https://hg.mozilla.org/mozilla-central/rev/07176b6eeedf
https://hg.mozilla.org/mozilla-central/rev/1ee79c909e1f
https://hg.mozilla.org/mozilla-central/rev/108cbd52aa7e
https://hg.mozilla.org/mozilla-central/rev/9339cf2e33b3
https://hg.mozilla.org/mozilla-central/rev/2fd2531aac12
https://hg.mozilla.org/mozilla-central/rev/0cea2a2958ae
https://hg.mozilla.org/mozilla-central/rev/622806dd51c5
https://hg.mozilla.org/mozilla-central/rev/4f954399289d
https://hg.mozilla.org/mozilla-central/rev/79473dc0e450
https://hg.mozilla.org/mozilla-central/rev/848f9fe24f1c
https://hg.mozilla.org/mozilla-central/rev/f49b4cddbc85
https://hg.mozilla.org/mozilla-central/rev/4ac1ba0956eb
https://hg.mozilla.org/mozilla-central/rev/8d9975a1d6f6
https://hg.mozilla.org/mozilla-central/rev/fa148ad38740
https://hg.mozilla.org/mozilla-central/rev/9f1540b140e5
https://hg.mozilla.org/mozilla-central/rev/6953adc7ca1a
https://hg.mozilla.org/mozilla-central/rev/909e0d60f7c3
https://hg.mozilla.org/mozilla-central/rev/e14ea6453892
https://hg.mozilla.org/mozilla-central/rev/c59957d6ca04
https://hg.mozilla.org/mozilla-central/rev/9a17ec827f59
https://hg.mozilla.org/mozilla-central/rev/cbf828cd48ae
https://hg.mozilla.org/mozilla-central/rev/07c0e58b5f16
https://hg.mozilla.org/mozilla-central/rev/62295fa19e50
https://hg.mozilla.org/mozilla-central/rev/6f368a13d9ed
https://hg.mozilla.org/mozilla-central/rev/5e8f4f902562
https://hg.mozilla.org/mozilla-central/rev/062647162efb
https://hg.mozilla.org/mozilla-central/rev/9a88876e7fa5
https://hg.mozilla.org/mozilla-central/rev/9e042218bfc0
https://hg.mozilla.org/mozilla-central/rev/39fcfeae16fa
https://hg.mozilla.org/mozilla-central/rev/7a662a29591b
https://hg.mozilla.org/mozilla-central/rev/d3a94d42ee8e
https://hg.mozilla.org/mozilla-central/rev/ef4b9a26ef43
https://hg.mozilla.org/mozilla-central/rev/0b13a7d48bc3
https://hg.mozilla.org/mozilla-central/rev/28fe9b60f5d5
Oh, I forgot to say, reducing the indent before the second argument is intentional. Old indentation is wrong.
Attachment #8660679 - Flags: review?(bugs)
Attachment #8660673 - Flags: review?(bugs) → review+
Attachment #8660674 - Flags: review?(bugs) → review+
Attachment #8660675 - Flags: review?(bugs) → review+
Attachment #8660676 - Flags: review?(bugs) → review+
Attachment #8660677 - Flags: review?(bugs) → review+
Attachment #8660678 - Flags: review?(bugs) → review+
I think that "MediaRecorder" prefix is necessary since names of following events are too generic.
Attachment #8660681 - Flags: review?(bugs)
Attachment #8660679 - Flags: review?(bugs) → review+
Attachment #8660680 - Flags: review?(bugs) → review+
Attachment #8660681 - Flags: review?(bugs) → review+
Attachment #8660682 - Flags: review?(bugs) → review+
I use DOM event name without "Moz" prefix.
Attachment #8660705 - Flags: review?(bugs)
Attachment #8660688 - Flags: review?(bugs) → review+
Attachment #8660689 - Flags: review?(bugs) → review+
Attachment #8660690 - Flags: review?(bugs) → review+
Attachment #8660692 - Flags: review?(bugs) → review+
Attachment #8660693 - Flags: review?(bugs) → review+
Attachment #8660694 - Flags: review?(bugs) → review+
Attachment #8660695 - Flags: review?(bugs) → review+
Attachment #8660696 - Flags: review?(bugs) → review+
Attachment #8660697 - Flags: review?(bugs) → review+
Attachment #8660698 - Flags: review?(bugs) → review+
Attachment #8660699 - Flags: review?(bugs) → review+
Attachment #8660700 - Flags: review?(bugs) → review+
Attachment #8660702 - Flags: review?(bugs) → review+
Attachment #8660703 - Flags: review?(bugs) → review+
Attachment #8660705 - Flags: review?(bugs) → review+
Attachment #8660706 - Flags: review?(bugs) → review+
Attachment #8660707 - Flags: review?(bugs) → review+
Attachment #8660708 - Flags: review?(bugs) → review+
Attachment #8660709 - Flags: review?(bugs) → review+
Attachment #8660710 - Flags: review?(bugs) → review+
Attachment #8660711 - Flags: review?(bugs) → review+
Attachment #8660712 - Flags: review?(bugs) → review+
Attachment #8660713 - Flags: review?(bugs) → review+
Attachment #8660714 - Flags: review?(bugs) → review+
Attachment #8660716 - Flags: review?(bugs) → review+
Attachment #8660717 - Flags: review?(bugs) → review+
Attachment #8660718 - Flags: review?(bugs) → review+
Attachment #8660719 - Flags: review?(bugs) → review+
Attachment #8660720 - Flags: review?(bugs) → review+
Attachment #8660721 - Flags: review?(bugs) → review+
Attachment #8660722 - Flags: review?(bugs) → review+
Attachment #8660723 - Flags: review?(bugs) → review+
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/15753ee8232fa1585dad78af9569a3d237cda675
changeset:  15753ee8232fa1585dad78af9569a3d237cda675
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:33 2015 +0900
description:
Bug 895274 part.229 Rename NS_SELECTION_CHANGE to eSelectionChange r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/9112165662f7d243128171a0ec9e54d300934ad1
changeset:  9112165662f7d243128171a0ec9e54d300934ad1
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:33 2015 +0900
description:
Bug 895274 part.230 Rename NS_SELECT_START to eSelectStart r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/c738a1dd461142aafb22c0775f6c52ebf6cb2671
changeset:  c738a1dd461142aafb22c0775f6c52ebf6cb2671
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:33 2015 +0900
description:
Bug 895274 part.231 Rename NS_SELECT_EVENT_START to eSelectEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/608fec053b04ed5cfea0368cf6defd6704083625
changeset:  608fec053b04ed5cfea0368cf6defd6704083625
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:33 2015 +0900
description:
Bug 895274 part.232 Rename NS_EDITOR_INPUT to eEditorInput r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/d7d2811a7c2d024b4d7e37307acc8f99898ee09c
changeset:  d7d2811a7c2d024b4d7e37307acc8f99898ee09c
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:33 2015 +0900
description:
Bug 895274 part.233 Rename NS_EDITOR_EVENT_START to eEditorEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/fa00a86758ce17894ee0367c53121d03cba174c4
changeset:  fa00a86758ce17894ee0367c53121d03cba174c4
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:34 2015 +0900
description:
Bug 895274 part.234 Rename NS_SPEAKERMANAGER_SPEAKERFORCEDCHANGE to eSpeakerForcedChange r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/fe3323ce2a4b7673166e00822ed8dcf0b6ab7d32
changeset:  fe3323ce2a4b7673166e00822ed8dcf0b6ab7d32
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:34 2015 +0900
description:
Bug 895274 part.235 Rename NS_SPEAKERMANAGER_EVENT_START to eSpeakerManagerEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/f1342c2e9f05519290110b65fec493db7f89eb33
changeset:  f1342c2e9f05519290110b65fec493db7f89eb33
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:34 2015 +0900
description:
Bug 895274 part.236 Rename NS_MEDIARECORDER_EVENT_START to eMediaRecorderEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/0ae5bfbea97eb87dce31008581395368fbbac6f9
changeset:  0ae5bfbea97eb87dce31008581395368fbbac6f9
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:34 2015 +0900
description:
Bug 895274 part.237 Rename NS_MEDIARECORDER_DATAAVAILABLE to eMediaRecorderDataAvailable r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/a74a014e8b29a10075fedc19d52c3126d7d11e08
changeset:  a74a014e8b29a10075fedc19d52c3126d7d11e08
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:34 2015 +0900
description:
Bug 895274 part.238 Rename NS_MEDIARECORDER_WARNING to eMediaRecorderWarning r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/e0c0342f606c0b5db6e1b93a668b027204b1d026
changeset:  e0c0342f606c0b5db6e1b93a668b027204b1d026
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:34 2015 +0900
description:
Bug 895274 part.239 Rename NS_MEDIARECORDER_STOP to eMediaRecorderStop r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/823a3bba6c8bfedc3fcfddca882a79fce66c6235
changeset:  823a3bba6c8bfedc3fcfddca882a79fce66c6235
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:34 2015 +0900
description:
Bug 895274 part.240 Rename NS_TOUCH_EVENT_START to eTouchEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/a92cec9902d70d52289ee2205b3f5f925c332e3c
changeset:  a92cec9902d70d52289ee2205b3f5f925c332e3c
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:34 2015 +0900
description:
Bug 895274 part.241 Rename NS_TOUCH_START to eTouchStart r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/79b8d374b4fd3c310e3504831c188db145e30a17
changeset:  79b8d374b4fd3c310e3504831c188db145e30a17
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:35 2015 +0900
description:
Bug 895274 part.242 Rename NS_TOUCH_MOVE to eTouchMove r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/aa8e1b6f6753c7f5d8413944b353bc2b18570a04
changeset:  aa8e1b6f6753c7f5d8413944b353bc2b18570a04
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:35 2015 +0900
description:
Bug 895274 part.243 Rename NS_TOUCH_END to eTouchEnd r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/641727472a5c88c547a87e64ea6a9bd1124a2af1
changeset:  641727472a5c88c547a87e64ea6a9bd1124a2af1
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:35 2015 +0900
description:
Bug 895274 part.244 Rename NS_TOUCH_CANCEL to eTouchCancel r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/2bbed060054e5118b27dfc6a3faf7b4c94dc101e
changeset:  2bbed060054e5118b27dfc6a3faf7b4c94dc101e
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:35 2015 +0900
description:
Bug 895274 part.245 Rename NS_SHOW_EVENT to eShow r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/e7592497917894a6597b8c003388f3944a4d4648
changeset:  e7592497917894a6597b8c003388f3944a4d4648
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:35 2015 +0900
description:
Bug 895274 part.246 Rename NS_MESSAGE to eMessage r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/5d2cba4706f66babe76ad2dd7883758a40552f26
changeset:  5d2cba4706f66babe76ad2dd7883758a40552f26
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:35 2015 +0900
description:
Bug 895274 part.247 Rename NS_MESSAGE_EVENT_START to eMessageEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/69d3b92fa23865536745acb23dcda72ebaad0a1a
changeset:  69d3b92fa23865536745acb23dcda72ebaad0a1a
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:35 2015 +0900
description:
Bug 895274 part.248 Rename NS_WEBAUDIO_EVENT_START to eAudioEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/827f184daa0fe3e174597b75d0787682c6596229
changeset:  827f184daa0fe3e174597b75d0787682c6596229
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:36 2015 +0900
description:
Bug 895274 part.249 Rename NS_AUDIO_PROCESS to eAudioProcess r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/bcc63510e6c5b228506b14f9e01342a4d86ae813
changeset:  bcc63510e6c5b228506b14f9e01342a4d86ae813
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:36 2015 +0900
description:
Bug 895274 part.250 Rename NS_AUDIO_COMPLETE to eAudioComplete r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/a42e812473515f470c71b5ffa0117a5941dddc39
changeset:  a42e812473515f470c71b5ffa0117a5941dddc39
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:36 2015 +0900
description:
Bug 895274 part.251 Rename NS_TRANSITION_EVENT_START to eTransitionEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/8f9e04ad1f2363ee504dc424656ef625faa57c5d
changeset:  8f9e04ad1f2363ee504dc424656ef625faa57c5d
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:36 2015 +0900
description:
Bug 895274 part.252 Rename NS_TRANSITION_END to eTransitionEnd r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/3aefc8bd19fd6ece1ea975fa0ba84a212ffbfc23
changeset:  3aefc8bd19fd6ece1ea975fa0ba84a212ffbfc23
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:36 2015 +0900
description:
Bug 895274 part.253 Rename NS_SIMPLE_GESTURE_EDGE_COMPLETED to eEdgeUICompleted r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/c391904401eea3bb82fd01aeab379da0c13e734d
changeset:  c391904401eea3bb82fd01aeab379da0c13e734d
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:36 2015 +0900
description:
Bug 895274 part.254 Rename NS_SIMPLE_GESTURE_EDGE_CANCELED to eEdgeUICanceled r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/8a9e0f1b03127eda04c7182b987168c69344b9ea
changeset:  8a9e0f1b03127eda04c7182b987168c69344b9ea
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:36 2015 +0900
description:
Bug 895274 part.255 Rename NS_SIMPLE_GESTURE_EDGE_STARTED to eEdgeUIStarted r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/85a692a19853eb3f3f6594ebedbe9170d8dbd103
changeset:  85a692a19853eb3f3f6594ebedbe9170d8dbd103
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:36 2015 +0900
description:
Bug 895274 part.256 Rename NS_SIMPLE_GESTURE_PRESSTAP to ePressTapGesture r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/820d0930ab430c74ccb7e60dd0968647bc2e6889
changeset:  820d0930ab430c74ccb7e60dd0968647bc2e6889
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:36 2015 +0900
description:
Bug 895274 part.257 Rename NS_SIMPLE_GESTURE_TAP to eTapGesture r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/9981f3b651a401d639ee0f881ce71185117ff956
changeset:  9981f3b651a401d639ee0f881ce71185117ff956
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:36 2015 +0900
description:
Bug 895274 part.258 Rename NS_SIMPLE_GESTURE_ROTATE to eRotateGesture r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/65489def5c252d57208ad763df49178df268113f
changeset:  65489def5c252d57208ad763df49178df268113f
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:36 2015 +0900
description:
Bug 895274 part.259 Rename NS_SIMPLE_GESTURE_ROTATE_UPDATE to eRotateGestureUpdate r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/4b22918a90f668f48b8f39d04b378315db0ac62d
changeset:  4b22918a90f668f48b8f39d04b378315db0ac62d
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:37 2015 +0900
description:
Bug 895274 part.260 Rename NS_SIMPLE_GESTURE_ROTATE_START to eRotateGestureStart r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/58cba751dfe5bbf823054f25ca8deeee5d3d39d8
changeset:  58cba751dfe5bbf823054f25ca8deeee5d3d39d8
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:37 2015 +0900
description:
Bug 895274 part.261 Rename NS_SIMPLE_GESTURE_MAGNIFY to eMagnifyGesture r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/2f0716ec8171759506434a0a1f82648dc8f56453
changeset:  2f0716ec8171759506434a0a1f82648dc8f56453
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:37 2015 +0900
description:
Bug 895274 part.262 Rename NS_SIMPLE_GESTURE_MAGNIFY_UPDATE to eMagnifyGestureUpdate r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/a61c8be5100bcaac9f792e6d8d11bd1098725ff7
changeset:  a61c8be5100bcaac9f792e6d8d11bd1098725ff7
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:37 2015 +0900
description:
Bug 895274 part.263 Rename NS_SIMPLE_GESTURE_MAGNIFY_START to eMagnifyGestureStart r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/9443d9e5318e8a792b817f745a58fce6bf11f95f
changeset:  9443d9e5318e8a792b817f745a58fce6bf11f95f
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:37 2015 +0900
description:
Bug 895274 part.264 Rename NS_SIMPLE_GESTURE_SWIPE to eSwipeGesture r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/7d3ff6e1cdfea1ce6fbdfdfb9eb9a72fc93557a2
changeset:  7d3ff6e1cdfea1ce6fbdfdfb9eb9a72fc93557a2
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:37 2015 +0900
description:
Bug 895274 part.265 Rename NS_SIMPLE_GESTURE_SWIPE_END to eSwipeGestureEnd r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/035f20d39060839225fc5e016789a9086a1cc5dc
changeset:  035f20d39060839225fc5e016789a9086a1cc5dc
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:37 2015 +0900
description:
Bug 895274 part.266 Rename NS_SIMPLE_GESTURE_SWIPE_UPDATE to eSwipeGestureUpdate r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/a158c3a14239946a596babf50b3886d34ca44792
changeset:  a158c3a14239946a596babf50b3886d34ca44792
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:37 2015 +0900
description:
Bug 895274 part.267 Rename NS_SIMPLE_GESTURE_SWIPE_START to eSwipeGestureStart r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/f2efb808b885285d98a7453a8b17b3878260ab9c
changeset:  f2efb808b885285d98a7453a8b17b3878260ab9c
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:38 2015 +0900
description:
Bug 895274 part.268 Rename NS_SIMPLE_GESTURE_SWIPE_MAY_START to eSwipeGestureMayStart r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/e69d4cf68a8729310ca15166fc3d8f0951753430
changeset:  e69d4cf68a8729310ca15166fc3d8f0951753430
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:38 2015 +0900
description:
Bug 895274 part.269 Rename NS_SIMPLE_GESTURE_EVENT_START to eSimpleGestureEventFirst r=smaug

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/b3f5c61b22ee65a236d04bdbbc4a3aa71c34435c
changeset:  b3f5c61b22ee65a236d04bdbbc4a3aa71c34435c
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Tue Sep 15 00:14:38 2015 +0900
description:
Bug 895274 part.270 Add comment to explain the naming rules of EventMessage r=smaug
Whiteboard: [leave open]
https://hg.mozilla.org/mozilla-central/rev/15753ee8232f
https://hg.mozilla.org/mozilla-central/rev/9112165662f7
https://hg.mozilla.org/mozilla-central/rev/c738a1dd4611
https://hg.mozilla.org/mozilla-central/rev/608fec053b04
https://hg.mozilla.org/mozilla-central/rev/d7d2811a7c2d
https://hg.mozilla.org/mozilla-central/rev/fa00a86758ce
https://hg.mozilla.org/mozilla-central/rev/fe3323ce2a4b
https://hg.mozilla.org/mozilla-central/rev/f1342c2e9f05
https://hg.mozilla.org/mozilla-central/rev/0ae5bfbea97e
https://hg.mozilla.org/mozilla-central/rev/a74a014e8b29
https://hg.mozilla.org/mozilla-central/rev/e0c0342f606c
https://hg.mozilla.org/mozilla-central/rev/823a3bba6c8b
https://hg.mozilla.org/mozilla-central/rev/a92cec9902d7
https://hg.mozilla.org/mozilla-central/rev/79b8d374b4fd
https://hg.mozilla.org/mozilla-central/rev/aa8e1b6f6753
https://hg.mozilla.org/mozilla-central/rev/641727472a5c
https://hg.mozilla.org/mozilla-central/rev/2bbed060054e
https://hg.mozilla.org/mozilla-central/rev/e75924979178
https://hg.mozilla.org/mozilla-central/rev/5d2cba4706f6
https://hg.mozilla.org/mozilla-central/rev/69d3b92fa238
https://hg.mozilla.org/mozilla-central/rev/827f184daa0f
https://hg.mozilla.org/mozilla-central/rev/bcc63510e6c5
https://hg.mozilla.org/mozilla-central/rev/a42e81247351
https://hg.mozilla.org/mozilla-central/rev/8f9e04ad1f23
https://hg.mozilla.org/mozilla-central/rev/3aefc8bd19fd
https://hg.mozilla.org/mozilla-central/rev/c391904401ee
https://hg.mozilla.org/mozilla-central/rev/8a9e0f1b0312
https://hg.mozilla.org/mozilla-central/rev/85a692a19853
https://hg.mozilla.org/mozilla-central/rev/820d0930ab43
https://hg.mozilla.org/mozilla-central/rev/9981f3b651a4
https://hg.mozilla.org/mozilla-central/rev/65489def5c25
https://hg.mozilla.org/mozilla-central/rev/4b22918a90f6
https://hg.mozilla.org/mozilla-central/rev/58cba751dfe5
https://hg.mozilla.org/mozilla-central/rev/2f0716ec8171
https://hg.mozilla.org/mozilla-central/rev/a61c8be5100b
https://hg.mozilla.org/mozilla-central/rev/9443d9e5318e
https://hg.mozilla.org/mozilla-central/rev/7d3ff6e1cdfe
https://hg.mozilla.org/mozilla-central/rev/035f20d39060
https://hg.mozilla.org/mozilla-central/rev/a158c3a14239
https://hg.mozilla.org/mozilla-central/rev/f2efb808b885
https://hg.mozilla.org/mozilla-central/rev/e69d4cf68a87
https://hg.mozilla.org/mozilla-central/rev/b3f5c61b22ee
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in before you can comment on or make changes to this bug.