[Webkit-unassigned] [Bug 15057] EditorClientGtk is missing some important keypress handling, patch attached
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Aug 23 09:27:41 PDT 2007
http://bugs.webkit.org/show_bug.cgi?id=15057
aroben at apple.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #16093| |review-
Flag| |
------- Comment #22 from aroben at apple.com 2007-08-23 09:27 PDT -------
(From update of attachment 16093)
+static const KeyEntry keyEntries[] = {
I think we might as well move this whole array inside interpretKeyEvent, since
it's only used in there.
+const char* Editor::interpretKeyEvent(const KeyboardEvent* evt)
Since we're touching this code anyway, I think it would be good to make this a
static helper function instead of a method on Editor, and perhaps give it a
better name like "commandForEvent".
+#if !PLATFORM(MAC)
+ const char* interpretKeyEvent(const KeyboardEvent* evt);
void handleKeypress(KeyboardEvent*);
+#endif
handleKeypress should not be within the #if !PLATFORM(MAC) block.
handleKeypress exists on all platforms, but it has two implementations, one in
Editor.cpp, and one in EditorMac.mm.
You should remove the declarations of WebView::interpretKeyEvent and
WebView::handleEditingKeyboardEvent from WebView.h.
It would also be good to be slightly more explicit in your ChangeLogs by saying
that the code was moved from WebView to Editor.
--
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
More information about the webkit-unassigned
mailing list