[Webkit-unassigned] [Bug 61272] [Chromium] On-the-spot IME support for windowless plug-ins

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jun 23 08:03:20 PDT 2011


https://bugs.webkit.org/show_bug.cgi?id=61272





--- Comment #18 from Hironori Bono <hbono at chromium.org>  2011-06-23 08:03:19 PST ---
Greetings Darin,

(In reply to comment #17)
> Note, also that WebInputEvent is intended to match up with WebCore::Platform*Event.  You are adding a case where that is not true.
> Also, I see that you are not eliminating WebWidget::*composition methods in favor of using input events for IME.  It just seems to me that we would be better served with an API on WebPlugin for IME composition that mirrors the API we have on WebWidget.
> Perhaps we could even distill that API into a separate interface, which both WebWidget and WebPlugin extend?

Thank you for your suggestion. I would like to update my change to satisfy your constraints. (I do not have any intention to stick to my opinion and to continue providing the worst user-experience to Chromium users.) But, I cannot figure out which option is better: copying the IME interfaces of WebWidget to WebPlugin, or; adding a new interface both to WebWidget and WebPlugin. In my personal opinion, I prefer to just copying the IME interfaces of WebWidget to WebPlugin because it is easier for me to implement it. Nevertheless, I'm afraid you do not like my choice and my work becomes in vain again. (I have already spent lots of time investigating Flash before implementing the current change, without noticing this work becomes in vain.) To prevent working for nothing again, I would like to clarify my choice is reasonable for you and remove any ambiguities. (Unfortunately, I cannot talk with you so often because of time difference.)

Regards,

Hironori Bono

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list