Bug 74685 - Give PlatformEvents a base class
Summary: Give PlatformEvents a base class
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Sam Weinig
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-15 21:35 PST by Sam Weinig
Modified: 2011-12-16 14:46 PST (History)
3 users (show)

See Also:


Attachments
Patch (148.17 KB, patch)
2011-12-15 21:36 PST, Sam Weinig
no flags Details | Formatted Diff | Diff
Patch (150.89 KB, patch)
2011-12-15 21:45 PST, Sam Weinig
no flags Details | Formatted Diff | Diff
Patch (150.99 KB, patch)
2011-12-15 22:39 PST, Sam Weinig
no flags Details | Formatted Diff | Diff
Patch (151.02 KB, patch)
2011-12-15 22:54 PST, Sam Weinig
no flags Details | Formatted Diff | Diff
Patch (151.21 KB, patch)
2011-12-16 09:48 PST, Sam Weinig
no flags Details | Formatted Diff | Diff
Patch (151.19 KB, patch)
2011-12-16 10:38 PST, Sam Weinig
no flags Details | Formatted Diff | Diff
Patch (164.22 KB, patch)
2011-12-16 10:45 PST, Sam Weinig
no flags Details | Formatted Diff | Diff
Patch (166.16 KB, patch)
2011-12-16 12:53 PST, Sam Weinig
andersca: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sam Weinig 2011-12-15 21:35:06 PST
Give PlatformEvents a base class
Comment 1 Sam Weinig 2011-12-15 21:36:44 PST
Created attachment 119561 [details]
Patch
Comment 2 Sam Weinig 2011-12-15 21:39:54 PST
Giving the EWS a chance to chew through this one. It touches a lot of platform files :(.
Comment 3 WebKit Review Bot 2011-12-15 21:41:02 PST
Attachment 119561 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1

Source/WebCore/platform/PlatformMouseEvent.h:97:  Should have a space between // and comment  [whitespace/comments] [4]
Source/WebCore/platform/PlatformMouseEvent.h:137:  The parameter name "button" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/PlatformMouseEvent.h:137:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/PlatformKeyboardEvent.h:69:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
Source/WebCore/platform/wx/MouseEventWx.cpp:40:  An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
Source/WebKit2/WebProcess/WebPage/WebPage.cpp:1187:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Source/WebCore/dom/KeyboardEvent.cpp:40:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Total errors found: 7 in 85 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 Sam Weinig 2011-12-15 21:45:53 PST
Created attachment 119562 [details]
Patch
Comment 5 WebKit Review Bot 2011-12-15 21:48:37 PST
Attachment 119562 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/CMakeLists.txt', u'Source/W..." exit_code: 1

Source/WebCore/platform/PlatformMouseEvent.h:97:  Should have a space between // and comment  [whitespace/comments] [4]
Source/WebCore/platform/PlatformMouseEvent.h:137:  The parameter name "button" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/PlatformMouseEvent.h:137:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebKit2/WebProcess/WebPage/WebPage.cpp:1187:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Source/WebCore/platform/PlatformKeyboardEvent.h:69:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
Source/WebCore/dom/KeyboardEvent.cpp:40:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Source/WebCore/platform/wx/MouseEventWx.cpp:40:  An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
Total errors found: 7 in 90 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 6 WebKit Review Bot 2011-12-15 22:06:45 PST
Comment on attachment 119562 [details]
Patch

Attachment 119562 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/10895582
Comment 7 Sam Weinig 2011-12-15 22:39:59 PST
Created attachment 119569 [details]
Patch
Comment 8 WebKit Review Bot 2011-12-15 22:42:38 PST
Attachment 119569 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/CMakeLists.txt', u'Source/W..." exit_code: 1

Source/WebCore/platform/PlatformMouseEvent.h:97:  Should have a space between // and comment  [whitespace/comments] [4]
Source/WebCore/platform/PlatformMouseEvent.h:137:  The parameter name "button" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/PlatformMouseEvent.h:137:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebKit2/WebProcess/WebPage/WebPage.cpp:1187:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Source/WebCore/platform/PlatformKeyboardEvent.h:69:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
Source/WebCore/dom/KeyboardEvent.cpp:40:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Source/WebCore/platform/wx/MouseEventWx.cpp:40:  An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
Total errors found: 7 in 90 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 9 Sam Weinig 2011-12-15 22:54:51 PST
Created attachment 119571 [details]
Patch
Comment 10 WebKit Review Bot 2011-12-15 22:58:09 PST
Attachment 119571 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/CMakeLists.txt', u'Source/W..." exit_code: 1

Source/WebCore/platform/PlatformMouseEvent.h:97:  Should have a space between // and comment  [whitespace/comments] [4]
Source/WebCore/platform/PlatformMouseEvent.h:137:  The parameter name "button" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/PlatformMouseEvent.h:137:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebKit2/WebProcess/WebPage/WebPage.cpp:1187:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Source/WebCore/platform/PlatformKeyboardEvent.h:69:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
Source/WebCore/dom/KeyboardEvent.cpp:40:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Source/WebCore/platform/wx/MouseEventWx.cpp:40:  An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
Total errors found: 7 in 90 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 11 WebKit Review Bot 2011-12-15 23:21:48 PST
Comment on attachment 119571 [details]
Patch

Attachment 119571 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/10914573
Comment 12 Early Warning System Bot 2011-12-16 00:03:25 PST
Comment on attachment 119571 [details]
Patch

Attachment 119571 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/10900593
Comment 13 Sam Weinig 2011-12-16 09:48:10 PST
Created attachment 119626 [details]
Patch
Comment 14 WebKit Review Bot 2011-12-16 09:49:43 PST
Attachment 119626 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/CMakeLists.txt', u'Source/W..." exit_code: 1

Source/WebCore/platform/PlatformMouseEvent.h:97:  Should have a space between // and comment  [whitespace/comments] [4]
Source/WebCore/platform/PlatformMouseEvent.h:137:  The parameter name "button" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/PlatformMouseEvent.h:137:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebKit2/WebProcess/WebPage/WebPage.cpp:1187:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Source/WebCore/platform/PlatformKeyboardEvent.h:69:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
Source/WebCore/dom/KeyboardEvent.cpp:40:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Source/WebCore/platform/wx/MouseEventWx.cpp:40:  An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
Total errors found: 7 in 90 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 15 Early Warning System Bot 2011-12-16 09:59:05 PST
Comment on attachment 119626 [details]
Patch

Attachment 119626 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/10904775
Comment 16 WebKit Review Bot 2011-12-16 10:28:56 PST
Comment on attachment 119626 [details]
Patch

Attachment 119626 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/10904784
Comment 17 Sam Weinig 2011-12-16 10:38:58 PST
Created attachment 119632 [details]
Patch
Comment 18 WebKit Review Bot 2011-12-16 10:43:04 PST
Attachment 119632 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/CMakeLists.txt', u'Source/W..." exit_code: 1

Source/WebCore/platform/PlatformMouseEvent.h:97:  Should have a space between // and comment  [whitespace/comments] [4]
Source/WebCore/platform/PlatformMouseEvent.h:137:  The parameter name "button" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/PlatformMouseEvent.h:137:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebKit2/WebProcess/WebPage/WebPage.cpp:1187:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Source/WebCore/platform/PlatformKeyboardEvent.h:69:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
Source/WebCore/dom/KeyboardEvent.cpp:40:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Source/WebCore/platform/wx/MouseEventWx.cpp:40:  An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
Total errors found: 7 in 90 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 19 Sam Weinig 2011-12-16 10:45:34 PST
Created attachment 119633 [details]
Patch
Comment 20 WebKit Review Bot 2011-12-16 10:48:59 PST
Attachment 119633 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/CMakeLists.txt', u'Source/W..." exit_code: 1

Source/WebCore/platform/PlatformMouseEvent.h:97:  Should have a space between // and comment  [whitespace/comments] [4]
Source/WebCore/platform/PlatformMouseEvent.h:137:  The parameter name "button" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/PlatformMouseEvent.h:137:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebKit2/WebProcess/WebPage/WebPage.cpp:1187:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Source/WebCore/platform/PlatformKeyboardEvent.h:69:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
Source/WebCore/dom/KeyboardEvent.cpp:40:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Source/WebCore/platform/wx/MouseEventWx.cpp:40:  An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
Total errors found: 7 in 91 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 21 Sam Weinig 2011-12-16 12:53:54 PST
Created attachment 119656 [details]
Patch
Comment 22 WebKit Review Bot 2011-12-16 13:07:23 PST
Attachment 119656 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/CMakeLists.txt', u'Source/W..." exit_code: 1

Source/WebKit2/WebProcess/WebPage/WebPage.cpp:1187:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Source/WebCore/platform/PlatformKeyboardEvent.h:69:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
Source/WebCore/dom/KeyboardEvent.cpp:40:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Total errors found: 3 in 91 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 23 Anders Carlsson 2011-12-16 13:16:19 PST
Comment on attachment 119656 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=119656&action=review

> Source/WebCore/platform/PlatformEvent.h:112
> +

You should add a protected destructor here as well, that way people won't inadvertly delete a platform event. The destructor doesn't need to be virtual.
Comment 24 Sam Weinig 2011-12-16 14:46:38 PST
Committed r103112: <http://trac.webkit.org/changeset/103112>