Closed Bug 316715 Opened 15 years ago Closed 15 years ago
should enable IME on applet and embed elements
This is regression of bug 55751. We should enable IME if applet or embed element have focus.
Status: NEW → ASSIGNED
Thank you, Kimura-san!
Sorry, there are some case that embed element creates box by CSS. We need to check for embed element is loaded plug-in too.
testcase1(object, embed, embed[wmode="opaque"]): http://bugzilla.mozilla.gr.jp/attachment.cgi?id=2974&action=view testcase2(alt content of applet): http://bugzilla.mozilla.gr.jp/attachment.cgi?id=2977&action=view testcase3(alt content of embed): http://bugzilla.mozilla.gr.jp/attachment.cgi?id=2978&action=view
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
+ if (nsObjectLoadingContent::Type() == eType_Plugin) + return nsIContent::IME_STATUS_ENABLE; + return nsIContent::GetDesiredIMEState(); 1) why not just Type(), like in IsFocusable()? 2) why not call the method of the direct superclass (nsGenericHTMLElement, if I recall correctly)?
1. If the method names are conflict by some inherited classes, is this better way? 2. Ah, this may be bad...
(In reply to comment #8) > 1. If the method names are conflict by some inherited classes, is this better > way? Sorry, I don't understand what you mean... is there another class in this hierarchy that has a Type() method? In general, I would leave off the class name, to make the code easier to read.
O.K. I filed bug 316867.
You need to log in before you can comment on or make changes to this bug.