Bug 1796044 - part 9: Make `intl.tsf.hack.ms_japanese_ime.do_not_return_no_layout_error_at_first_char` a static pref r=m_kato

Depends on D159811

Differential Revision: https://phabricator.services.mozilla.com/D159812

modules/libpref/init/StaticPrefList.yaml

@@ -6903,6 +6903,15 @@
     value: true
     mirror: always
 
+  # Whether Gecko returns previous character rect or TS_E_NOLAYOUT from
+  # ITfContextView::GetTextExt() when the specified range is the first
+  # character of selected clause of composition string if and only if Japanese TIP
+  # of Microsoft is active.
+-   name: intl.tsf.hack.ms_japanese_ime.do_not_return_no_layout_error_at_first_char
+    type: bool
+    value: true
+    mirror: always
+
   # Whether Gecko returns caret rect before composition string or TS_E_NOLAYOUT
   # from ITfContextView::GetTextExt() when the specified range is larger than
   # composition start offset if and only Simplified Chinese TIP of Microsoft

modules/libpref/init/all.js

@@ -2560,10 +2560,6 @@ pref("font.size.monospace.x-math", 13);
   // field.
   pref("intl.tsf.hack.atok.search_input_scope_disabled", false);
   // Whether use previous character rect for the result of
-  // ITfContextView::GetTextExt() if the specified range is the first
-  // character of selected clause of composition string.
-  pref("intl.tsf.hack.ms_japanese_ime.do_not_return_no_layout_error_at_first_char", true);
-  // Whether use previous character rect for the result of
   // ITfContextView::GetTextExt() if the specified range is the caret of
   // composition string.
   pref("intl.tsf.hack.ms_japanese_ime.do_not_return_no_layout_error_at_caret", true);

widget/windows/TSFTextStore.cpp

@@ -1753,10 +1753,6 @@ class TSFPrefs final {
     return s##aName##Value;                                                   \
   }
 
-  DECL_AND_IMPL_BOOL_PREF(
-      "intl.tsf.hack.ms_japanese_ime.do_not_return_no_layout_error_at_first_"
-      "char",
-      DoNotReturnNoLayoutErrorToMSJapaneseIMEAtFirstChar, true)
   DECL_AND_IMPL_BOOL_PREF(
       "intl.tsf.hack.ms_japanese_ime.do_not_return_no_layout_error_at_caret",
       DoNotReturnNoLayoutErrorToMSJapaneseIMEAtCaret, true)
@@ -4680,7 +4676,8 @@ bool TSFTextStore::MaybeHackNoErrorLayoutBugs(LONG& aACPStart, LONG& aACPEnd) {
       // available, we should use it as the result.
       // Note that according to bug 1609675, MS-IME for Japanese itself does
       // not handle TS_E_NOLAYOUT correctly at least on Build 18363.657 (1909).
-      if (TSFPrefs::DoNotReturnNoLayoutErrorToMSJapaneseIMEAtFirstChar() &&
+      if (StaticPrefs::
+              intl_tsf_hack_ms_japanese_ime_do_not_return_no_layout_error_at_first_char() &&
           aACPStart < aACPEnd) {
         aACPEnd = aACPStart;
         break;