Bug 1273510 part.2 Modify comment in IMMHandler::SetCandidateWindow() r?m_kato draft
authorMasayuki Nakano <masayuki@d-toybox.com>
Tue, 09 Aug 2016 17:46:44 +0900
changeset 401572 0dc0cc421cb59699c4e54d2070365b0febb37d78
parent 401571 837144a262cfbaf4660b7b8b3041ac463564ec00
child 401573 f5f2fda000acc02a4377a106e8320804d94b8bbd
push id26491
push usermasayuki@d-toybox.com
push dateWed, 17 Aug 2016 05:31:28 +0000
reviewersm_kato
bugs1273510
milestone51.0a1
Bug 1273510 part.2 Modify comment in IMMHandler::SetCandidateWindow() r?m_kato MozReview-Commit-ID: KX1frgDFsPi
widget/windows/IMMHandler.cpp
--- a/widget/windows/IMMHandler.cpp
+++ b/widget/windows/IMMHandler.cpp
@@ -2751,18 +2751,19 @@ IMMHandler::OnKeyDownEvent(nsWindow* aWi
       return false;
   }
 }
 
 // static
 void
 IMMHandler::SetCandidateWindow(nsWindow* aWindow, CANDIDATEFORM* aForm)
 {
-  // Hack for ATOK.  ATOK (Japanese IME) refers native caret position at
-  // deciding candidate window position.
+  // Hack for ATOK 2011 - 2016 (Japanese IME).  They refer native caret
+  // position at deciding candidate window position.  Note that we cannot
+  // check active IME since TIPs are wrapped and hidden by CUAS.
   if (aWindow->PluginHasFocus()) {
     // We cannot retrieve proper character height from plugin.  Therefore,
     // we should assume that the caret height is always 20px since if less than
     // this height, candidate window may overlap with composition string when
     // there is no enough space under composition string to show candidate
     // window.
     static const int32_t kCaretHeight = 20;
     if (sNativeCaretIsCreatedForPlugin) {