Bug 57416 – Make correspondence of file name related with event to other port.

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[ChangSeok Oh]

Some file names are different from other port' one.
For instance, let's say about MouseEventGtk.cpp
It defines PlatformMouseEvent class for gtk port.
But the other port has same class in different form of file like PlatformMouseEventMac.mm, PlatformMouseEventEfl.cpp and PlatformMouseEventQT.cpp

I think this ruins a correspondence among ports and make user hard to find file.
In my opinion. gtk port should also follow other ports naming rule.

[ChangSeok Oh]

Created attachment 87668 [details]
Proposed patch

This patch just changes three file names.

KeyEventGtk.cpp -> PlatformKeyboardEventGtk.cpp
MouseEventGtk.cpp -> PlatformMouseEventGtk.cpp
WheelEventGtk.cpp -> PlatformWheelEventGtk.cpp

[Eric Seidel (no email)]

Comment on attachment 87668 [details]
Proposed patch

Seems fine.  I can't remember if teh cq can handle renames or not.

[WebKit Commit Bot]

Comment on attachment 87668 [details]
Proposed patch

Rejecting attachment 87668 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=eseidel-cq-sf', 'ap..." exit_code: 2

Last 500 characters of output:
ects to file Source/WebCore/GNUmakefile.am.rej
patching file Source/WebCore/platform/gtk/KeyEventGtk.cpp
rm 'Source/WebCore/platform/gtk/KeyEventGtk.cpp'
patching file Source/WebCore/platform/gtk/MouseEventGtk.cpp
rm 'Source/WebCore/platform/gtk/MouseEventGtk.cpp'
patching file Source/WebCore/platform/gtk/WheelEventGtk.cpp
rm 'Source/WebCore/platform/gtk/WheelEventGtk.cpp'

Failed to run "[u'/Projects/CommitQueue/Tools/Scripts/svn-apply', u'--reviewer', u'Eric Seidel', u'--force']" exit_code: 1

Full output: http://queues.webkit.org/results/8374755

[ChangSeok Oh]

Created attachment 88966 [details]
revised patch to meet latest version(r83419)

[ChangSeok Oh]

Created attachment 88967 [details]
revised patch to meet latest version(r83419)

To fix build break.

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 88967 [details]:

java/lc3/JavaObject/JavaObjectToLong-003-n.html bug 58218 (author: [e-mail])
The commit-queue is continuing to process your patch.

[WebKit Commit Bot]

Comment on attachment 88967 [details]
revised patch to meet latest version(r83419)

Clearing flags on attachment: 88967

Committed r83429: <http://trac.webkit.org/changeset/83429>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.