[Webkit-unassigned] [Bug 145598] [GTK] Get rid of GetEditorCommandsForKeyEvent sync message

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jun 10 03:25:25 PDT 2015


--- Comment #6 from Zan Dobersek <zan at falconsigh.net> ---
Comment on attachment 254159
  --> https://bugs.webkit.org/attachment.cgi?id=254159

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

>>> Source/WebCore/dom/KeyboardEvent.cpp:233
>>> +#endif
>> Why not use the m_handledByInputMethod boolean, like it's used for the Cocoa platform?
> Because we don't need to duplicate that, this is just a convenient method, to avoid having to get the platform event, check if it's null, and then get the boolean.

I don't think using m_handledByInputMethod the same way it's used for PLATFORM(COCOA) && USE(APPKIT) is duplicating. I do think adding a new GTK-specific method that queries some state that a KeyboardEvent constructor could already determine is duplicating.

But given that this is a Web-exposed class, any platform-specific code is regrettable.

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150610/23901c1a/attachment.html>

More information about the webkit-unassigned mailing list