Bug 1284422 part.2 Fix odd indent of MOZ_LOG() in TSFTextStore.cpp (mainly caused by replacing from PR_LOG()) r?m_kato draft
authorMasayuki Nakano <masayuki@d-toybox.com>
Tue, 05 Jul 2016 18:33:20 +0900
changeset 384516 6bcf89b0fd406043a59146ee41852f8ab9cb7ebd
parent 384515 e65979b78c83a97d6d523b0abe19fae74dca42fd
child 384517 c743ddfc745a407b6d684dfda241f0f24f766fc5
push id22289
push usermasayuki@d-toybox.com
push dateWed, 06 Jul 2016 14:02:00 +0000
reviewersm_kato
bugs1284422
milestone50.0a1
Bug 1284422 part.2 Fix odd indent of MOZ_LOG() in TSFTextStore.cpp (mainly caused by replacing from PR_LOG()) r?m_kato MozReview-Commit-ID: 88EW9lN1DMD
widget/windows/TSFTextStore.cpp
--- a/widget/windows/TSFTextStore.cpp
+++ b/widget/windows/TSFTextStore.cpp
@@ -1023,29 +1023,29 @@ TSFStaticSink::OnActivated(DWORD dwProfi
     mOnActivatedCalled = true;
     mActiveTIPGUID = guidProfile;
     mLangID = langid;
     mIsIMM_IME = IsIMM_IME(hkl);
     GetTIPDescription(rclsid, mLangID, guidProfile,
                       mActiveTIPKeyboardDescription);
   }
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFStaticSink::OnActivated(dwProfileType=%s (0x%08X), "
-          "langid=0x%08X, rclsid=%s, catid=%s, guidProfile=%s, hkl=0x%08X, "
-          "dwFlags=0x%08X (TF_IPSINK_FLAG_ACTIVE: %s)), mIsIMM_IME=%s, "
-          "mActiveTIPDescription=\"%s\"",
-          this, dwProfileType == TF_PROFILETYPE_INPUTPROCESSOR ?
-                  "TF_PROFILETYPE_INPUTPROCESSOR" :
-                dwProfileType == TF_PROFILETYPE_KEYBOARDLAYOUT ?
-                  "TF_PROFILETYPE_KEYBOARDLAYOUT" : "Unknown", dwProfileType,
-          langid, GetCLSIDNameStr(rclsid).get(), GetGUIDNameStr(catid).get(),
-          GetGUIDNameStr(guidProfile).get(), hkl, dwFlags,
-          GetBoolName(dwFlags & TF_IPSINK_FLAG_ACTIVE),
-          GetBoolName(mIsIMM_IME),
-          NS_ConvertUTF16toUTF8(mActiveTIPKeyboardDescription).get()));
+    ("0x%p TSFStaticSink::OnActivated(dwProfileType=%s (0x%08X), "
+     "langid=0x%08X, rclsid=%s, catid=%s, guidProfile=%s, hkl=0x%08X, "
+     "dwFlags=0x%08X (TF_IPSINK_FLAG_ACTIVE: %s)), mIsIMM_IME=%s, "
+     "mActiveTIPDescription=\"%s\"",
+     this, dwProfileType == TF_PROFILETYPE_INPUTPROCESSOR ?
+             "TF_PROFILETYPE_INPUTPROCESSOR" :
+           dwProfileType == TF_PROFILETYPE_KEYBOARDLAYOUT ?
+             "TF_PROFILETYPE_KEYBOARDLAYOUT" : "Unknown", dwProfileType,
+     langid, GetCLSIDNameStr(rclsid).get(), GetGUIDNameStr(catid).get(),
+     GetGUIDNameStr(guidProfile).get(), hkl, dwFlags,
+     GetBoolName(dwFlags & TF_IPSINK_FLAG_ACTIVE),
+     GetBoolName(mIsIMM_IME),
+     NS_ConvertUTF16toUTF8(mActiveTIPKeyboardDescription).get()));
   return S_OK;
 }
 
 bool
 TSFStaticSink::EnsureInitActiveTIPKeyboard()
 {
   if (mOnActivatedCalled) {
     return true;
@@ -1102,19 +1102,19 @@ TSFStaticSink::GetTIPDescription(REFCLSI
   BSTR description = nullptr;
   HRESULT hr =
     mInputProcessorProfiles->GetLanguageProfileDescription(aTextService,
                                                            aLangID,
                                                            aProfile,
                                                            &description);
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFStaticSink::InitActiveTIPDescription() FAILED "
-            "due to GetLanguageProfileDescription() failure, hr=0x%08X",
-            this, hr));
+      ("0x%p   TSFStaticSink::InitActiveTIPDescription() FAILED "
+       "due to GetLanguageProfileDescription() failure, hr=0x%08X",
+       this, hr));
     return;
   }
 
   if (description && description[0]) {
     aDescription.Assign(description);
   }
   ::SysFreeString(description);
 }
@@ -1626,21 +1626,21 @@ TSFTextStore::FlushPendingActions()
        "FAILED due to BeginNativeInputTransaction() failure", this));
     return;
   }
   for (uint32_t i = 0; i < mPendingActions.Length(); i++) {
     PendingAction& action = mPendingActions[i];
     switch (action.mType) {
       case PendingAction::COMPOSITION_START: {
         MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-               ("0x%p   TSFTextStore::FlushPendingActions() "
-                "flushing COMPOSITION_START={ mSelectionStart=%d, "
-                "mSelectionLength=%d }, mDestroyed=%s",
-                this, action.mSelectionStart, action.mSelectionLength,
-                GetBoolName(mDestroyed)));
+          ("0x%p   TSFTextStore::FlushPendingActions() "
+           "flushing COMPOSITION_START={ mSelectionStart=%d, "
+           "mSelectionLength=%d }, mDestroyed=%s",
+           this, action.mSelectionStart, action.mSelectionLength,
+           GetBoolName(mDestroyed)));
 
         if (mDestroyed) {
           MOZ_LOG(sTextStoreLog, LogLevel::Warning,
             ("0x%p   TSFTextStore::FlushPendingActions() "
              "IGNORED pending compositionstart due to already destroyed",
              this));
           break;
         }
@@ -1650,30 +1650,30 @@ TSFTextStore::FlushPendingActions()
           WidgetSelectionEvent selectionSet(true, eSetSelection, mWidget);
           mWidget->InitEvent(selectionSet);
           selectionSet.mOffset = static_cast<uint32_t>(action.mSelectionStart);
           selectionSet.mLength = static_cast<uint32_t>(action.mSelectionLength);
           selectionSet.mReversed = false;
           DispatchEvent(selectionSet);
           if (!selectionSet.mSucceeded) {
             MOZ_LOG(sTextStoreLog, LogLevel::Error,
-                   ("0x%p   TSFTextStore::FlushPendingActions() "
-                    "FAILED due to eSetSelection failure", this));
+              ("0x%p   TSFTextStore::FlushPendingActions() "
+               "FAILED due to eSetSelection failure", this));
             break;
           }
         }
 
         // eCompositionStart always causes
         // NOTIFY_IME_OF_COMPOSITION_EVENT_HANDLED.  Therefore, we should
         // wait to clear mContentForTSF until it's notified.
         mDeferClearingContentForTSF = true;
 
         MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-               ("0x%p   TSFTextStore::FlushPendingActions() "
-                "dispatching compositionstart event...", this));
+          ("0x%p   TSFTextStore::FlushPendingActions() "
+           "dispatching compositionstart event...", this));
         WidgetEventTime eventTime = mWidget->CurrentMessageWidgetEventTime();
         nsEventStatus status;
         rv = mDispatcher->StartComposition(status, &eventTime);
         if (NS_WARN_IF(NS_FAILED(rv))) {
           MOZ_LOG(sTextStoreLog, LogLevel::Error,
             ("0x%p   TSFTextStore::FlushPendingActions() "
              "FAILED to dispatch compositionstart event, "
              "IsComposingInContent()=%s",
@@ -1682,21 +1682,22 @@ TSFTextStore::FlushPendingActions()
         }
         if (!mWidget || mWidget->Destroyed()) {
           break;
         }
         break;
       }
       case PendingAction::COMPOSITION_UPDATE: {
         MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-               ("0x%p   TSFTextStore::FlushPendingActions() "
-                "flushing COMPOSITION_UPDATE={ mData=\"%s\", "
-                "mRanges=0x%p, mRanges->Length()=%d }",
-                this, NS_ConvertUTF16toUTF8(action.mData).get(), action.mRanges.get(),
-                action.mRanges ? action.mRanges->Length() : 0));
+          ("0x%p   TSFTextStore::FlushPendingActions() "
+           "flushing COMPOSITION_UPDATE={ mData=\"%s\", "
+           "mRanges=0x%p, mRanges->Length()=%d }",
+           this, NS_ConvertUTF16toUTF8(action.mData).get(),
+           action.mRanges.get(),
+           action.mRanges ? action.mRanges->Length() : 0));
 
         // eCompositionChange causes a DOM text event, the IME will be notified
         // of NOTIFY_IME_OF_COMPOSITION_EVENT_HANDLED.  In this case, we
         // should not clear mContentForTSF until we notify the IME of the
         // composition update.
         mDeferClearingContentForTSF = true;
 
         rv = mDispatcher->SetPendingComposition(action.mData,
@@ -1724,51 +1725,51 @@ TSFTextStore::FlushPendingActions()
             mDeferClearingContentForTSF = !IsComposingInContent();
           }
           // Be aware, the mWidget might already have been destroyed.
         }
         break;
       }
       case PendingAction::COMPOSITION_END: {
         MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-               ("0x%p   TSFTextStore::FlushPendingActions() "
-                "flushing COMPOSITION_END={ mData=\"%s\" }",
-                this, NS_ConvertUTF16toUTF8(action.mData).get()));
+          ("0x%p   TSFTextStore::FlushPendingActions() "
+           "flushing COMPOSITION_END={ mData=\"%s\" }",
+           this, NS_ConvertUTF16toUTF8(action.mData).get()));
 
         // Dispatching eCompositionCommit causes a DOM text event, then,
         // the IME will be notified of NOTIFY_IME_OF_COMPOSITION_EVENT_HANDLED.
         // In this case, we should not clear mContentForTSFuntil we notify
         // the IME of the composition update.
         mDeferClearingContentForTSF = true;
 
         MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-               ("0x%p   TSFTextStore::FlushPendingActions(), "
-                "dispatching compositioncommit event...", this));
+          ("0x%p   TSFTextStore::FlushPendingActions(), "
+           "dispatching compositioncommit event...", this));
         WidgetEventTime eventTime = mWidget->CurrentMessageWidgetEventTime();
         nsEventStatus status;
         rv = mDispatcher->CommitComposition(status, &action.mData, &eventTime);
         if (NS_WARN_IF(NS_FAILED(rv))) {
           MOZ_LOG(sTextStoreLog, LogLevel::Error,
             ("0x%p   TSFTextStore::FlushPendingActions() "
              "FAILED to dispatch compositioncommit event, "
              "IsComposingInContent()=%s",
              this, GetBoolName(IsComposingInContent())));
           mDeferClearingContentForTSF = !IsComposingInContent();
         }
         break;
       }
       case PendingAction::SET_SELECTION: {
         MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-               ("0x%p   TSFTextStore::FlushPendingActions() "
-                "flushing SET_SELECTION={ mSelectionStart=%d, "
-                "mSelectionLength=%d, mSelectionReversed=%s }, "
-                "mDestroyed=%s",
-                this, action.mSelectionStart, action.mSelectionLength,
-                GetBoolName(action.mSelectionReversed),
-                GetBoolName(mDestroyed)));
+          ("0x%p   TSFTextStore::FlushPendingActions() "
+           "flushing SET_SELECTION={ mSelectionStart=%d, "
+           "mSelectionLength=%d, mSelectionReversed=%s }, "
+           "mDestroyed=%s",
+           this, action.mSelectionStart, action.mSelectionLength,
+           GetBoolName(action.mSelectionReversed),
+           GetBoolName(mDestroyed)));
 
         if (mDestroyed) {
           MOZ_LOG(sTextStoreLog, LogLevel::Warning,
             ("0x%p   TSFTextStore::FlushPendingActions() "
              "IGNORED pending selectionset due to already destroyed",
              this));
           break;
         }
@@ -1785,18 +1786,18 @@ TSFTextStore::FlushPendingActions()
         MOZ_CRASH("unexpected action type");
     }
 
     if (mWidget && !mWidget->Destroyed()) {
       continue;
     }
 
     MOZ_LOG(sTextStoreLog, LogLevel::Info,
-           ("0x%p   TSFTextStore::FlushPendingActions(), "
-            "qutting since the mWidget has gone", this));
+      ("0x%p   TSFTextStore::FlushPendingActions(), "
+       "qutting since the mWidget has gone", this));
     break;
   }
   mPendingActions.Clear();
 }
 
 void
 TSFTextStore::MaybeFlushPendingNotifications()
 {
@@ -1819,19 +1820,19 @@ TSFTextStore::MaybeFlushPendingNotificat
       ("0x%p   TSFTextStore::MaybeFlushPendingNotifications(), "
        "calling TSFTextStore::CommitCompositionInternal(true)...", this));
     mDeferCommittingComposition = mDeferCancellingComposition = false;
     CommitCompositionInternal(true);
   }
 
   if (mDeferNotifyingTSF) {
     MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-           ("0x%p   TSFTextStore::MaybeFlushPendingNotifications(), "
-            "putting off flushing pending notifications due to being "
-            "dispatching events...", this));
+      ("0x%p   TSFTextStore::MaybeFlushPendingNotifications(), "
+       "putting off flushing pending notifications due to being "
+       "dispatching events...", this));
     return;
   }
 
   if (mPendingDestroy) {
     Destroy();
     return;
   }
 
@@ -1842,41 +1843,41 @@ TSFTextStore::MaybeFlushPendingNotificat
       ("0x%p   TSFTextStore::MaybeFlushPendingNotifications(), "
        "does nothing because this has already destroyed completely...", this));
     return;
   }
 
   if (!mDeferClearingContentForTSF && mContentForTSF.IsInitialized()) {
     mContentForTSF.Clear();
     MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-           ("0x%p   TSFTextStore::MaybeFlushPendingNotifications(), "
-            "mContentForTSF is cleared", this));
+      ("0x%p   TSFTextStore::MaybeFlushPendingNotifications(), "
+       "mContentForTSF is cleared", this));
   }
 
   // When there is no cached content, we can sync actual contents and TSF/TIP
   // expecting contents.
   if (!mContentForTSF.IsInitialized()) {
     if (mPendingTextChangeData.IsValid()) {
       MOZ_LOG(sTextStoreLog, LogLevel::Info,
-             ("0x%p   TSFTextStore::MaybeFlushPendingNotifications(), "
-              "calling TSFTextStore::NotifyTSFOfTextChange()...", this));
+        ("0x%p   TSFTextStore::MaybeFlushPendingNotifications(), "
+         "calling TSFTextStore::NotifyTSFOfTextChange()...", this));
       NotifyTSFOfTextChange();
     }
     if (mPendingSelectionChangeData.IsValid()) {
       MOZ_LOG(sTextStoreLog, LogLevel::Info,
-             ("0x%p   TSFTextStore::MaybeFlushPendingNotifications(), "
-              "calling TSFTextStore::NotifyTSFOfSelectionChange()...", this));
+        ("0x%p   TSFTextStore::MaybeFlushPendingNotifications(), "
+         "calling TSFTextStore::NotifyTSFOfSelectionChange()...", this));
       NotifyTSFOfSelectionChange();
     }
   }
 
   if (mHasReturnedNoLayoutError) {
     MOZ_LOG(sTextStoreLog, LogLevel::Info,
-           ("0x%p   TSFTextStore::MaybeFlushPendingNotifications(), "
-            "calling TSFTextStore::NotifyTSFOfLayoutChange()...", this));
+      ("0x%p   TSFTextStore::MaybeFlushPendingNotifications(), "
+       "calling TSFTextStore::NotifyTSFOfLayoutChange()...", this));
     NotifyTSFOfLayoutChange();
   }
 }
 
 STDMETHODIMP
 TSFTextStore::GetStatus(TS_STATUS* pdcs)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
@@ -1896,106 +1897,106 @@ TSFTextStore::GetStatus(TS_STATUS* pdcs)
 STDMETHODIMP
 TSFTextStore::QueryInsert(LONG acpTestStart,
                           LONG acpTestEnd,
                           ULONG cch,
                           LONG* pacpResultStart,
                           LONG* pacpResultEnd)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::QueryInsert(acpTestStart=%ld, "
-          "acpTestEnd=%ld, cch=%lu, pacpResultStart=0x%p, pacpResultEnd=0x%p)",
-          this, acpTestStart, acpTestEnd, cch, acpTestStart, acpTestEnd));
+    ("0x%p TSFTextStore::QueryInsert(acpTestStart=%ld, "
+     "acpTestEnd=%ld, cch=%lu, pacpResultStart=0x%p, pacpResultEnd=0x%p)",
+     this, acpTestStart, acpTestEnd, cch, acpTestStart, acpTestEnd));
 
   if (!pacpResultStart || !pacpResultEnd) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::QueryInsert() FAILED due to "
-            "the null argument", this));
+      ("0x%p   TSFTextStore::QueryInsert() FAILED due to "
+       "the null argument", this));
     return E_INVALIDARG;
   }
 
   if (acpTestStart < 0 || acpTestStart > acpTestEnd) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::QueryInsert() FAILED due to "
-            "wrong argument", this));
+      ("0x%p   TSFTextStore::QueryInsert() FAILED due to "
+       "wrong argument", this));
     return E_INVALIDARG;
   }
 
   // XXX need to adjust to cluster boundary
   // Assume we are given good offsets for now
   const TSFStaticSink* kSink = TSFStaticSink::GetInstance();
   if (IsWin8OrLater() && !mComposition.IsComposing() &&
       ((sHackQueryInsertForMSTraditionalTIP &&
          (kSink->IsMSChangJieActive() || kSink->IsMSQuickQuickActive())) ||
        (sHackQueryInsertForMSSimplifiedTIP &&
          (kSink->IsMSPinyinActive() || kSink->IsMSWubiActive())))) {
     MOZ_LOG(sTextStoreLog, LogLevel::Warning,
-            ("0x%p   TSFTextStore::QueryInsert() WARNING using different "
-             "result for the TIP", this));
+      ("0x%p   TSFTextStore::QueryInsert() WARNING using different "
+       "result for the TIP", this));
     // Chinese TIPs of Microsoft assume that QueryInsert() returns selected
     // range which should be removed.
     *pacpResultStart = acpTestStart;
     *pacpResultEnd = acpTestEnd;
   } else {
     *pacpResultStart = acpTestStart;
     *pacpResultEnd = acpTestStart + cch;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p  TSFTextStore::QueryInsert() succeeded: "
-          "*pacpResultStart=%ld, *pacpResultEnd=%ld)",
-          this, *pacpResultStart, *pacpResultEnd));
+    ("0x%p  TSFTextStore::QueryInsert() succeeded: "
+     "*pacpResultStart=%ld, *pacpResultEnd=%ld)",
+     this, *pacpResultStart, *pacpResultEnd));
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::GetSelection(ULONG ulIndex,
                            ULONG ulCount,
                            TS_SELECTION_ACP* pSelection,
                            ULONG* pcFetched)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::GetSelection(ulIndex=%lu, ulCount=%lu, "
-          "pSelection=0x%p, pcFetched=0x%p)",
-          this, ulIndex, ulCount, pSelection, pcFetched));
+    ("0x%p TSFTextStore::GetSelection(ulIndex=%lu, ulCount=%lu, "
+     "pSelection=0x%p, pcFetched=0x%p)",
+     this, ulIndex, ulCount, pSelection, pcFetched));
 
   if (!IsReadLocked()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetSelection() FAILED due to not locked",
-            this));
+      ("0x%p   TSFTextStore::GetSelection() FAILED due to not locked",
+       this));
     return TS_E_NOLOCK;
   }
   if (!ulCount || !pSelection || !pcFetched) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetSelection() FAILED due to "
-            "null argument", this));
+      ("0x%p   TSFTextStore::GetSelection() FAILED due to "
+       "null argument", this));
     return E_INVALIDARG;
   }
 
   *pcFetched = 0;
 
   if (ulIndex != static_cast<ULONG>(TS_DEFAULT_SELECTION) &&
       ulIndex != 0) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetSelection() FAILED due to "
-            "unsupported selection", this));
+      ("0x%p   TSFTextStore::GetSelection() FAILED due to "
+       "unsupported selection", this));
     return TS_E_NOSELECTION;
   }
 
   Selection& selectionForTSF = SelectionForTSFRef();
   if (selectionForTSF.IsDirty()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetSelection() FAILED due to "
-            "SelectionForTSFRef() failure", this));
+      ("0x%p   TSFTextStore::GetSelection() FAILED due to "
+       "SelectionForTSFRef() failure", this));
     return E_FAIL;
   }
   *pSelection = selectionForTSF.ACP();
   *pcFetched = 1;
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::GetSelection() succeeded", this));
+    ("0x%p   TSFTextStore::GetSelection() succeeded", this));
   return S_OK;
 }
 
 bool
 TSFTextStore::IsComposingInContent() const
 {
   if (!mDispatcher) {
     return false;
@@ -2008,91 +2009,91 @@ TSFTextStore::IsComposingInContent() con
 
 TSFTextStore::Content&
 TSFTextStore::ContentForTSFRef()
 {
   // This should be called when the document is locked or the content hasn't
   // been abandoned yet.
   if (NS_WARN_IF(!IsReadLocked() && !mContentForTSF.IsInitialized())) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::ContentForTSFRef(), FAILED, due to "
-            "called wrong timing, IsReadLocked()=%s, "
-            "mContentForTSF.IsInitialized()=%s",
-            this, GetBoolName(IsReadLocked()),
-            GetBoolName(mContentForTSF.IsInitialized())));
+      ("0x%p   TSFTextStore::ContentForTSFRef(), FAILED, due to "
+       "called wrong timing, IsReadLocked()=%s, "
+       "mContentForTSF.IsInitialized()=%s",
+       this, GetBoolName(IsReadLocked()),
+       GetBoolName(mContentForTSF.IsInitialized())));
     mContentForTSF.Clear();
     return mContentForTSF;
   }
 
   Selection& selectionForTSF = SelectionForTSFRef();
   if (selectionForTSF.IsDirty()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::ContentForTSFRef(), FAILED, due to "
-            "SelectionForTSFRef() failure", this));
+      ("0x%p   TSFTextStore::ContentForTSFRef(), FAILED, due to "
+       "SelectionForTSFRef() failure", this));
     mContentForTSF.Clear();
     return mContentForTSF;
   }
 
   if (!mContentForTSF.IsInitialized()) {
     nsAutoString text;
     if (NS_WARN_IF(!GetCurrentText(text))) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-             ("0x%p   TSFTextStore::ContentForTSFRef(), FAILED, due to "
-              "GetCurrentText() failure", this));
+        ("0x%p   TSFTextStore::ContentForTSFRef(), FAILED, due to "
+         "GetCurrentText() failure", this));
       mContentForTSF.Clear();
       return mContentForTSF;
     }
 
     mContentForTSF.Init(text);
     // Basically, the cached content which is expected by TSF/TIP should be
     // cleared after active composition is committed or the document lock is
     // unlocked.  However, in e10s mode, content will be modified
     // asynchronously.  In such case, mDeferClearingContentForTSF may be
     // true until whole dispatched events are handled by the focused editor.
     mDeferClearingContentForTSF = false;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::ContentForTSFRef(): "
-          "mContentForTSF={ mText=\"%s\" (Length()=%u), "
-          "mLastCompositionString=\"%s\" (Length()=%u), "
-          "mMinTextModifiedOffset=%u }",
-          this, mContentForTSF.Text().Length() <= 20 ?
-            NS_ConvertUTF16toUTF8(mContentForTSF.Text()).get() : "<omitted>",
-          mContentForTSF.Text().Length(),
-          NS_ConvertUTF16toUTF8(mContentForTSF.LastCompositionString()).get(),
-          mContentForTSF.LastCompositionString().Length(),
-          mContentForTSF.MinTextModifiedOffset()));
+    ("0x%p   TSFTextStore::ContentForTSFRef(): "
+     "mContentForTSF={ mText=\"%s\" (Length()=%u), "
+     "mLastCompositionString=\"%s\" (Length()=%u), "
+     "mMinTextModifiedOffset=%u }",
+     this, mContentForTSF.Text().Length() <= 20 ?
+       NS_ConvertUTF16toUTF8(mContentForTSF.Text()).get() : "<omitted>",
+     mContentForTSF.Text().Length(),
+     NS_ConvertUTF16toUTF8(mContentForTSF.LastCompositionString()).get(),
+     mContentForTSF.LastCompositionString().Length(),
+     mContentForTSF.MinTextModifiedOffset()));
 
   return mContentForTSF;
 }
 
 bool
 TSFTextStore::GetCurrentText(nsAString& aTextContent)
 {
   if (mContentForTSF.IsInitialized()) {
     aTextContent = mContentForTSF.Text();
     return true;
   }
 
   MOZ_ASSERT(!mDestroyed);
   MOZ_ASSERT(mWidget && !mWidget->Destroyed());
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::GetCurrentText(): "
-          "retrieving text from the content...", this));
+    ("0x%p   TSFTextStore::GetCurrentText(): "
+     "retrieving text from the content...", this));
 
   WidgetQueryContentEvent queryText(true, eQueryTextContent, mWidget);
   queryText.InitForQueryTextContent(0, UINT32_MAX);
   mWidget->InitEvent(queryText);
   DispatchEvent(queryText);
   if (NS_WARN_IF(!queryText.mSucceeded)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetCurrentText(), FAILED, due to "
-            "eQueryTextContent failure", this));
+      ("0x%p   TSFTextStore::GetCurrentText(), FAILED, due to "
+       "eQueryTextContent failure", this));
     aTextContent.Truncate();
     return false;
   }
 
   aTextContent = queryText.mReply.mString;
   return true;
 }
 
@@ -2116,21 +2117,21 @@ TSFTextStore::SelectionForTSFRef()
 
     mSelectionForTSF.SetSelection(querySelection.mReply.mOffset,
                                   querySelection.mReply.mString.Length(),
                                   querySelection.mReply.mReversed,
                                   querySelection.GetWritingMode());
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::SelectionForTSFRef(): "
-          "acpStart=%d, acpEnd=%d (length=%d), reverted=%s",
-          this, mSelectionForTSF.StartOffset(), mSelectionForTSF.EndOffset(),
-          mSelectionForTSF.Length(),
-          GetBoolName(mSelectionForTSF.IsReversed())));
+    ("0x%p   TSFTextStore::SelectionForTSFRef(): "
+     "acpStart=%d, acpEnd=%d (length=%d), reverted=%s",
+     this, mSelectionForTSF.StartOffset(), mSelectionForTSF.EndOffset(),
+     mSelectionForTSF.Length(),
+     GetBoolName(mSelectionForTSF.IsReversed())));
 
   return mSelectionForTSF;
 }
 
 static HRESULT
 GetRangeExtent(ITfRange* aRange, LONG* aStart, LONG* aLength)
 {
   RefPtr<ITfRangeACP> rangeACP;
@@ -2164,171 +2165,171 @@ TSFTextStore::GetDisplayAttribute(ITfPro
   NS_ENSURE_TRUE(aResult, E_FAIL);
 
   HRESULT hr;
 
   if (MOZ_LOG_TEST(sTextStoreLog, LogLevel::Debug)) {
     LONG start = 0, length = 0;
     hr = GetRangeExtent(aRange, &start, &length);
     MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-           ("0x%p   TSFTextStore::GetDisplayAttribute(): "
-            "GetDisplayAttribute range=%ld-%ld (hr=%s)",
-            this, start - mComposition.mStart,
-            start - mComposition.mStart + length,
-            GetCommonReturnValueName(hr)));
+      ("0x%p   TSFTextStore::GetDisplayAttribute(): "
+       "GetDisplayAttribute range=%ld-%ld (hr=%s)",
+       this, start - mComposition.mStart,
+       start - mComposition.mStart + length,
+       GetCommonReturnValueName(hr)));
   }
 
   VARIANT propValue;
   ::VariantInit(&propValue);
   hr = aAttrProperty->GetValue(TfEditCookie(mEditCookie), aRange, &propValue);
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetDisplayAttribute() FAILED due to "
-            "ITfProperty::GetValue() failed", this));
+      ("0x%p   TSFTextStore::GetDisplayAttribute() FAILED due to "
+       "ITfProperty::GetValue() failed", this));
     return hr;
   }
   if (VT_I4 != propValue.vt) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetDisplayAttribute() FAILED due to "
-            "ITfProperty::GetValue() returns non-VT_I4 value", this));
+      ("0x%p   TSFTextStore::GetDisplayAttribute() FAILED due to "
+       "ITfProperty::GetValue() returns non-VT_I4 value", this));
     ::VariantClear(&propValue);
     return E_FAIL;
   }
 
   NS_ENSURE_TRUE(sCategoryMgr, E_FAIL);
   GUID guid;
   hr = sCategoryMgr->GetGUID(DWORD(propValue.lVal), &guid);
   ::VariantClear(&propValue);
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetDisplayAttribute() FAILED due to "
-            "ITfCategoryMgr::GetGUID() failed", this));
+      ("0x%p   TSFTextStore::GetDisplayAttribute() FAILED due to "
+       "ITfCategoryMgr::GetGUID() failed", this));
     return hr;
   }
 
   NS_ENSURE_TRUE(sDisplayAttrMgr, E_FAIL);
   RefPtr<ITfDisplayAttributeInfo> info;
   hr = sDisplayAttrMgr->GetDisplayAttributeInfo(guid, getter_AddRefs(info),
                                                 nullptr);
   if (FAILED(hr) || !info) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetDisplayAttribute() FAILED due to "
-            "ITfDisplayAttributeMgr::GetDisplayAttributeInfo() failed", this));
+      ("0x%p   TSFTextStore::GetDisplayAttribute() FAILED due to "
+       "ITfDisplayAttributeMgr::GetDisplayAttributeInfo() failed", this));
     return hr;
   }
 
   hr = info->GetAttributeInfo(aResult);
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetDisplayAttribute() FAILED due to "
-            "ITfDisplayAttributeInfo::GetAttributeInfo() failed", this));
+      ("0x%p   TSFTextStore::GetDisplayAttribute() FAILED due to "
+       "ITfDisplayAttributeInfo::GetAttributeInfo() failed", this));
     return hr;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::GetDisplayAttribute() succeeded: "
-          "Result={ %s }", this, GetDisplayAttrStr(*aResult).get()));
+    ("0x%p   TSFTextStore::GetDisplayAttribute() succeeded: "
+     "Result={ %s }", this, GetDisplayAttrStr(*aResult).get()));
   return S_OK;
 }
 
 HRESULT
 TSFTextStore::RestartCompositionIfNecessary(ITfRange* aRangeNew)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::RestartCompositionIfNecessary("
-          "aRangeNew=0x%p), mComposition.mView=0x%p",
-          this, aRangeNew, mComposition.mView.get()));
+    ("0x%p   TSFTextStore::RestartCompositionIfNecessary("
+     "aRangeNew=0x%p), mComposition.mView=0x%p",
+     this, aRangeNew, mComposition.mView.get()));
 
   if (!mComposition.IsComposing()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::RestartCompositionIfNecessary() FAILED "
-            "due to no composition view", this));
+      ("0x%p   TSFTextStore::RestartCompositionIfNecessary() FAILED "
+       "due to no composition view", this));
     return E_FAIL;
   }
 
   HRESULT hr;
   RefPtr<ITfCompositionView> pComposition(mComposition.mView);
   RefPtr<ITfRange> composingRange(aRangeNew);
   if (!composingRange) {
     hr = pComposition->GetRange(getter_AddRefs(composingRange));
     if (FAILED(hr)) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-             ("0x%p   TSFTextStore::RestartCompositionIfNecessary() "
-              "FAILED due to pComposition->GetRange() failure", this));
+        ("0x%p   TSFTextStore::RestartCompositionIfNecessary() "
+         "FAILED due to pComposition->GetRange() failure", this));
       return hr;
     }
   }
 
   // Get starting offset of the composition
   LONG compStart = 0, compLength = 0;
   hr = GetRangeExtent(composingRange, &compStart, &compLength);
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::RestartCompositionIfNecessary() FAILED "
-            "due to GetRangeExtent() failure", this));
+      ("0x%p   TSFTextStore::RestartCompositionIfNecessary() FAILED "
+       "due to GetRangeExtent() failure", this));
     return hr;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::RestartCompositionIfNecessary(), "
-          "range=%ld-%ld, mComposition={ mStart=%ld, mString.Length()=%lu }",
-          this, compStart, compStart + compLength, mComposition.mStart,
-          mComposition.mString.Length()));
+    ("0x%p   TSFTextStore::RestartCompositionIfNecessary(), "
+     "range=%ld-%ld, mComposition={ mStart=%ld, mString.Length()=%lu }",
+     this, compStart, compStart + compLength, mComposition.mStart,
+     mComposition.mString.Length()));
 
   if (mComposition.mStart != compStart ||
       mComposition.mString.Length() != (ULONG)compLength) {
     // If the queried composition length is different from the length
     // of our composition string, OnUpdateComposition is being called
     // because a part of the original composition was committed.
     hr = RestartComposition(pComposition, composingRange);
     if (FAILED(hr)) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-             ("0x%p   TSFTextStore::RestartCompositionIfNecessary() "
-              "FAILED due to RestartComposition() failure", this));
+        ("0x%p   TSFTextStore::RestartCompositionIfNecessary() "
+         "FAILED due to RestartComposition() failure", this));
       return hr;
     }
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::RestartCompositionIfNecessary() succeeded",
-          this));
+    ("0x%p   TSFTextStore::RestartCompositionIfNecessary() succeeded",
+     this));
   return S_OK;
 }
 
 HRESULT
 TSFTextStore::RestartComposition(ITfCompositionView* aCompositionView,
                                  ITfRange* aNewRange)
 {
   Selection& selectionForTSF = SelectionForTSFRef();
 
   LONG newStart, newLength;
   HRESULT hr = GetRangeExtent(aNewRange, &newStart, &newLength);
   LONG newEnd = newStart + newLength;
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::RestartComposition(aCompositionView=0x%p, "
-          "aNewRange=0x%p { newStart=%d, newLength=%d }), "
-          "mComposition={ mStart=%d, mCompositionString.Length()=%d }, "
-          "selectionForTSF={ IsDirty()=%s, StartOffset()=%d, Length()=%d }",
-          this, aCompositionView, aNewRange, newStart, newLength,
-          mComposition.mStart, mComposition.mString.Length(),
-          GetBoolName(selectionForTSF.IsDirty()),
-          selectionForTSF.StartOffset(), selectionForTSF.Length()));
+    ("0x%p   TSFTextStore::RestartComposition(aCompositionView=0x%p, "
+     "aNewRange=0x%p { newStart=%d, newLength=%d }), "
+     "mComposition={ mStart=%d, mCompositionString.Length()=%d }, "
+     "selectionForTSF={ IsDirty()=%s, StartOffset()=%d, Length()=%d }",
+     this, aCompositionView, aNewRange, newStart, newLength,
+     mComposition.mStart, mComposition.mString.Length(),
+     GetBoolName(selectionForTSF.IsDirty()),
+     selectionForTSF.StartOffset(), selectionForTSF.Length()));
 
   if (selectionForTSF.IsDirty()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::RestartComposition() FAILED "
-            "due to SelectionForTSFRef() failure", this));
+      ("0x%p   TSFTextStore::RestartComposition() FAILED "
+       "due to SelectionForTSFRef() failure", this));
     return E_FAIL;
   }
 
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::RestartComposition() FAILED "
-            "due to GetRangeExtent() failure", this));
+      ("0x%p   TSFTextStore::RestartComposition() FAILED "
+       "due to GetRangeExtent() failure", this));
     return hr;
   }
 
   // If the new range has no overlap with the crrent range, we just commit
   // the composition and restart new composition with the new range but
   // current selection range should be preserved.
   if (newStart >= mComposition.EndOffset() || newEnd <= mComposition.mStart) {
     RecordCompositionEndAction();
@@ -2354,18 +2355,18 @@ TSFTextStore::RestartComposition(ITfComp
   // Remove the overlapped part from the commit string.
   nsAutoString commitString(mComposition.mString);
   commitString.Cut(keepComposingStartOffset - mComposition.mStart,
                    keepComposingLength);
   // Update the composition string.
   Content& contentForTSF = ContentForTSFRef();
   if (!contentForTSF.IsInitialized()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::RestartComposition() FAILED "
-            "due to ContentForTSFRef() failure", this));
+      ("0x%p   TSFTextStore::RestartComposition() FAILED "
+       "due to ContentForTSFRef() failure", this));
     return E_FAIL;
   }
   contentForTSF.ReplaceTextWith(mComposition.mStart,
                                 mComposition.mString.Length(),
                                 commitString);
   // Record a compositionupdate action for commit the part of composing string.
   PendingAction* action = LastOrNewPendingCompositionUpdate();
   action->mData = mComposition.mString;
@@ -2387,22 +2388,22 @@ TSFTextStore::RestartComposition(ITfComp
   // Restore the latest text content and selection.
   contentForTSF.ReplaceSelectedTextWith(
     nsDependentSubstring(oldComposition.mString,
                          keepComposingStartOffset - oldComposition.mStart,
                          keepComposingLength));
   selectionForTSF = oldSelection;
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::RestartComposition() succeeded, "
-          "mComposition={ mStart=%d, mCompositionString.Length()=%d }, "
-          "selectionForTSF={ IsDirty()=%s, StartOffset()=%d, Length()=%d }",
-          this, mComposition.mStart, mComposition.mString.Length(),
-          GetBoolName(selectionForTSF.IsDirty()),
-          selectionForTSF.StartOffset(), selectionForTSF.Length()));
+    ("0x%p   TSFTextStore::RestartComposition() succeeded, "
+     "mComposition={ mStart=%d, mCompositionString.Length()=%d }, "
+     "selectionForTSF={ IsDirty()=%s, StartOffset()=%d, Length()=%d }",
+     this, mComposition.mStart, mComposition.mString.Length(),
+     GetBoolName(selectionForTSF.IsDirty()),
+     selectionForTSF.StartOffset(), selectionForTSF.Length()));
 
   return S_OK;
 }
 
 static bool
 GetColor(const TF_DA_COLOR& aTSFColor, nscolor& aResult)
 {
   switch (aTSFColor.type) {
@@ -2445,73 +2446,73 @@ GetLineStyle(TF_DA_LINESTYLE aTSFLineSty
       return false;
   }
 }
 
 HRESULT
 TSFTextStore::RecordCompositionUpdateAction()
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::RecordCompositionUpdateAction(), "
-          "mComposition={ mView=0x%p, mStart=%d, mString=\"%s\" "
-          "(Length()=%d) }",
-          this, mComposition.mView.get(), mComposition.mStart,
-          NS_ConvertUTF16toUTF8(mComposition.mString).get(),
-          mComposition.mString.Length()));
+    ("0x%p   TSFTextStore::RecordCompositionUpdateAction(), "
+     "mComposition={ mView=0x%p, mStart=%d, mString=\"%s\" "
+     "(Length()=%d) }",
+     this, mComposition.mView.get(), mComposition.mStart,
+     NS_ConvertUTF16toUTF8(mComposition.mString).get(),
+     mComposition.mString.Length()));
 
   if (!mComposition.IsComposing()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::RecordCompositionUpdateAction() FAILED "
-            "due to no composition view", this));
+      ("0x%p   TSFTextStore::RecordCompositionUpdateAction() FAILED "
+       "due to no composition view", this));
     return E_FAIL;
   }
 
   // Getting display attributes is *really* complicated!
   // We first get the context and the property objects to query for
   // attributes, but since a big range can have a variety of values for
   // the attribute, we have to find out all the ranges that have distinct
   // attribute values. Then we query for what the value represents through
   // the display attribute manager and translate that to TextRange to be
   // sent in eCompositionChange
 
   RefPtr<ITfProperty> attrPropetry;
   HRESULT hr = mContext->GetProperty(GUID_PROP_ATTRIBUTE,
                                      getter_AddRefs(attrPropetry));
   if (FAILED(hr) || !attrPropetry) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::RecordCompositionUpdateAction() FAILED "
-            "due to mContext->GetProperty() failure", this));
+      ("0x%p   TSFTextStore::RecordCompositionUpdateAction() FAILED "
+       "due to mContext->GetProperty() failure", this));
     return FAILED(hr) ? hr : E_FAIL;
   }
 
   RefPtr<ITfRange> composingRange;
   hr = mComposition.mView->GetRange(getter_AddRefs(composingRange));
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::RecordCompositionUpdateAction() "
-            "FAILED due to mComposition.mView->GetRange() failure", this));
+      ("0x%p   TSFTextStore::RecordCompositionUpdateAction() "
+       "FAILED due to mComposition.mView->GetRange() failure", this));
     return hr;
   }
 
   RefPtr<IEnumTfRanges> enumRanges;
   hr = attrPropetry->EnumRanges(TfEditCookie(mEditCookie),
                                 getter_AddRefs(enumRanges), composingRange);
   if (FAILED(hr) || !enumRanges) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::RecordCompositionUpdateAction() FAILED "
-            "due to attrPropetry->EnumRanges() failure", this));
+      ("0x%p   TSFTextStore::RecordCompositionUpdateAction() FAILED "
+       "due to attrPropetry->EnumRanges() failure", this));
     return FAILED(hr) ? hr : E_FAIL;
   }
 
   // First, put the log of content and selection here.
   Selection& selectionForTSF = SelectionForTSFRef();
   if (selectionForTSF.IsDirty()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::RecordCompositionUpdateAction() FAILED "
-            "due to SelectionForTSFRef() failure", this));
+      ("0x%p   TSFTextStore::RecordCompositionUpdateAction() FAILED "
+       "due to SelectionForTSFRef() failure", this));
     return E_FAIL;
   }
 
   PendingAction* action = LastOrNewPendingCompositionUpdate();
   action->mData = mComposition.mString;
   // The ranges might already have been initialized, however, if this is
   // called again, that means we need to overwrite the ranges with current
   // information.
@@ -2537,29 +2538,29 @@ TSFTextStore::RecordCompositionUpdateAct
     LONG start = std::min(std::max(rangeStart, mComposition.mStart),
                           mComposition.EndOffset());
     LONG end = std::max(std::min(rangeStart + rangeLength,
                                  mComposition.EndOffset()),
                         mComposition.mStart);
     LONG length = end - start;
     if (length < 0) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-             ("0x%p   TSFTextStore::RecordCompositionUpdateAction() "
-              "ignores invalid range (%d-%d)",
-              this, rangeStart - mComposition.mStart,
-              rangeStart - mComposition.mStart + rangeLength));
+        ("0x%p   TSFTextStore::RecordCompositionUpdateAction() "
+         "ignores invalid range (%d-%d)",
+         this, rangeStart - mComposition.mStart,
+         rangeStart - mComposition.mStart + rangeLength));
       continue;
     }
     if (!length) {
       MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-             ("0x%p   TSFTextStore::RecordCompositionUpdateAction() "
-              "ignores a range due to outside of the composition or empty "
-              "(%d-%d)",
-              this, rangeStart - mComposition.mStart,
-              rangeStart - mComposition.mStart + rangeLength));
+        ("0x%p   TSFTextStore::RecordCompositionUpdateAction() "
+         "ignores a range due to outside of the composition or empty "
+         "(%d-%d)",
+         this, rangeStart - mComposition.mStart,
+         rangeStart - mComposition.mStart + rangeLength));
       continue;
     }
 
     TextRange newRange;
     newRange.mStartOffset = uint32_t(start - mComposition.mStart);
     // The end of the last range in the array is
     // always kept at the end of composition
     newRange.mEndOffset = mComposition.mString.Length();
@@ -2637,82 +2638,82 @@ TSFTextStore::RecordCompositionUpdateAct
     caretRange.mStartOffset = caretRange.mEndOffset = caretPosition;
     caretRange.mRangeType = TextRangeType::eCaret;
     action->mRanges->AppendElement(caretRange);
   }
 
   action->mIncomplete = false;
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::RecordCompositionUpdateAction() "
-          "succeeded", this));
+    ("0x%p   TSFTextStore::RecordCompositionUpdateAction() "
+     "succeeded", this));
 
   return S_OK;
 }
 
 HRESULT
 TSFTextStore::SetSelectionInternal(const TS_SELECTION_ACP* pSelection,
                                    bool aDispatchCompositionChangeEvent)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::SetSelectionInternal(pSelection={ "
-          "acpStart=%ld, acpEnd=%ld, style={ ase=%s, fInterimChar=%s} }, "
-          "aDispatchCompositionChangeEvent=%s), mComposition.IsComposing()=%s",
-          this, pSelection->acpStart, pSelection->acpEnd,
-          GetActiveSelEndName(pSelection->style.ase),
-          GetBoolName(pSelection->style.fInterimChar),
-          GetBoolName(aDispatchCompositionChangeEvent),
-          GetBoolName(mComposition.IsComposing())));
+    ("0x%p   TSFTextStore::SetSelectionInternal(pSelection={ "
+     "acpStart=%ld, acpEnd=%ld, style={ ase=%s, fInterimChar=%s} }, "
+     "aDispatchCompositionChangeEvent=%s), mComposition.IsComposing()=%s",
+     this, pSelection->acpStart, pSelection->acpEnd,
+     GetActiveSelEndName(pSelection->style.ase),
+     GetBoolName(pSelection->style.fInterimChar),
+     GetBoolName(aDispatchCompositionChangeEvent),
+     GetBoolName(mComposition.IsComposing())));
 
   MOZ_ASSERT(IsReadWriteLocked());
 
   Selection& selectionForTSF = SelectionForTSFRef();
   if (selectionForTSF.IsDirty()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-       ("0x%p   TSFTextStore::SetSelectionInternal() FAILED due to "
-        "SelectionForTSFRef() failure", this));
+      ("0x%p   TSFTextStore::SetSelectionInternal() FAILED due to "
+       "SelectionForTSFRef() failure", this));
     return E_FAIL;
   }
 
   // If actually the range is not changing, we should do nothing.
   // Perhaps, we can ignore the difference change because it must not be
   // important for following edit.
   if (selectionForTSF.EqualsExceptDirection(*pSelection)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-       ("0x%p   TSFTextStore::SetSelectionInternal() Succeeded but "
-        "did nothing because the selection range isn't changing", this));
+      ("0x%p   TSFTextStore::SetSelectionInternal() Succeeded but "
+       "did nothing because the selection range isn't changing", this));
     selectionForTSF.SetSelection(*pSelection);
     return S_OK;
   }
 
   if (mComposition.IsComposing()) {
     if (aDispatchCompositionChangeEvent) {
       HRESULT hr = RestartCompositionIfNecessary();
       if (FAILED(hr)) {
         MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::SetSelectionInternal() FAILED due to "
-            "RestartCompositionIfNecessary() failure", this));
+          ("0x%p   TSFTextStore::SetSelectionInternal() FAILED due to "
+           "RestartCompositionIfNecessary() failure", this));
         return hr;
       }
     }
     if (pSelection->acpStart < mComposition.mStart ||
         pSelection->acpEnd > mComposition.EndOffset()) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-         ("0x%p   TSFTextStore::SetSelectionInternal() FAILED due to "
-          "the selection being out of the composition string", this));
+        ("0x%p   TSFTextStore::SetSelectionInternal() FAILED due to "
+         "the selection being out of the composition string", this));
       return TS_E_INVALIDPOS;
     }
     // Emulate selection during compositions
     selectionForTSF.SetSelection(*pSelection);
     if (aDispatchCompositionChangeEvent) {
       HRESULT hr = RecordCompositionUpdateAction();
       if (FAILED(hr)) {
         MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::SetSelectionInternal() FAILED due to "
-            "RecordCompositionUpdateAction() failure", this));
+          ("0x%p   TSFTextStore::SetSelectionInternal() FAILED due to "
+           "RecordCompositionUpdateAction() failure", this));
         return hr;
       }
     }
     return S_OK;
   }
 
   TS_SELECTION_ACP selectionInContent(*pSelection);
 
@@ -2738,18 +2739,18 @@ TSFTextStore::SetSelectionInternal(const
       // It seems that moving caret into middle of the new text is odd.
       // Perhaps, end of it is expected by users in most cases.
       selectionInContent.acpStart = mPendingTextChangeData.mAddedEndOffset;
       selectionInContent.acpEnd = selectionInContent.acpStart;
     } else {
       // Otherwise, i.e., setting range has already gone, we cannot set
       // selection properly.
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-         ("0x%p   TSFTextStore::SetSelectionInternal() FAILED due to "
-          "there is unknown content change", this));
+        ("0x%p   TSFTextStore::SetSelectionInternal() FAILED due to "
+         "there is unknown content change", this));
       return E_FAIL;
     }
   }
 
   CompleteLastActionIfStillIncomplete();
   PendingAction* action = mPendingActions.AppendElement();
   action->mType = PendingAction::SET_SELECTION;
   action->mSelectionStart = selectionInContent.acpStart;
@@ -2763,53 +2764,53 @@ TSFTextStore::SetSelectionInternal(const
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::SetSelection(ULONG ulCount,
                            const TS_SELECTION_ACP* pSelection)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::SetSelection(ulCount=%lu, pSelection=%p { "
-          "acpStart=%ld, acpEnd=%ld, style={ ase=%s, fInterimChar=%s } }), "
-          "mComposition.IsComposing()=%s",
-          this, ulCount, pSelection,
-          pSelection ? pSelection->acpStart : 0,
-          pSelection ? pSelection->acpEnd : 0,
-          pSelection ? GetActiveSelEndName(pSelection->style.ase) : "",
-          pSelection ? GetBoolName(pSelection->style.fInterimChar) : "",
-          GetBoolName(mComposition.IsComposing())));
+    ("0x%p TSFTextStore::SetSelection(ulCount=%lu, pSelection=%p { "
+     "acpStart=%ld, acpEnd=%ld, style={ ase=%s, fInterimChar=%s } }), "
+     "mComposition.IsComposing()=%s",
+     this, ulCount, pSelection,
+     pSelection ? pSelection->acpStart : 0,
+     pSelection ? pSelection->acpEnd : 0,
+     pSelection ? GetActiveSelEndName(pSelection->style.ase) : "",
+     pSelection ? GetBoolName(pSelection->style.fInterimChar) : "",
+     GetBoolName(mComposition.IsComposing())));
 
   if (!IsReadWriteLocked()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::SetSelection() FAILED due to "
-            "not locked (read-write)", this));
+      ("0x%p   TSFTextStore::SetSelection() FAILED due to "
+       "not locked (read-write)", this));
     return TS_E_NOLOCK;
   }
   if (ulCount != 1) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::SetSelection() FAILED due to "
-            "trying setting multiple selection", this));
+      ("0x%p   TSFTextStore::SetSelection() FAILED due to "
+       "trying setting multiple selection", this));
     return E_INVALIDARG;
   }
   if (!pSelection) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::SetSelection() FAILED due to "
-            "null argument", this));
+      ("0x%p   TSFTextStore::SetSelection() FAILED due to "
+       "null argument", this));
     return E_INVALIDARG;
   }
 
   HRESULT hr = SetSelectionInternal(pSelection, true);
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::SetSelection() FAILED due to "
-            "SetSelectionInternal() failure", this));
+      ("0x%p   TSFTextStore::SetSelection() FAILED due to "
+       "SetSelectionInternal() failure", this));
   } else {
     MOZ_LOG(sTextStoreLog, LogLevel::Info,
-           ("0x%p   TSFTextStore::SetSelection() succeeded", this));
+      ("0x%p   TSFTextStore::SetSelection() succeeded", this));
   }
   return hr;
 }
 
 STDMETHODIMP
 TSFTextStore::GetText(LONG acpStart,
                       LONG acpEnd,
                       WCHAR* pchPlain,
@@ -2827,64 +2828,64 @@ TSFTextStore::GetText(LONG acpStart,
      "mString.Length()=%lu, IsComposing()=%s }",
      this, acpStart, acpEnd, pchPlain, cchPlainReq, pcchPlainOut,
      prgRunInfo, ulRunInfoReq, pulRunInfoOut, pacpNext,
      mComposition.mStart, mComposition.mString.Length(),
      GetBoolName(mComposition.IsComposing())));
 
   if (!IsReadLocked()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetText() FAILED due to "
-            "not locked (read)", this));
+      ("0x%p   TSFTextStore::GetText() FAILED due to "
+       "not locked (read)", this));
     return TS_E_NOLOCK;
   }
 
   if (!pcchPlainOut || (!pchPlain && !prgRunInfo) ||
       !cchPlainReq != !pchPlain || !ulRunInfoReq != !prgRunInfo) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetText() FAILED due to "
-            "invalid argument", this));
+      ("0x%p   TSFTextStore::GetText() FAILED due to "
+       "invalid argument", this));
     return E_INVALIDARG;
   }
 
   if (acpStart < 0 || acpEnd < -1 || (acpEnd != -1 && acpStart > acpEnd)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetText() FAILED due to "
-            "invalid position", this));
+      ("0x%p   TSFTextStore::GetText() FAILED due to "
+       "invalid position", this));
     return TS_E_INVALIDPOS;
   }
 
   // Making sure to null-terminate string just to be on the safe side
   *pcchPlainOut = 0;
   if (pchPlain && cchPlainReq) *pchPlain = 0;
   if (pulRunInfoOut) *pulRunInfoOut = 0;
   if (pacpNext) *pacpNext = acpStart;
   if (prgRunInfo && ulRunInfoReq) {
     prgRunInfo->uCount = 0;
     prgRunInfo->type = TS_RT_PLAIN;
   }
 
   Content& contentForTSF = ContentForTSFRef();
   if (!contentForTSF.IsInitialized()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetText() FAILED due to "
-            "ContentForTSFRef() failure", this));
+      ("0x%p   TSFTextStore::GetText() FAILED due to "
+       "ContentForTSFRef() failure", this));
     return E_FAIL;
   }
   if (contentForTSF.Text().Length() < static_cast<uint32_t>(acpStart)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetText() FAILED due to "
-            "acpStart is larger offset than the actual text length", this));
+      ("0x%p   TSFTextStore::GetText() FAILED due to "
+       "acpStart is larger offset than the actual text length", this));
     return TS_E_INVALIDPOS;
   }
   if (acpEnd != -1 &&
       contentForTSF.Text().Length() < static_cast<uint32_t>(acpEnd)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetText() FAILED due to "
-            "acpEnd is larger offset than the actual text length", this));
+      ("0x%p   TSFTextStore::GetText() FAILED due to "
+       "acpEnd is larger offset than the actual text length", this));
     return TS_E_INVALIDPOS;
   }
   uint32_t length = (acpEnd == -1) ?
     contentForTSF.Text().Length() - static_cast<uint32_t>(acpStart) :
     static_cast<uint32_t>(acpEnd - acpStart);
   if (cchPlainReq && cchPlainReq - 1 < length) {
     length = cchPlainReq - 1;
   }
@@ -2900,135 +2901,135 @@ TSFTextStore::GetText(LONG acpStart,
       prgRunInfo->uCount = length;
       prgRunInfo->type = TS_RT_PLAIN;
       if (pulRunInfoOut) *pulRunInfoOut = 1;
     }
     if (pacpNext) *pacpNext = acpStart + length;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::GetText() succeeded: pcchPlainOut=0x%p, "
-          "*prgRunInfo={ uCount=%lu, type=%s }, *pulRunInfoOut=%lu, "
-          "*pacpNext=%ld)",
-          this, pcchPlainOut, prgRunInfo ? prgRunInfo->uCount : 0,
-          prgRunInfo ? GetTextRunTypeName(prgRunInfo->type) : "N/A",
-          pulRunInfoOut ? *pulRunInfoOut : 0, pacpNext ? *pacpNext : 0));
+    ("0x%p   TSFTextStore::GetText() succeeded: pcchPlainOut=0x%p, "
+     "*prgRunInfo={ uCount=%lu, type=%s }, *pulRunInfoOut=%lu, "
+     "*pacpNext=%ld)",
+     this, pcchPlainOut, prgRunInfo ? prgRunInfo->uCount : 0,
+     prgRunInfo ? GetTextRunTypeName(prgRunInfo->type) : "N/A",
+     pulRunInfoOut ? *pulRunInfoOut : 0, pacpNext ? *pacpNext : 0));
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::SetText(DWORD dwFlags,
                       LONG acpStart,
                       LONG acpEnd,
                       const WCHAR* pchText,
                       ULONG cch,
                       TS_TEXTCHANGE* pChange)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::SetText(dwFlags=%s, acpStart=%ld, "
-          "acpEnd=%ld, pchText=0x%p \"%s\", cch=%lu, pChange=0x%p), "
-          "mComposition.IsComposing()=%s",
-          this, dwFlags == TS_ST_CORRECTION ? "TS_ST_CORRECTION" :
-                                              "not-specified",
-          acpStart, acpEnd, pchText,
-          pchText && cch ?
-            NS_ConvertUTF16toUTF8(pchText, cch).get() : "",
-          cch, pChange, GetBoolName(mComposition.IsComposing())));
+    ("0x%p TSFTextStore::SetText(dwFlags=%s, acpStart=%ld, "
+     "acpEnd=%ld, pchText=0x%p \"%s\", cch=%lu, pChange=0x%p), "
+     "mComposition.IsComposing()=%s",
+     this, dwFlags == TS_ST_CORRECTION ? "TS_ST_CORRECTION" :
+                                         "not-specified",
+     acpStart, acpEnd, pchText,
+     pchText && cch ?
+       NS_ConvertUTF16toUTF8(pchText, cch).get() : "",
+     cch, pChange, GetBoolName(mComposition.IsComposing())));
 
   // Per SDK documentation, and since we don't have better
   // ways to do this, this method acts as a helper to
   // call SetSelection followed by InsertTextAtSelection
   if (!IsReadWriteLocked()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::SetText() FAILED due to "
-            "not locked (read)", this));
+      ("0x%p   TSFTextStore::SetText() FAILED due to "
+       "not locked (read)", this));
     return TS_E_NOLOCK;
   }
 
   TS_SELECTION_ACP selection;
   selection.acpStart = acpStart;
   selection.acpEnd = acpEnd;
   selection.style.ase = TS_AE_END;
   selection.style.fInterimChar = 0;
   // Set selection to desired range
   HRESULT hr = SetSelectionInternal(&selection);
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::SetText() FAILED due to "
-            "SetSelectionInternal() failure", this));
+      ("0x%p   TSFTextStore::SetText() FAILED due to "
+       "SetSelectionInternal() failure", this));
     return hr;
   }
   // Replace just selected text
   if (!InsertTextAtSelectionInternal(nsDependentSubstring(pchText, cch),
                                      pChange)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::SetText() FAILED due to "
-            "InsertTextAtSelectionInternal() failure", this));
+      ("0x%p   TSFTextStore::SetText() FAILED due to "
+       "InsertTextAtSelectionInternal() failure", this));
     return E_FAIL;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::SetText() succeeded: pChange={ "
-          "acpStart=%ld, acpOldEnd=%ld, acpNewEnd=%ld }",
-          this, pChange ? pChange->acpStart  : 0,
-          pChange ? pChange->acpOldEnd : 0, pChange ? pChange->acpNewEnd : 0));
+    ("0x%p   TSFTextStore::SetText() succeeded: pChange={ "
+     "acpStart=%ld, acpOldEnd=%ld, acpNewEnd=%ld }",
+     this, pChange ? pChange->acpStart  : 0,
+     pChange ? pChange->acpOldEnd : 0, pChange ? pChange->acpNewEnd : 0));
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::GetFormattedText(LONG acpStart,
                                LONG acpEnd,
                                IDataObject** ppDataObject)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::GetFormattedText() called "
-          "but not supported (E_NOTIMPL)", this));
+    ("0x%p TSFTextStore::GetFormattedText() called "
+     "but not supported (E_NOTIMPL)", this));
 
   // no support for formatted text
   return E_NOTIMPL;
 }
 
 STDMETHODIMP
 TSFTextStore::GetEmbedded(LONG acpPos,
                           REFGUID rguidService,
                           REFIID riid,
                           IUnknown** ppunk)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::GetEmbedded() called "
-          "but not supported (E_NOTIMPL)", this));
+    ("0x%p TSFTextStore::GetEmbedded() called "
+     "but not supported (E_NOTIMPL)", this));
 
   // embedded objects are not supported
   return E_NOTIMPL;
 }
 
 STDMETHODIMP
 TSFTextStore::QueryInsertEmbedded(const GUID* pguidService,
                                   const FORMATETC* pFormatEtc,
                                   BOOL* pfInsertable)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::QueryInsertEmbedded() called "
-          "but not supported, *pfInsertable=FALSE (S_OK)", this));
+    ("0x%p TSFTextStore::QueryInsertEmbedded() called "
+     "but not supported, *pfInsertable=FALSE (S_OK)", this));
 
   // embedded objects are not supported
   *pfInsertable = FALSE;
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::InsertEmbedded(DWORD dwFlags,
                              LONG acpStart,
                              LONG acpEnd,
                              IDataObject* pDataObject,
                              TS_TEXTCHANGE* pChange)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::InsertEmbedded() called "
-          "but not supported (E_NOTIMPL)", this));
+    ("0x%p TSFTextStore::InsertEmbedded() called "
+     "but not supported (E_NOTIMPL)", this));
 
   // embedded objects are not supported
   return E_NOTIMPL;
 }
 
 void
 TSFTextStore::SetInputScope(const nsString& aHTMLInputType,
                             const nsString& aHTMLInputInputMode)
@@ -3107,83 +3108,83 @@ TSFTextStore::GetAttrID(int32_t aIndex)
 }
 
 HRESULT
 TSFTextStore::HandleRequestAttrs(DWORD aFlags,
                                  ULONG aFilterCount,
                                  const TS_ATTRID* aFilterAttrs)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::HandleRequestAttrs(aFlags=%s, "
-          "aFilterCount=%u)",
-          this, GetFindFlagName(aFlags).get(), aFilterCount));
+    ("0x%p TSFTextStore::HandleRequestAttrs(aFlags=%s, "
+     "aFilterCount=%u)",
+     this, GetFindFlagName(aFlags).get(), aFilterCount));
 
   // This is a little weird! RequestSupportedAttrs gives us advanced notice
   // of a support query via RetrieveRequestedAttrs for a specific attribute.
   // RetrieveRequestedAttrs needs to return valid data for all attributes we
   // support, but the text service will only want the input scope object
   // returned in RetrieveRequestedAttrs if the dwFlags passed in here contains
   // TS_ATTR_FIND_WANT_VALUE.
   for (int32_t i = 0; i < NUM_OF_SUPPORTED_ATTRS; i++) {
     mRequestedAttrs[i] = false;
   }
   mRequestedAttrValues = !!(aFlags & TS_ATTR_FIND_WANT_VALUE);
 
   for (uint32_t i = 0; i < aFilterCount; i++) {
     MOZ_LOG(sTextStoreLog, LogLevel::Info,
-           ("0x%p   TSFTextStore::HandleRequestAttrs(), "
-            "requested attr=%s",
-            this, GetGUIDNameStrWithTable(aFilterAttrs[i]).get()));
+      ("0x%p   TSFTextStore::HandleRequestAttrs(), "
+       "requested attr=%s",
+       this, GetGUIDNameStrWithTable(aFilterAttrs[i]).get()));
     int32_t index = GetRequestedAttrIndex(aFilterAttrs[i]);
     if (index != eNotSupported) {
       mRequestedAttrs[index] = true;
     }
   }
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::RequestSupportedAttrs(DWORD dwFlags,
                                     ULONG cFilterAttrs,
                                     const TS_ATTRID* paFilterAttrs)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::RequestSupportedAttrs(dwFlags=%s, "
-          "cFilterAttrs=%lu)",
-          this, GetFindFlagName(dwFlags).get(), cFilterAttrs));
+    ("0x%p TSFTextStore::RequestSupportedAttrs(dwFlags=%s, "
+     "cFilterAttrs=%lu)",
+     this, GetFindFlagName(dwFlags).get(), cFilterAttrs));
 
   return HandleRequestAttrs(dwFlags, cFilterAttrs, paFilterAttrs);
 }
 
 STDMETHODIMP
 TSFTextStore::RequestAttrsAtPosition(LONG acpPos,
                                      ULONG cFilterAttrs,
                                      const TS_ATTRID* paFilterAttrs,
                                      DWORD dwFlags)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::RequestAttrsAtPosition(acpPos=%ld, "
-          "cFilterAttrs=%lu, dwFlags=%s)",
-          this, acpPos, cFilterAttrs, GetFindFlagName(dwFlags).get()));
+    ("0x%p TSFTextStore::RequestAttrsAtPosition(acpPos=%ld, "
+     "cFilterAttrs=%lu, dwFlags=%s)",
+     this, acpPos, cFilterAttrs, GetFindFlagName(dwFlags).get()));
 
   return HandleRequestAttrs(dwFlags | TS_ATTR_FIND_WANT_VALUE,
                             cFilterAttrs, paFilterAttrs);
 }
 
 STDMETHODIMP
 TSFTextStore::RequestAttrsTransitioningAtPosition(LONG acpPos,
                                                   ULONG cFilterAttrs,
                                                   const TS_ATTRID* paFilterAttr,
                                                   DWORD dwFlags)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::RequestAttrsTransitioningAtPosition("
-          "acpPos=%ld, cFilterAttrs=%lu, dwFlags=%s) called but not supported "
-          "(S_OK)",
-          this, acpPos, cFilterAttrs, GetFindFlagName(dwFlags).get()));
+    ("0x%p TSFTextStore::RequestAttrsTransitioningAtPosition("
+     "acpPos=%ld, cFilterAttrs=%lu, dwFlags=%s) called but not supported "
+     "(S_OK)",
+     this, acpPos, cFilterAttrs, GetFindFlagName(dwFlags).get()));
 
   // no per character attributes defined
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::FindNextAttrTransition(LONG acpStart,
                                      LONG acpHalt,
@@ -3191,74 +3192,74 @@ TSFTextStore::FindNextAttrTransition(LON
                                      const TS_ATTRID* paFilterAttrs,
                                      DWORD dwFlags,
                                      LONG* pacpNext,
                                      BOOL* pfFound,
                                      LONG* plFoundOffset)
 {
   if (!pacpNext || !pfFound || !plFoundOffset) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("  0x%p TSFTextStore::FindNextAttrTransition() FAILED due to "
-            "null argument", this));
+      ("  0x%p TSFTextStore::FindNextAttrTransition() FAILED due to "
+       "null argument", this));
     return E_INVALIDARG;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::FindNextAttrTransition() called "
-          "but not supported (S_OK)", this));
+    ("0x%p   TSFTextStore::FindNextAttrTransition() called "
+     "but not supported (S_OK)", this));
 
   // no per character attributes defined
   *pacpNext = *plFoundOffset = acpHalt;
   *pfFound = FALSE;
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::RetrieveRequestedAttrs(ULONG ulCount,
                                      TS_ATTRVAL* paAttrVals,
                                      ULONG* pcFetched)
 {
   if (!pcFetched || !paAttrVals) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p TSFTextStore::RetrieveRequestedAttrs() FAILED due to "
-            "null argument", this));
+      ("0x%p TSFTextStore::RetrieveRequestedAttrs() FAILED due to "
+       "null argument", this));
     return E_INVALIDARG;
   }
 
   ULONG expectedCount = 0;
   for (int32_t i = 0; i < NUM_OF_SUPPORTED_ATTRS; i++) {
     if (mRequestedAttrs[i]) {
       expectedCount++;
     }
   }
   if (ulCount < expectedCount) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p TSFTextStore::RetrieveRequestedAttrs() FAILED due to "
-            "not enough count ulCount=%u, expectedCount=%u",
-            this, ulCount, expectedCount));
+      ("0x%p TSFTextStore::RetrieveRequestedAttrs() FAILED due to "
+       "not enough count ulCount=%u, expectedCount=%u",
+       this, ulCount, expectedCount));
     return E_INVALIDARG;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::RetrieveRequestedAttrs() called "
-          "ulCount=%d, mRequestedAttrValues=%s",
-          this, ulCount, GetBoolName(mRequestedAttrValues)));
+    ("0x%p TSFTextStore::RetrieveRequestedAttrs() called "
+     "ulCount=%d, mRequestedAttrValues=%s",
+     this, ulCount, GetBoolName(mRequestedAttrValues)));
 
   int32_t count = 0;
   for (int32_t i = 0; i < NUM_OF_SUPPORTED_ATTRS; i++) {
     if (!mRequestedAttrs[i]) {
       continue;
     }
     mRequestedAttrs[i] = false;
 
     TS_ATTRID attrID = GetAttrID(i);
 
     MOZ_LOG(sTextStoreLog, LogLevel::Info,
-           ("0x%p   TSFTextStore::RetrieveRequestedAttrs() for %s",
-            this, GetGUIDNameStrWithTable(attrID).get()));
+      ("0x%p   TSFTextStore::RetrieveRequestedAttrs() for %s",
+       this, GetGUIDNameStrWithTable(attrID).get()));
 
     paAttrVals[count].idAttr = attrID;
     paAttrVals[count].dwOverlapId = 0;
 
     if (!mRequestedAttrValues) {
       paAttrVals[count].varValue.vt = VT_EMPTY;
     } else {
       switch (i) {
@@ -3294,126 +3295,126 @@ TSFTextStore::RetrieveRequestedAttrs(ULO
   mRequestedAttrValues = false;
 
   if (count) {
     *pcFetched = count;
     return S_OK;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::RetrieveRequestedAttrs() called "
-          "for unknown TS_ATTRVAL, *pcFetched=0 (S_OK)", this));
+    ("0x%p   TSFTextStore::RetrieveRequestedAttrs() called "
+     "for unknown TS_ATTRVAL, *pcFetched=0 (S_OK)", this));
 
   paAttrVals->dwOverlapId = 0;
   paAttrVals->varValue.vt = VT_EMPTY;
   *pcFetched = 0;
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::GetEndACP(LONG* pacp)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::GetEndACP(pacp=0x%p)", this, pacp));
+    ("0x%p TSFTextStore::GetEndACP(pacp=0x%p)", this, pacp));
 
   if (!IsReadLocked()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetEndACP() FAILED due to "
-            "not locked (read)", this));
+      ("0x%p   TSFTextStore::GetEndACP() FAILED due to "
+       "not locked (read)", this));
     return TS_E_NOLOCK;
   }
 
   if (!pacp) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetEndACP() FAILED due to "
-            "null argument", this));
+      ("0x%p   TSFTextStore::GetEndACP() FAILED due to "
+       "null argument", this));
     return E_INVALIDARG;
   }
 
   Content& contentForTSF = ContentForTSFRef();
   if (!contentForTSF.IsInitialized()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetEndACP() FAILED due to "
-            "ContentForTSFRef() failure", this));
+      ("0x%p   TSFTextStore::GetEndACP() FAILED due to "
+       "ContentForTSFRef() failure", this));
     return E_FAIL;
   }
   *pacp = static_cast<LONG>(contentForTSF.Text().Length());
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::GetActiveView(TsViewCookie* pvcView)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::GetActiveView(pvcView=0x%p)",
-          this, pvcView));
+    ("0x%p TSFTextStore::GetActiveView(pvcView=0x%p)",
+     this, pvcView));
 
   if (!pvcView) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetActiveView() FAILED due to "
-            "null argument", this));
+      ("0x%p   TSFTextStore::GetActiveView() FAILED due to "
+       "null argument", this));
     return E_INVALIDARG;
   }
 
   *pvcView = TEXTSTORE_DEFAULT_VIEW;
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::GetActiveView() succeeded: *pvcView=%ld",
-          this, *pvcView));
+    ("0x%p   TSFTextStore::GetActiveView() succeeded: *pvcView=%ld",
+     this, *pvcView));
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::GetACPFromPoint(TsViewCookie vcView,
                               const POINT* pt,
                               DWORD dwFlags,
                               LONG* pacp)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::GetACPFromPoint(pvcView=%d, pt=%p (x=%d, "
-          "y=%d), dwFlags=%s, pacp=%p, mDeferNotifyingTSF=%s, "
-          "mWaitingQueryLayout=%s",
-          this, vcView, pt, pt ? pt->x : 0, pt ? pt->y : 0,
-          GetACPFromPointFlagName(dwFlags).get(), pacp,
-          GetBoolName(mDeferNotifyingTSF), GetBoolName(mWaitingQueryLayout)));
+    ("0x%p TSFTextStore::GetACPFromPoint(pvcView=%d, pt=%p (x=%d, "
+     "y=%d), dwFlags=%s, pacp=%p, mDeferNotifyingTSF=%s, "
+     "mWaitingQueryLayout=%s",
+     this, vcView, pt, pt ? pt->x : 0, pt ? pt->y : 0,
+     GetACPFromPointFlagName(dwFlags).get(), pacp,
+     GetBoolName(mDeferNotifyingTSF), GetBoolName(mWaitingQueryLayout)));
 
   if (!IsReadLocked()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to "
-            "not locked (read)", this));
+      ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to "
+       "not locked (read)", this));
     return TS_E_NOLOCK;
   }
 
   if (vcView != TEXTSTORE_DEFAULT_VIEW) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to "
-            "called with invalid view", this));
+      ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to "
+       "called with invalid view", this));
     return E_INVALIDARG;
   }
 
   if (!pt) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to "
-            "null pt", this));
+      ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to "
+       "null pt", this));
     return E_INVALIDARG;
   }
 
   if (!pacp) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to "
-            "null pacp", this));
+      ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to "
+       "null pacp", this));
     return E_INVALIDARG;
   }
 
   mWaitingQueryLayout = false;
 
   if (mDestroyed || mContentForTSF.IsLayoutChanged()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetACPFromPoint() returned "
-            "TS_E_NOLAYOUT", this));
+      ("0x%p   TSFTextStore::GetACPFromPoint() returned "
+       "TS_E_NOLAYOUT", this));
     mHasReturnedNoLayoutError = true;
     return TS_E_NOLAYOUT;
   }
 
   LayoutDeviceIntPoint ourPt(pt->x, pt->y);
   // Convert to widget relative coordinates from screen's.
   ourPt -= mWidget->WidgetToScreenOffset();
 
@@ -3424,41 +3425,41 @@ TSFTextStore::GetACPFromPoint(TsViewCook
   mWidget->InitEvent(charAtPt, &ourPt);
 
   // FYI: WidgetQueryContentEvent may cause flushing pending layout and it
   //      may cause focus change or something.
   RefPtr<TSFTextStore> kungFuDeathGrip(this);
   DispatchEvent(charAtPt);
   if (!mWidget || mWidget->Destroyed()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to "
-            "mWidget was destroyed during eQueryCharacterAtPoint", this));
+      ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to "
+       "mWidget was destroyed during eQueryCharacterAtPoint", this));
     return E_FAIL;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::GetACPFromPoint(), charAtPt={ "
-          "mSucceeded=%s, mReply={ mOffset=%u, mTentativeCaretOffset=%u }}",
-          this, GetBoolName(charAtPt.mSucceeded), charAtPt.mReply.mOffset,
-          charAtPt.mReply.mTentativeCaretOffset));
+    ("0x%p   TSFTextStore::GetACPFromPoint(), charAtPt={ "
+     "mSucceeded=%s, mReply={ mOffset=%u, mTentativeCaretOffset=%u }}",
+     this, GetBoolName(charAtPt.mSucceeded), charAtPt.mReply.mOffset,
+     charAtPt.mReply.mTentativeCaretOffset));
 
   if (NS_WARN_IF(!charAtPt.mSucceeded)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to "
-            "eQueryCharacterAtPoint failure", this));
+      ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to "
+       "eQueryCharacterAtPoint failure", this));
     return E_FAIL;
   }
 
   // If dwFlags isn't set and the point isn't in any character's bounding box,
   // we should return TS_E_INVALIDPOINT.
   if (!(dwFlags & GXFPF_NEAREST) &&
       charAtPt.mReply.mOffset == WidgetQueryContentEvent::NOT_FOUND) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to the "
-            "point contained by no bounding box", this));
+      ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to the "
+       "point contained by no bounding box", this));
     return TS_E_INVALIDPOINT;
   }
 
   // Although, we're not sure if mTentativeCaretOffset becomes NOT_FOUND,
   // let's assume that there is no content in such case.
   if (NS_WARN_IF(charAtPt.mReply.mTentativeCaretOffset ==
                    WidgetQueryContentEvent::NOT_FOUND)) {
     charAtPt.mReply.mTentativeCaretOffset = 0;
@@ -3485,80 +3486,80 @@ TSFTextStore::GetACPFromPoint(TsViewCook
     // So, use tentative offset for now.
     offset = charAtPt.mReply.mTentativeCaretOffset;
 
     // However, if it's after the last character, we need to decrement the
     // offset.
     Content& contentForTSF = ContentForTSFRef();
     if (!contentForTSF.IsInitialized()) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-             ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to "
-              "ContentForTSFRef() failure", this));
+        ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to "
+         "ContentForTSFRef() failure", this));
       return E_FAIL;
     }
     if (contentForTSF.Text().Length() <= offset) {
       // If the tentative caret is after the last character, let's return
       // the last character's offset.
       offset = contentForTSF.Text().Length() - 1;
     }
   }
 
   if (NS_WARN_IF(offset > LONG_MAX)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to out of "
-            "range of the result", this));
+      ("0x%p   TSFTextStore::GetACPFromPoint() FAILED due to out of "
+       "range of the result", this));
     return TS_E_INVALIDPOINT;
   }
 
   *pacp = static_cast<LONG>(offset);
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::GetACPFromPoint() succeeded: *pacp=%d",
-          this, *pacp));
+    ("0x%p   TSFTextStore::GetACPFromPoint() succeeded: *pacp=%d",
+     this, *pacp));
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::GetTextExt(TsViewCookie vcView,
                          LONG acpStart,
                          LONG acpEnd,
                          RECT* prc,
                          BOOL* pfClipped)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::GetTextExt(vcView=%ld, "
-          "acpStart=%ld, acpEnd=%ld, prc=0x%p, pfClipped=0x%p), "
-          "mDeferNotifyingTSF=%s, mWaitingQueryLayout=%s",
-          this, vcView, acpStart, acpEnd, prc, pfClipped,
-          GetBoolName(mDeferNotifyingTSF), GetBoolName(mWaitingQueryLayout)));
+    ("0x%p TSFTextStore::GetTextExt(vcView=%ld, "
+     "acpStart=%ld, acpEnd=%ld, prc=0x%p, pfClipped=0x%p), "
+     "mDeferNotifyingTSF=%s, mWaitingQueryLayout=%s",
+     this, vcView, acpStart, acpEnd, prc, pfClipped,
+     GetBoolName(mDeferNotifyingTSF), GetBoolName(mWaitingQueryLayout)));
 
   if (!IsReadLocked()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetTextExt() FAILED due to "
-            "not locked (read)", this));
+      ("0x%p   TSFTextStore::GetTextExt() FAILED due to "
+       "not locked (read)", this));
     return TS_E_NOLOCK;
   }
 
   if (vcView != TEXTSTORE_DEFAULT_VIEW) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetTextExt() FAILED due to "
-            "called with invalid view", this));
+      ("0x%p   TSFTextStore::GetTextExt() FAILED due to "
+       "called with invalid view", this));
     return E_INVALIDARG;
   }
 
   if (!prc || !pfClipped) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetTextExt() FAILED due to "
-            "null argument", this));
+      ("0x%p   TSFTextStore::GetTextExt() FAILED due to "
+       "null argument", this));
     return E_INVALIDARG;
   }
 
   if (acpStart < 0 || acpEnd < acpStart) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetTextExt() FAILED due to "
-            "invalid position", this));
+      ("0x%p   TSFTextStore::GetTextExt() FAILED due to "
+       "invalid position", this));
     return TS_E_INVALIDPOS;
   }
 
   mWaitingQueryLayout = false;
 
   // NOTE: TSF (at least on Win 8.1) doesn't return TS_E_NOLAYOUT to the
   // caller even if we return it.  It's converted to just E_FAIL.
   // However, this is fixed on Win 10.
@@ -3588,81 +3589,81 @@ TSFTextStore::GetTextExt(TsViewCookie vc
       // Therefore, if the first character at the retrieving range rect is
       // available, we should use it as the result.
       if ((kIsMSOfficeJapaneseIME2010 ||
            sDoNotReturnNoLayoutErrorToMSJapaneseIMEAtFirstChar) &&
           !mContentForTSF.IsLayoutChangedAfter(acpStart) &&
           acpStart < acpEnd) {
         acpEnd = acpStart;
         MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-               ("0x%p   TSFTextStore::GetTextExt() hacked the offsets "
-                "of the first character of changing range of the composition "
-                "string for TIP acpStart=%d, acpEnd=%d",
-                this, acpStart, acpEnd));
+          ("0x%p   TSFTextStore::GetTextExt() hacked the offsets "
+           "of the first character of changing range of the composition "
+           "string for TIP acpStart=%d, acpEnd=%d",
+           this, acpStart, acpEnd));
       }
       // Although, the condition is not clear, MS-IME sometimes retrieves the
       // caret rect immediately after modifying the composition string but
       // before unlocking the document.  In such case, we should return the
       // nearest character rect.
       else if ((kIsMSOfficeJapaneseIME2010 ||
                 sDoNotReturnNoLayoutErrorToMSJapaneseIMEAtCaret) &&
                acpStart == acpEnd &&
                selectionForTSF.IsCollapsed() &&
                selectionForTSF.EndOffset() == acpEnd) {
         acpEnd = acpStart = mContentForTSF.MinOffsetOfLayoutChanged();
         MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-               ("0x%p   TSFTextStore::GetTextExt() hacked the offsets "
-                "of the caret of the composition string for TIP acpStart=%d, "
-                "acpEnd=%d", this, acpStart, acpEnd));
+          ("0x%p   TSFTextStore::GetTextExt() hacked the offsets "
+           "of the caret of the composition string for TIP acpStart=%d, "
+           "acpEnd=%d", this, acpStart, acpEnd));
       }
     }
     // Free ChangJie 2010 and Easy Changjei 1.0.12.0 doesn't handle
     // ITfContextView::GetTextExt() properly.  Prehaps, it's due to the bug of
     // TSF.  We need to check if this is necessary on Windows 10 before
     // disabling this on Windows 10.
     else if ((sDoNotReturnNoLayoutErrorToFreeChangJie &&
               kSink->IsFreeChangJieActive()) ||
              (sDoNotReturnNoLayoutErrorToEasyChangjei &&
               kSink->IsEasyChangjeiActive())) {
       acpEnd = mComposition.mStart;
       acpStart = std::min(acpStart, acpEnd);
       MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-             ("0x%p   TSFTextStore::GetTextExt() hacked the offsets for "
-              "TIP acpStart=%d, acpEnd=%d", this, acpStart, acpEnd));
+        ("0x%p   TSFTextStore::GetTextExt() hacked the offsets for "
+         "TIP acpStart=%d, acpEnd=%d", this, acpStart, acpEnd));
     }
     // Some Chinese TIPs of Microsoft doesn't show candidate window in e10s
     // mode on Win8 or later.
     else if (IsWin8OrLater() &&
              ((sDoNotReturnNoLayoutErrorToMSTraditionalTIP &&
                (kSink->IsMSChangJieActive() ||
                 kSink->IsMSQuickQuickActive())) ||
               (sDoNotReturnNoLayoutErrorToMSSimplifiedTIP &&
                 (kSink->IsMSPinyinActive() ||
                  kSink->IsMSWubiActive())))) {
       acpEnd = mComposition.mStart;
       acpStart = std::min(acpStart, acpEnd);
       MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-             ("0x%p   TSFTextStore::GetTextExt() hacked the offsets for "
-              "TIP acpStart=%d, acpEnd=%d", this, acpStart, acpEnd));
+        ("0x%p   TSFTextStore::GetTextExt() hacked the offsets for "
+         "TIP acpStart=%d, acpEnd=%d", this, acpStart, acpEnd));
     }
   }
 
   if (mContentForTSF.IsLayoutChangedAfter(acpEnd)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetTextExt() returned TS_E_NOLAYOUT "
-            "(acpEnd=%d)", this, acpEnd));
+      ("0x%p   TSFTextStore::GetTextExt() returned TS_E_NOLAYOUT "
+       "(acpEnd=%d)", this, acpEnd));
     mHasReturnedNoLayoutError = true;
     return TS_E_NOLAYOUT;
   }
 
   if (mDestroyed) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetTextExt() returned TS_E_NOLAYOUT "
-            "(acpEnd=%d) because this has already been destroyed",
-            this, acpEnd));
+      ("0x%p   TSFTextStore::GetTextExt() returned TS_E_NOLAYOUT "
+       "(acpEnd=%d) because this has already been destroyed",
+       this, acpEnd));
     mHasReturnedNoLayoutError = true;
     return TS_E_NOLAYOUT;
   }
 
   // use eQueryTextRect to get rect in system, screen coordinates
   WidgetQueryContentEvent event(true, eQueryTextRect, mWidget);
   mWidget->InitEvent(event);
 
@@ -3676,45 +3677,45 @@ TSFTextStore::GetTextExt(TsViewCookie vc
     options.mRelativeToInsertionPoint = true;
     startOffset -= mComposition.mStart;
   }
   event.InitForQueryTextRect(startOffset, acpEnd - acpStart, options);
 
   DispatchEvent(event);
   if (NS_WARN_IF(!event.mSucceeded)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetTextExt() FAILED due to "
-            "eQueryTextRect failure", this));
+      ("0x%p   TSFTextStore::GetTextExt() FAILED due to "
+       "eQueryTextRect failure", this));
     return TS_E_INVALIDPOS; // but unexpected failure, maybe.
   }
   // IMEs don't like empty rects, fix here
   if (event.mReply.mRect.width <= 0)
     event.mReply.mRect.width = 1;
   if (event.mReply.mRect.height <= 0)
     event.mReply.mRect.height = 1;
 
   // convert to unclipped screen rect
   nsWindow* refWindow = static_cast<nsWindow*>(
     event.mReply.mFocusedWidget ? event.mReply.mFocusedWidget : mWidget);
   // Result rect is in top level widget coordinates
   refWindow = refWindow->GetTopLevelWindow(false);
   if (!refWindow) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetTextExt() FAILED due to "
-            "no top level window", this));
+      ("0x%p   TSFTextStore::GetTextExt() FAILED due to "
+       "no top level window", this));
     return E_FAIL;
   }
 
   event.mReply.mRect.MoveBy(refWindow->WidgetToScreenOffset());
 
   // get bounding screen rect to test for clipping
   if (!GetScreenExtInternal(*prc)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetTextExt() FAILED due to "
-            "GetScreenExtInternal() failure", this));
+      ("0x%p   TSFTextStore::GetTextExt() FAILED due to "
+       "GetScreenExtInternal() failure", this));
     return E_FAIL;
   }
 
   // clip text rect to bounding rect
   RECT textRect;
   ::SetRect(&textRect, event.mReply.mRect.x, event.mReply.mRect.y,
             event.mReply.mRect.XMost(), event.mReply.mRect.YMost());
   if (!::IntersectRect(prc, prc, &textRect))
@@ -3730,278 +3731,278 @@ TSFTextStore::GetTextExt(TsViewCookie vc
   if (sCreateNativeCaretForATOK && kSink->IsATOKActive() &&
       mComposition.IsComposing() &&
       mComposition.mStart <= acpStart && mComposition.EndOffset() >= acpStart &&
       mComposition.mStart <= acpEnd && mComposition.EndOffset() >= acpEnd) {
     CreateNativeCaret();
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::GetTextExt() succeeded: "
-          "*prc={ left=%ld, top=%ld, right=%ld, bottom=%ld }, *pfClipped=%s",
-          this, prc->left, prc->top, prc->right, prc->bottom,
-          GetBoolName(*pfClipped)));
+    ("0x%p   TSFTextStore::GetTextExt() succeeded: "
+     "*prc={ left=%ld, top=%ld, right=%ld, bottom=%ld }, *pfClipped=%s",
+     this, prc->left, prc->top, prc->right, prc->bottom,
+     GetBoolName(*pfClipped)));
 
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::GetScreenExt(TsViewCookie vcView,
                            RECT* prc)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::GetScreenExt(vcView=%ld, prc=0x%p)",
-          this, vcView, prc));
+    ("0x%p TSFTextStore::GetScreenExt(vcView=%ld, prc=0x%p)",
+     this, vcView, prc));
 
   if (vcView != TEXTSTORE_DEFAULT_VIEW) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetScreenExt() FAILED due to "
-            "called with invalid view", this));
+      ("0x%p   TSFTextStore::GetScreenExt() FAILED due to "
+       "called with invalid view", this));
     return E_INVALIDARG;
   }
 
   if (!prc) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetScreenExt() FAILED due to "
-            "null argument", this));
+      ("0x%p   TSFTextStore::GetScreenExt() FAILED due to "
+       "null argument", this));
     return E_INVALIDARG;
   }
 
   if (mDestroyed) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetScreenExt() returns empty rect "
-            "due to already destroyed", this));
+      ("0x%p   TSFTextStore::GetScreenExt() returns empty rect "
+       "due to already destroyed", this));
     prc->left = prc->top = prc->right = prc->left = 0;
     return S_OK;
   }
 
   if (!GetScreenExtInternal(*prc)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetScreenExt() FAILED due to "
-            "GetScreenExtInternal() failure", this));
+      ("0x%p   TSFTextStore::GetScreenExt() FAILED due to "
+       "GetScreenExtInternal() failure", this));
     return E_FAIL;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::GetScreenExt() succeeded: "
-          "*prc={ left=%ld, top=%ld, right=%ld, bottom=%ld }",
-          this, prc->left, prc->top, prc->right, prc->bottom));
+    ("0x%p   TSFTextStore::GetScreenExt() succeeded: "
+     "*prc={ left=%ld, top=%ld, right=%ld, bottom=%ld }",
+     this, prc->left, prc->top, prc->right, prc->bottom));
   return S_OK;
 }
 
 bool
 TSFTextStore::GetScreenExtInternal(RECT& aScreenExt)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::GetScreenExtInternal()", this));
+    ("0x%p   TSFTextStore::GetScreenExtInternal()", this));
 
   MOZ_ASSERT(!mDestroyed);
 
   // use NS_QUERY_EDITOR_RECT to get rect in system, screen coordinates
   WidgetQueryContentEvent event(true, eQueryEditorRect, mWidget);
   mWidget->InitEvent(event);
   DispatchEvent(event);
   if (!event.mSucceeded) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetScreenExtInternal() FAILED due to "
-            "eQueryEditorRect failure", this));
+      ("0x%p   TSFTextStore::GetScreenExtInternal() FAILED due to "
+       "eQueryEditorRect failure", this));
     return false;
   }
 
   nsWindow* refWindow = static_cast<nsWindow*>(
     event.mReply.mFocusedWidget ?
       event.mReply.mFocusedWidget : mWidget);
   // Result rect is in top level widget coordinates
   refWindow = refWindow->GetTopLevelWindow(false);
   if (!refWindow) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetScreenExtInternal() FAILED due to "
-            "no top level window", this));
+      ("0x%p   TSFTextStore::GetScreenExtInternal() FAILED due to "
+       "no top level window", this));
     return false;
   }
 
   LayoutDeviceIntRect boundRect;
   if (NS_FAILED(refWindow->GetClientBounds(boundRect))) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetScreenExtInternal() FAILED due to "
-            "failed to get the client bounds", this));
+      ("0x%p   TSFTextStore::GetScreenExtInternal() FAILED due to "
+       "failed to get the client bounds", this));
     return false;
   }
 
   boundRect.MoveTo(0, 0);
 
   // Clip frame rect to window rect
   boundRect.IntersectRect(event.mReply.mRect, boundRect);
   if (!boundRect.IsEmpty()) {
     boundRect.MoveBy(refWindow->WidgetToScreenOffset());
     ::SetRect(&aScreenExt, boundRect.x, boundRect.y,
               boundRect.XMost(), boundRect.YMost());
   } else {
     ::SetRectEmpty(&aScreenExt);
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::GetScreenExtInternal() succeeded: "
-          "aScreenExt={ left=%ld, top=%ld, right=%ld, bottom=%ld }",
-          this, aScreenExt.left, aScreenExt.top,
-          aScreenExt.right, aScreenExt.bottom));
+    ("0x%p   TSFTextStore::GetScreenExtInternal() succeeded: "
+     "aScreenExt={ left=%ld, top=%ld, right=%ld, bottom=%ld }",
+     this, aScreenExt.left, aScreenExt.top,
+     aScreenExt.right, aScreenExt.bottom));
   return true;
 }
 
 STDMETHODIMP
 TSFTextStore::GetWnd(TsViewCookie vcView,
                      HWND* phwnd)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::GetWnd(vcView=%ld, phwnd=0x%p), "
-          "mWidget=0x%p",
-          this, vcView, phwnd, mWidget.get()));
+    ("0x%p TSFTextStore::GetWnd(vcView=%ld, phwnd=0x%p), "
+     "mWidget=0x%p",
+     this, vcView, phwnd, mWidget.get()));
 
   if (vcView != TEXTSTORE_DEFAULT_VIEW) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetWnd() FAILED due to "
-            "called with invalid view", this));
+      ("0x%p   TSFTextStore::GetWnd() FAILED due to "
+       "called with invalid view", this));
     return E_INVALIDARG;
   }
 
   if (!phwnd) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::GetScreenExt() FAILED due to "
-            "null argument", this));
+      ("0x%p   TSFTextStore::GetScreenExt() FAILED due to "
+       "null argument", this));
     return E_INVALIDARG;
   }
 
   *phwnd = mWidget ? mWidget->GetWindowHandle() : nullptr;
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::GetWnd() succeeded: *phwnd=0x%p",
-          this, static_cast<void*>(*phwnd)));
+    ("0x%p   TSFTextStore::GetWnd() succeeded: *phwnd=0x%p",
+     this, static_cast<void*>(*phwnd)));
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::InsertTextAtSelection(DWORD dwFlags,
                                     const WCHAR* pchText,
                                     ULONG cch,
                                     LONG* pacpStart,
                                     LONG* pacpEnd,
                                     TS_TEXTCHANGE* pChange)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::InsertTextAtSelection(dwFlags=%s, "
-          "pchText=0x%p \"%s\", cch=%lu, pacpStart=0x%p, pacpEnd=0x%p, "
-          "pChange=0x%p), IsComposing()=%s",
-          this, dwFlags == 0 ? "0" :
-                dwFlags == TF_IAS_NOQUERY ? "TF_IAS_NOQUERY" :
-                dwFlags == TF_IAS_QUERYONLY ? "TF_IAS_QUERYONLY" : "Unknown",
-          pchText,
-          pchText && cch ? NS_ConvertUTF16toUTF8(pchText, cch).get() : "",
-          cch, pacpStart, pacpEnd, pChange,
-          GetBoolName(mComposition.IsComposing())));
+    ("0x%p TSFTextStore::InsertTextAtSelection(dwFlags=%s, "
+     "pchText=0x%p \"%s\", cch=%lu, pacpStart=0x%p, pacpEnd=0x%p, "
+     "pChange=0x%p), IsComposing()=%s",
+     this, dwFlags == 0 ? "0" :
+           dwFlags == TF_IAS_NOQUERY ? "TF_IAS_NOQUERY" :
+           dwFlags == TF_IAS_QUERYONLY ? "TF_IAS_QUERYONLY" : "Unknown",
+     pchText,
+     pchText && cch ? NS_ConvertUTF16toUTF8(pchText, cch).get() : "",
+     cch, pacpStart, pacpEnd, pChange,
+     GetBoolName(mComposition.IsComposing())));
 
   if (cch && !pchText) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
-            "null pchText", this));
+      ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
+       "null pchText", this));
     return E_INVALIDARG;
   }
 
   if (TS_IAS_QUERYONLY == dwFlags) {
     if (!IsReadLocked()) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-             ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
-              "not locked (read)", this));
+        ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
+         "not locked (read)", this));
       return TS_E_NOLOCK;
     }
 
     if (!pacpStart || !pacpEnd) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-             ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
-              "null argument", this));
+        ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
+         "null argument", this));
       return E_INVALIDARG;
     }
 
     // Get selection first
     Selection& selectionForTSF = SelectionForTSFRef();
     if (selectionForTSF.IsDirty()) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-             ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
-              "SelectionForTSFRef() failure", this));
+        ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
+         "SelectionForTSFRef() failure", this));
       return E_FAIL;
     }
 
     // Simulate text insertion
     *pacpStart = selectionForTSF.StartOffset();
     *pacpEnd = selectionForTSF.EndOffset();
     if (pChange) {
       pChange->acpStart = selectionForTSF.StartOffset();
       pChange->acpOldEnd = selectionForTSF.EndOffset();
       pChange->acpNewEnd =
         selectionForTSF.StartOffset() + static_cast<LONG>(cch);
     }
   } else {
     if (!IsReadWriteLocked()) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-             ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
-              "not locked (read-write)", this));
+        ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
+         "not locked (read-write)", this));
       return TS_E_NOLOCK;
     }
 
     if (!pChange) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-             ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
-              "null pChange", this));
+        ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
+         "null pChange", this));
       return E_INVALIDARG;
     }
 
     if (TS_IAS_NOQUERY != dwFlags && (!pacpStart || !pacpEnd)) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-             ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
-              "null argument", this));
+        ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
+         "null argument", this));
       return E_INVALIDARG;
     }
 
     if (!InsertTextAtSelectionInternal(nsDependentSubstring(pchText, cch),
                                        pChange)) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-             ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
-              "InsertTextAtSelectionInternal() failure", this));
+        ("0x%p   TSFTextStore::InsertTextAtSelection() FAILED due to "
+         "InsertTextAtSelectionInternal() failure", this));
       return E_FAIL;
     }
 
     if (TS_IAS_NOQUERY != dwFlags) {
       *pacpStart = pChange->acpStart;
       *pacpEnd = pChange->acpNewEnd;
     }
   }
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::InsertTextAtSelection() succeeded: "
-          "*pacpStart=%ld, *pacpEnd=%ld, "
-          "*pChange={ acpStart=%ld, acpOldEnd=%ld, acpNewEnd=%ld })",
-          this, pacpStart ? *pacpStart : 0, pacpEnd ? *pacpEnd : 0,
-          pChange ? pChange->acpStart: 0, pChange ? pChange->acpOldEnd : 0,
-          pChange ? pChange->acpNewEnd : 0));
+    ("0x%p   TSFTextStore::InsertTextAtSelection() succeeded: "
+     "*pacpStart=%ld, *pacpEnd=%ld, "
+     "*pChange={ acpStart=%ld, acpOldEnd=%ld, acpNewEnd=%ld })",
+     this, pacpStart ? *pacpStart : 0, pacpEnd ? *pacpEnd : 0,
+     pChange ? pChange->acpStart: 0, pChange ? pChange->acpOldEnd : 0,
+     pChange ? pChange->acpNewEnd : 0));
   return S_OK;
 }
 
 bool
 TSFTextStore::InsertTextAtSelectionInternal(const nsAString& aInsertStr,
                                             TS_TEXTCHANGE* aTextChange)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::InsertTextAtSelectionInternal("
-          "aInsertStr=\"%s\", aTextChange=0x%p), IsComposing=%s",
-          this, NS_ConvertUTF16toUTF8(aInsertStr).get(), aTextChange,
-          GetBoolName(mComposition.IsComposing())));
+    ("0x%p   TSFTextStore::InsertTextAtSelectionInternal("
+     "aInsertStr=\"%s\", aTextChange=0x%p), IsComposing=%s",
+     this, NS_ConvertUTF16toUTF8(aInsertStr).get(), aTextChange,
+     GetBoolName(mComposition.IsComposing())));
 
   Content& contentForTSF = ContentForTSFRef();
   if (!contentForTSF.IsInitialized()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::InsertTextAtSelectionInternal() failed "
-            "due to ContentForTSFRef() failure()", this));
+      ("0x%p   TSFTextStore::InsertTextAtSelectionInternal() failed "
+       "due to ContentForTSFRef() failure()", this));
     return false;
   }
 
   TS_SELECTION_ACP oldSelection = contentForTSF.Selection().ACP();
   if (!mComposition.IsComposing()) {
     // Use a temporary composition to contain the text
     PendingAction* compositionStart = mPendingActions.AppendElement();
     compositionStart->mType = PendingAction::COMPOSITION_START;
@@ -4010,105 +4011,105 @@ TSFTextStore::InsertTextAtSelectionInter
       oldSelection.acpEnd - oldSelection.acpStart;
     compositionStart->mAdjustSelection = false;
 
     PendingAction* compositionEnd = mPendingActions.AppendElement();
     compositionEnd->mType = PendingAction::COMPOSITION_END;
     compositionEnd->mData = aInsertStr;
 
     MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-            ("0x%p   TSFTextStore::InsertTextAtSelectionInternal() "
-             "appending pending compositionstart and compositionend... "
-             "PendingCompositionStart={ mSelectionStart=%d, "
-             "mSelectionLength=%d }, PendingCompositionEnd={ mData=\"%s\" "
-             "(Length()=%u) }",
-             this, compositionStart->mSelectionStart,
-             compositionStart->mSelectionLength,
-             NS_ConvertUTF16toUTF8(compositionEnd->mData).get(),
-             compositionEnd->mData.Length()));
+      ("0x%p   TSFTextStore::InsertTextAtSelectionInternal() "
+       "appending pending compositionstart and compositionend... "
+       "PendingCompositionStart={ mSelectionStart=%d, "
+       "mSelectionLength=%d }, PendingCompositionEnd={ mData=\"%s\" "
+       "(Length()=%u) }",
+       this, compositionStart->mSelectionStart,
+       compositionStart->mSelectionLength,
+       NS_ConvertUTF16toUTF8(compositionEnd->mData).get(),
+       compositionEnd->mData.Length()));
   }
 
   contentForTSF.ReplaceSelectedTextWith(aInsertStr);
 
   if (aTextChange) {
     aTextChange->acpStart = oldSelection.acpStart;
     aTextChange->acpOldEnd = oldSelection.acpEnd;
     aTextChange->acpNewEnd = contentForTSF.Selection().EndOffset();
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::InsertTextAtSelectionInternal() "
-          "succeeded: mWidget=0x%p, mWidget->Destroyed()=%s, aTextChange={ "
-          "acpStart=%ld, acpOldEnd=%ld, acpNewEnd=%ld }",
-          this, mWidget.get(),
-          GetBoolName(mWidget ? mWidget->Destroyed() : true),
-          aTextChange ? aTextChange->acpStart : 0,
-          aTextChange ? aTextChange->acpOldEnd : 0,
-          aTextChange ? aTextChange->acpNewEnd : 0));
+    ("0x%p   TSFTextStore::InsertTextAtSelectionInternal() "
+     "succeeded: mWidget=0x%p, mWidget->Destroyed()=%s, aTextChange={ "
+     "acpStart=%ld, acpOldEnd=%ld, acpNewEnd=%ld }",
+     this, mWidget.get(),
+     GetBoolName(mWidget ? mWidget->Destroyed() : true),
+     aTextChange ? aTextChange->acpStart : 0,
+     aTextChange ? aTextChange->acpOldEnd : 0,
+     aTextChange ? aTextChange->acpNewEnd : 0));
   return true;
 }
 
 STDMETHODIMP
 TSFTextStore::InsertEmbeddedAtSelection(DWORD dwFlags,
                                         IDataObject* pDataObject,
                                         LONG* pacpStart,
                                         LONG* pacpEnd,
                                         TS_TEXTCHANGE* pChange)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::InsertEmbeddedAtSelection() called "
-          "but not supported (E_NOTIMPL)", this));
+    ("0x%p TSFTextStore::InsertEmbeddedAtSelection() called "
+     "but not supported (E_NOTIMPL)", this));
 
   // embedded objects are not supported
   return E_NOTIMPL;
 }
 
 HRESULT
 TSFTextStore::RecordCompositionStartAction(ITfCompositionView* aComposition,
                                            ITfRange* aRange,
                                            bool aPreserveSelection)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::RecordCompositionStartAction("
-          "aComposition=0x%p, aRange=0x%p, aPreserveSelection=%s), "
-          "mComposition.mView=0x%p",
-          this, aComposition, aRange, GetBoolName(aPreserveSelection),
-          mComposition.mView.get()));
+    ("0x%p   TSFTextStore::RecordCompositionStartAction("
+     "aComposition=0x%p, aRange=0x%p, aPreserveSelection=%s), "
+     "mComposition.mView=0x%p",
+     this, aComposition, aRange, GetBoolName(aPreserveSelection),
+     mComposition.mView.get()));
 
   LONG start = 0, length = 0;
   HRESULT hr = GetRangeExtent(aRange, &start, &length);
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::RecordCompositionStartAction() FAILED "
-            "due to GetRangeExtent() failure", this));
+      ("0x%p   TSFTextStore::RecordCompositionStartAction() FAILED "
+       "due to GetRangeExtent() failure", this));
     return hr;
   }
 
   return RecordCompositionStartAction(aComposition, start, length,
                                       aPreserveSelection);
 }
 
 HRESULT
 TSFTextStore::RecordCompositionStartAction(ITfCompositionView* aComposition,
                                            LONG aStart,
                                            LONG aLength,
                                            bool aPreserveSelection)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::RecordCompositionStartAction("
-          "aComposition=0x%p, aStart=%d, aLength=%d, aPreserveSelection=%s), "
-          "mComposition.mView=0x%p",
-          this, aComposition, aStart, aLength, GetBoolName(aPreserveSelection),
-          mComposition.mView.get()));
+    ("0x%p   TSFTextStore::RecordCompositionStartAction("
+     "aComposition=0x%p, aStart=%d, aLength=%d, aPreserveSelection=%s), "
+     "mComposition.mView=0x%p",
+     this, aComposition, aStart, aLength, GetBoolName(aPreserveSelection),
+     mComposition.mView.get()));
 
   Content& contentForTSF = ContentForTSFRef();
   if (!contentForTSF.IsInitialized()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::RecordCompositionStartAction() FAILED "
-            "due to ContentForTSFRef() failure", this));
+      ("0x%p   TSFTextStore::RecordCompositionStartAction() FAILED "
+       "due to ContentForTSFRef() failure", this));
     return E_FAIL;
   }
 
   CompleteLastActionIfStillIncomplete();
 
   // TIP may have inserted text at selection before calling
   // OnStartComposition().  In this case, we've already created a pair of
   // pending compositionstart and pending compositionend.  If the pending
@@ -4119,38 +4120,38 @@ TSFTextStore::RecordCompositionStartActi
       WasTextInsertedWithoutCompositionAt(aStart, aLength)) {
     const PendingAction& pendingCompositionEnd = mPendingActions.LastElement();
     const PendingAction& pendingCompositionStart =
       mPendingActions[mPendingActions.Length() - 2];
     contentForTSF.RestoreCommittedComposition(
       aComposition, pendingCompositionStart, pendingCompositionEnd);
     mPendingActions.RemoveElementAt(mPendingActions.Length() - 1);
     MOZ_LOG(sTextStoreLog, LogLevel::Info,
-           ("0x%p   TSFTextStore::RecordCompositionStartAction() "
-            "succeeded: restoring the committed string as composing string, "
-            "mComposition={ mStart=%ld, mString.Length()=%ld, "
-            "mSelectionForTSF={ acpStart=%ld, acpEnd=%ld, style.ase=%s, "
-            "style.fInterimChar=%s } }",
-            this, mComposition.mStart, mComposition.mString.Length(),
-            mSelectionForTSF.StartOffset(), mSelectionForTSF.EndOffset(),
-            GetActiveSelEndName(mSelectionForTSF.ActiveSelEnd()),
-            GetBoolName(mSelectionForTSF.IsInterimChar())));
+      ("0x%p   TSFTextStore::RecordCompositionStartAction() "
+       "succeeded: restoring the committed string as composing string, "
+       "mComposition={ mStart=%ld, mString.Length()=%ld, "
+       "mSelectionForTSF={ acpStart=%ld, acpEnd=%ld, style.ase=%s, "
+       "style.fInterimChar=%s } }",
+       this, mComposition.mStart, mComposition.mString.Length(),
+       mSelectionForTSF.StartOffset(), mSelectionForTSF.EndOffset(),
+       GetActiveSelEndName(mSelectionForTSF.ActiveSelEnd()),
+       GetBoolName(mSelectionForTSF.IsInterimChar())));
     return S_OK;
   }
 
   PendingAction* action = mPendingActions.AppendElement();
   action->mType = PendingAction::COMPOSITION_START;
   action->mSelectionStart = aStart;
   action->mSelectionLength = aLength;
 
   Selection& selectionForTSF = SelectionForTSFRef();
   if (selectionForTSF.IsDirty()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::RecordCompositionStartAction() FAILED "
-            "due to SelectionForTSFRef() failure", this));
+      ("0x%p   TSFTextStore::RecordCompositionStartAction() FAILED "
+       "due to SelectionForTSFRef() failure", this));
     action->mAdjustSelection = true;
   } else if (selectionForTSF.MinOffset() != aStart ||
              selectionForTSF.MaxOffset() != aStart + aLength) {
     // If new composition range is different from current selection range,
     // we need to set selection before dispatching compositionstart event.
     action->mAdjustSelection = true;
   } else {
     // We shouldn't dispatch selection set event before dispatching
@@ -4159,48 +4160,48 @@ TSFTextStore::RecordCompositionStartActi
     // it are lossy data of HTML contents.
     action->mAdjustSelection = false;
   }
 
   contentForTSF.StartComposition(aComposition, *action, aPreserveSelection);
   action->mData = mComposition.mString;
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::RecordCompositionStartAction() succeeded: "
-          "mComposition={ mStart=%ld, mString.Length()=%ld, "
-          "mSelectionForTSF={ acpStart=%ld, acpEnd=%ld, style.ase=%s, "
-          "style.fInterimChar=%s } }",
-          this, mComposition.mStart, mComposition.mString.Length(),
-          mSelectionForTSF.StartOffset(), mSelectionForTSF.EndOffset(),
-          GetActiveSelEndName(mSelectionForTSF.ActiveSelEnd()),
-          GetBoolName(mSelectionForTSF.IsInterimChar())));
+    ("0x%p   TSFTextStore::RecordCompositionStartAction() succeeded: "
+     "mComposition={ mStart=%ld, mString.Length()=%ld, "
+     "mSelectionForTSF={ acpStart=%ld, acpEnd=%ld, style.ase=%s, "
+     "style.fInterimChar=%s } }",
+     this, mComposition.mStart, mComposition.mString.Length(),
+     mSelectionForTSF.StartOffset(), mSelectionForTSF.EndOffset(),
+     GetActiveSelEndName(mSelectionForTSF.ActiveSelEnd()),
+     GetBoolName(mSelectionForTSF.IsInterimChar())));
   return S_OK;
 }
 
 HRESULT
 TSFTextStore::RecordCompositionEndAction()
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::RecordCompositionEndAction(), "
-          "mComposition={ mView=0x%p, mString=\"%s\" }",
-          this, mComposition.mView.get(),
-          NS_ConvertUTF16toUTF8(mComposition.mString).get()));
+    ("0x%p   TSFTextStore::RecordCompositionEndAction(), "
+     "mComposition={ mView=0x%p, mString=\"%s\" }",
+     this, mComposition.mView.get(),
+     NS_ConvertUTF16toUTF8(mComposition.mString).get()));
 
   MOZ_ASSERT(mComposition.IsComposing());
 
   CompleteLastActionIfStillIncomplete();
   PendingAction* action = mPendingActions.AppendElement();
   action->mType = PendingAction::COMPOSITION_END;
   action->mData = mComposition.mString;
 
   Content& contentForTSF = ContentForTSFRef();
   if (!contentForTSF.IsInitialized()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::RecordCompositionEndAction() FAILED due "
-            "to ContentForTSFRef() failure", this));
+      ("0x%p   TSFTextStore::RecordCompositionEndAction() FAILED due "
+       "to ContentForTSFRef() failure", this));
     return E_FAIL;
   }
   contentForTSF.EndComposition(*action);
 
   // If this composition was restart but the composition doesn't modify
   // anything, we should remove the pending composition for preventing to
   // dispatch redundant composition events.
   for (size_t i = mPendingActions.Length(), j = 1; i > 0; --i, ++j) {
@@ -4215,214 +4216,214 @@ TSFTextStore::RecordCompositionEndAction
         setSelection->mType = PendingAction::SET_SELECTION;
         setSelection->mSelectionStart = pendingAction.mSelectionStart;
         setSelection->mSelectionLength = pendingAction.mSelectionLength;
         setSelection->mSelectionReversed = false;
       }
       // Remove the redundant pending composition.
       mPendingActions.RemoveElementsAt(i - 1, j);
       MOZ_LOG(sTextStoreLog, LogLevel::Info,
-             ("0x%p   TSFTextStore::RecordCompositionEndAction(), "
-              "succeeded, but the composition was canceled due to redundant",
-              this));
+        ("0x%p   TSFTextStore::RecordCompositionEndAction(), "
+         "succeeded, but the composition was canceled due to redundant",
+         this));
       return S_OK;
     }
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::RecordCompositionEndAction(), succeeded",
-          this));
+    ("0x%p   TSFTextStore::RecordCompositionEndAction(), succeeded",
+     this));
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::OnStartComposition(ITfCompositionView* pComposition,
                                  BOOL* pfOk)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::OnStartComposition(pComposition=0x%p, "
-          "pfOk=0x%p), mComposition.mView=0x%p",
-          this, pComposition, pfOk, mComposition.mView.get()));
+    ("0x%p TSFTextStore::OnStartComposition(pComposition=0x%p, "
+     "pfOk=0x%p), mComposition.mView=0x%p",
+     this, pComposition, pfOk, mComposition.mView.get()));
 
   AutoPendingActionAndContentFlusher flusher(this);
 
   *pfOk = FALSE;
 
   // Only one composition at a time
   if (mComposition.IsComposing()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::OnStartComposition() FAILED due to "
-            "there is another composition already (but returns S_OK)", this));
+      ("0x%p   TSFTextStore::OnStartComposition() FAILED due to "
+       "there is another composition already (but returns S_OK)", this));
     return S_OK;
   }
 
   RefPtr<ITfRange> range;
   HRESULT hr = pComposition->GetRange(getter_AddRefs(range));
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::OnStartComposition() FAILED due to "
-            "pComposition->GetRange() failure", this));
+      ("0x%p   TSFTextStore::OnStartComposition() FAILED due to "
+       "pComposition->GetRange() failure", this));
     return hr;
   }
   hr = RecordCompositionStartAction(pComposition, range, false);
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::OnStartComposition() FAILED due to "
-            "RecordCompositionStartAction() failure", this));
+      ("0x%p   TSFTextStore::OnStartComposition() FAILED due to "
+       "RecordCompositionStartAction() failure", this));
     return hr;
   }
 
   *pfOk = TRUE;
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::OnStartComposition() succeeded", this));
+    ("0x%p   TSFTextStore::OnStartComposition() succeeded", this));
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::OnUpdateComposition(ITfCompositionView* pComposition,
                                   ITfRange* pRangeNew)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::OnUpdateComposition(pComposition=0x%p, "
-          "pRangeNew=0x%p), mComposition.mView=0x%p",
-          this, pComposition, pRangeNew, mComposition.mView.get()));
+    ("0x%p TSFTextStore::OnUpdateComposition(pComposition=0x%p, "
+     "pRangeNew=0x%p), mComposition.mView=0x%p",
+     this, pComposition, pRangeNew, mComposition.mView.get()));
 
   AutoPendingActionAndContentFlusher flusher(this);
 
   if (!mDocumentMgr || !mContext) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::OnUpdateComposition() FAILED due to "
-            "not ready for the composition", this));
+      ("0x%p   TSFTextStore::OnUpdateComposition() FAILED due to "
+       "not ready for the composition", this));
     return E_UNEXPECTED;
   }
   if (!mComposition.IsComposing()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::OnUpdateComposition() FAILED due to "
-            "no active composition", this));
+      ("0x%p   TSFTextStore::OnUpdateComposition() FAILED due to "
+       "no active composition", this));
     return E_UNEXPECTED;
   }
   if (mComposition.mView != pComposition) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::OnUpdateComposition() FAILED due to "
-            "different composition view specified", this));
+      ("0x%p   TSFTextStore::OnUpdateComposition() FAILED due to "
+       "different composition view specified", this));
     return E_UNEXPECTED;
   }
 
   // pRangeNew is null when the update is not complete
   if (!pRangeNew) {
     PendingAction* action = LastOrNewPendingCompositionUpdate();
     action->mIncomplete = true;
     MOZ_LOG(sTextStoreLog, LogLevel::Info,
-           ("0x%p   TSFTextStore::OnUpdateComposition() succeeded but "
-            "not complete", this));
+      ("0x%p   TSFTextStore::OnUpdateComposition() succeeded but "
+       "not complete", this));
     return S_OK;
   }
 
   HRESULT hr = RestartCompositionIfNecessary(pRangeNew);
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::OnUpdateComposition() FAILED due to "
-            "RestartCompositionIfNecessary() failure", this));
+      ("0x%p   TSFTextStore::OnUpdateComposition() FAILED due to "
+       "RestartCompositionIfNecessary() failure", this));
     return hr;
   }
 
   hr = RecordCompositionUpdateAction();
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::OnUpdateComposition() FAILED due to "
-            "RecordCompositionUpdateAction() failure", this));
+      ("0x%p   TSFTextStore::OnUpdateComposition() FAILED due to "
+       "RecordCompositionUpdateAction() failure", this));
     return hr;
   }
 
   if (MOZ_LOG_TEST(sTextStoreLog, LogLevel::Info)) {
     Selection& selectionForTSF = SelectionForTSFRef();
     if (selectionForTSF.IsDirty()) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-             ("0x%p   TSFTextStore::OnUpdateComposition() FAILED due to "
-              "SelectionForTSFRef() failure", this));
+        ("0x%p   TSFTextStore::OnUpdateComposition() FAILED due to "
+         "SelectionForTSFRef() failure", this));
       return E_FAIL;
     }
     MOZ_LOG(sTextStoreLog, LogLevel::Info,
-           ("0x%p   TSFTextStore::OnUpdateComposition() succeeded: "
-            "mComposition={ mStart=%ld, mString=\"%s\" }, "
-            "SelectionForTSFRef()={ acpStart=%ld, acpEnd=%ld, style.ase=%s }",
-            this, mComposition.mStart,
-            NS_ConvertUTF16toUTF8(mComposition.mString).get(),
-            selectionForTSF.StartOffset(), selectionForTSF.EndOffset(),
-            GetActiveSelEndName(selectionForTSF.ActiveSelEnd())));
+      ("0x%p   TSFTextStore::OnUpdateComposition() succeeded: "
+       "mComposition={ mStart=%ld, mString=\"%s\" }, "
+       "SelectionForTSFRef()={ acpStart=%ld, acpEnd=%ld, style.ase=%s }",
+       this, mComposition.mStart,
+       NS_ConvertUTF16toUTF8(mComposition.mString).get(),
+       selectionForTSF.StartOffset(), selectionForTSF.EndOffset(),
+       GetActiveSelEndName(selectionForTSF.ActiveSelEnd())));
   }
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::OnEndComposition(ITfCompositionView* pComposition)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::OnEndComposition(pComposition=0x%p), "
-          "mComposition={ mView=0x%p, mString=\"%s\" }",
-          this, pComposition, mComposition.mView.get(),
-          NS_ConvertUTF16toUTF8(mComposition.mString).get()));
+    ("0x%p TSFTextStore::OnEndComposition(pComposition=0x%p), "
+     "mComposition={ mView=0x%p, mString=\"%s\" }",
+     this, pComposition, mComposition.mView.get(),
+     NS_ConvertUTF16toUTF8(mComposition.mString).get()));
 
   AutoPendingActionAndContentFlusher flusher(this);
 
   if (!mComposition.IsComposing()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::OnEndComposition() FAILED due to "
-            "no active composition", this));
+      ("0x%p   TSFTextStore::OnEndComposition() FAILED due to "
+       "no active composition", this));
     return E_UNEXPECTED;
   }
 
   if (mComposition.mView != pComposition) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::OnEndComposition() FAILED due to "
-            "different composition view specified", this));
+      ("0x%p   TSFTextStore::OnEndComposition() FAILED due to "
+       "different composition view specified", this));
     return E_UNEXPECTED;
   }
 
   HRESULT hr = RecordCompositionEndAction();
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::OnEndComposition() FAILED due to "
-            "RecordCompositionEndAction() failure", this));
+      ("0x%p   TSFTextStore::OnEndComposition() FAILED due to "
+       "RecordCompositionEndAction() failure", this));
     return hr;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::OnEndComposition(), succeeded", this));
+    ("0x%p   TSFTextStore::OnEndComposition(), succeeded", this));
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::AdviseMouseSink(ITfRangeACP* range,
                               ITfMouseSink* pSink,
                               DWORD* pdwCookie)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::AdviseMouseSink(range=0x%p, pSink=0x%p, "
-          "pdwCookie=0x%p)", this, range, pSink, pdwCookie));
+    ("0x%p TSFTextStore::AdviseMouseSink(range=0x%p, pSink=0x%p, "
+     "pdwCookie=0x%p)", this, range, pSink, pdwCookie));
 
   if (!pdwCookie) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::AdviseMouseSink() FAILED due to the "
-            "pdwCookie is null", this));
+      ("0x%p   TSFTextStore::AdviseMouseSink() FAILED due to the "
+       "pdwCookie is null", this));
     return E_INVALIDARG;
   }
   // Initialize the result with invalid cookie for safety.
   *pdwCookie = MouseTracker::kInvalidCookie;
 
   if (!range) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::AdviseMouseSink() FAILED due to the "
-            "range is null", this));
+      ("0x%p   TSFTextStore::AdviseMouseSink() FAILED due to the "
+       "range is null", this));
     return E_INVALIDARG;
   }
   if (!pSink) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::AdviseMouseSink() FAILED due to the "
-            "pSink is null", this));
+      ("0x%p   TSFTextStore::AdviseMouseSink() FAILED due to the "
+       "pSink is null", this));
     return E_INVALIDARG;
   }
 
   // Looking for an unusing tracker.
   MouseTracker* tracker = nullptr;
   for (size_t i = 0; i < mMouseTrackers.Length(); i++) {
     if (mMouseTrackers[i].IsUsing()) {
       continue;
@@ -4431,83 +4432,83 @@ TSFTextStore::AdviseMouseSink(ITfRangeAC
   }
   // If there is no unusing tracker, create new one.
   // XXX Should we make limitation of the number of installs?
   if (!tracker) {
     tracker = mMouseTrackers.AppendElement();
     HRESULT hr = tracker->Init(this);
     if (FAILED(hr)) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-             ("0x%p   TSFTextStore::AdviseMouseSink() FAILED due to "
-              "failure of MouseTracker::Init()", this));
+        ("0x%p   TSFTextStore::AdviseMouseSink() FAILED due to "
+         "failure of MouseTracker::Init()", this));
       return hr;
     }
   }
   HRESULT hr = tracker->AdviseSink(this, range, pSink);
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::AdviseMouseSink() FAILED due to failure "
-            "of MouseTracker::Init()", this));
+      ("0x%p   TSFTextStore::AdviseMouseSink() FAILED due to failure "
+       "of MouseTracker::Init()", this));
     return hr;
   }
   *pdwCookie = tracker->Cookie();
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::AdviseMouseSink(), succeeded, "
-          "*pdwCookie=%d", this, *pdwCookie));
+    ("0x%p   TSFTextStore::AdviseMouseSink(), succeeded, "
+     "*pdwCookie=%d", this, *pdwCookie));
   return S_OK;
 }
 
 STDMETHODIMP
 TSFTextStore::UnadviseMouseSink(DWORD dwCookie)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p TSFTextStore::UnadviseMouseSink(dwCookie=%d)",
-          this, dwCookie));
+    ("0x%p TSFTextStore::UnadviseMouseSink(dwCookie=%d)",
+     this, dwCookie));
   if (dwCookie == MouseTracker::kInvalidCookie) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::UnadviseMouseSink() FAILED due to "
-            "the cookie is invalid value", this));
+      ("0x%p   TSFTextStore::UnadviseMouseSink() FAILED due to "
+       "the cookie is invalid value", this));
     return E_INVALIDARG;
   }
   // The cookie value must be an index of mMouseTrackers.
   // We can use this shortcut for now.
   if (static_cast<size_t>(dwCookie) >= mMouseTrackers.Length()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::UnadviseMouseSink() FAILED due to "
-            "the cookie is too large value", this));
+      ("0x%p   TSFTextStore::UnadviseMouseSink() FAILED due to "
+       "the cookie is too large value", this));
     return E_INVALIDARG;
   }
   MouseTracker& tracker = mMouseTrackers[dwCookie];
   if (!tracker.IsUsing()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::UnadviseMouseSink() FAILED due to "
-            "the found tracker uninstalled already", this));
+      ("0x%p   TSFTextStore::UnadviseMouseSink() FAILED due to "
+       "the found tracker uninstalled already", this));
     return E_INVALIDARG;
   }
   tracker.UnadviseSink();
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::UnadviseMouseSink(), succeeded", this));
+    ("0x%p   TSFTextStore::UnadviseMouseSink(), succeeded", this));
   return S_OK;
 }
 
 // static
 nsresult
 TSFTextStore::OnFocusChange(bool aGotFocus,
                             nsWindowBase* aFocusedWidget,
                             const InputContext& aContext)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("  TSFTextStore::OnFocusChange(aGotFocus=%s, "
-          "aFocusedWidget=0x%p, aContext={ mIMEState={ mEnabled=%s }, "
-          "mHTMLInputType=\"%s\" }), "
-          "sThreadMgr=0x%p, sEnabledTextStore=0x%p",
-          GetBoolName(aGotFocus), aFocusedWidget,
-          GetIMEEnabledName(aContext.mIMEState.mEnabled),
-          NS_ConvertUTF16toUTF8(aContext.mHTMLInputType).get(),
-          sThreadMgr.get(), sEnabledTextStore.get()));
+    ("  TSFTextStore::OnFocusChange(aGotFocus=%s, "
+     "aFocusedWidget=0x%p, aContext={ mIMEState={ mEnabled=%s }, "
+     "mHTMLInputType=\"%s\" }), "
+     "sThreadMgr=0x%p, sEnabledTextStore=0x%p",
+     GetBoolName(aGotFocus), aFocusedWidget,
+     GetIMEEnabledName(aContext.mIMEState.mEnabled),
+     NS_ConvertUTF16toUTF8(aContext.mHTMLInputType).get(),
+     sThreadMgr.get(), sEnabledTextStore.get()));
 
   if (NS_WARN_IF(!IsInTSFMode())) {
     return NS_ERROR_NOT_AVAILABLE;
   }
 
   RefPtr<ITfDocumentMgr> prevFocusedDocumentMgr;
 
   // If currently sEnableTextStore has focus, notifies TSF of losing focus.
@@ -4529,28 +4530,28 @@ TSFTextStore::OnFocusChange(bool aGotFoc
   }
 
   // If this is a notification of blur, move focus to the dummy document
   // manager.
   if (!aGotFocus || !aContext.mIMEState.IsEditable()) {
     HRESULT hr = sThreadMgr->SetFocus(sDisabledDocumentMgr);
     if (NS_WARN_IF(FAILED(hr))) {
       MOZ_LOG(sTextStoreLog, LogLevel::Error,
-             ("  TSFTextStore::OnFocusChange() FAILED due to "
-              "ITfThreadMgr::SetFocus() failure"));
+        ("  TSFTextStore::OnFocusChange() FAILED due to "
+         "ITfThreadMgr::SetFocus() failure"));
       return NS_ERROR_FAILURE;
     }
     return NS_OK;
   }
 
   // If an editor is getting focus, create new TextStore and set focus.
   if (NS_WARN_IF(!CreateAndSetFocus(aFocusedWidget, aContext))) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("  TSFTextStore::OnFocusChange() FAILED due to "
-            "ITfThreadMgr::CreateAndSetFocus() failure"));
+      ("  TSFTextStore::OnFocusChange() FAILED due to "
+       "ITfThreadMgr::CreateAndSetFocus() failure"));
     // If setting focus, we should destroy the TextStore completely because
     // it causes memory leak.
     if (sEnabledTextStore) {
       sEnabledTextStore->Destroy();
       sEnabledTextStore = nullptr;
     }
     return NS_ERROR_FAILURE;
   }
@@ -4564,51 +4565,51 @@ TSFTextStore::CreateAndSetFocus(nsWindow
 {
   // TSF might do something which causes that we need to access static methods
   // of TSFTextStore.  At that time, sEnabledTextStore may be necessary.
   // So, we should set sEnabledTextStore directly.
   RefPtr<TSFTextStore> textStore = new TSFTextStore();
   sEnabledTextStore = textStore;
   if (NS_WARN_IF(!textStore->Init(aFocusedWidget))) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("  TSFTextStore::CreateAndSetFocus() FAILED due to "
-            "TSFTextStore::Init() failure"));
+      ("  TSFTextStore::CreateAndSetFocus() FAILED due to "
+       "TSFTextStore::Init() failure"));
     return false;
   }
   if (NS_WARN_IF(!textStore->mDocumentMgr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("  TSFTextStore::CreateAndSetFocus() FAILED due to "
-            "invalid TSFTextStore::mDocumentMgr"));
+      ("  TSFTextStore::CreateAndSetFocus() FAILED due to "
+       "invalid TSFTextStore::mDocumentMgr"));
     return false;
   }
   if (aContext.mIMEState.mEnabled == IMEState::PASSWORD) {
     MarkContextAsKeyboardDisabled(textStore->mContext);
     RefPtr<ITfContext> topContext;
     textStore->mDocumentMgr->GetTop(getter_AddRefs(topContext));
     if (topContext && topContext != textStore->mContext) {
       MarkContextAsKeyboardDisabled(topContext);
     }
   }
   HRESULT hr = sThreadMgr->SetFocus(textStore->mDocumentMgr);
   if (NS_WARN_IF(FAILED(hr))) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("  TSFTextStore::CreateAndSetFocus() FAILED due to "
-            "ITfTheadMgr::SetFocus() failure"));
+      ("  TSFTextStore::CreateAndSetFocus() FAILED due to "
+       "ITfTheadMgr::SetFocus() failure"));
     return false;
   }
   // Use AssociateFocus() for ensuring that any native focus event
   // never steal focus from our documentMgr.
   RefPtr<ITfDocumentMgr> prevFocusedDocumentMgr;
   hr = sThreadMgr->AssociateFocus(aFocusedWidget->GetWindowHandle(),
                                   textStore->mDocumentMgr,
                                   getter_AddRefs(prevFocusedDocumentMgr));
   if (NS_WARN_IF(FAILED(hr))) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("  TSFTextStore::CreateAndSetFocus() FAILED due to "
-            "ITfTheadMgr::AssociateFocus() failure"));
+      ("  TSFTextStore::CreateAndSetFocus() FAILED due to "
+       "ITfTheadMgr::AssociateFocus() failure"));
     return false;
   }
   textStore->SetInputScope(aContext.mHTMLInputType,
                            aContext.mHTMLInputInputmode);
 
   if (textStore->mSink) {
     MOZ_LOG(sTextStoreLog, LogLevel::Info,
       ("  TSFTextStore::CreateAndSetFocus(), calling "
@@ -4638,35 +4639,35 @@ TSFTextStore::GetIMEUpdatePreference()
 }
 
 nsresult
 TSFTextStore::OnTextChangeInternal(const IMENotification& aIMENotification)
 {
   const TextChangeDataBase& textChangeData = aIMENotification.mTextChangeData;
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::OnTextChangeInternal(aIMENotification={ "
-          "mMessage=0x%08X, mTextChangeData={ mStartOffset=%lu, "
-          "mRemovedEndOffset=%lu, mAddedEndOffset=%lu, "
-          "mCausedOnlyByComposition=%s, "
-          "mIncludingChangesDuringComposition=%s, "
-          "mIncludingChangesWithoutComposition=%s }), "
-          "mDestroyed=%s, mSink=0x%p, mSinkMask=%s, "
-          "mComposition.IsComposing()=%s",
-          this, aIMENotification.mMessage,
-          textChangeData.mStartOffset,
-          textChangeData.mRemovedEndOffset,
-          textChangeData.mAddedEndOffset,
-          GetBoolName(textChangeData.mCausedOnlyByComposition),
-          GetBoolName(textChangeData.mIncludingChangesDuringComposition),
-          GetBoolName(textChangeData.mIncludingChangesWithoutComposition),
-          GetBoolName(mDestroyed),
-          mSink.get(),
-          GetSinkMaskNameStr(mSinkMask).get(),
-          GetBoolName(mComposition.IsComposing())));
+    ("0x%p   TSFTextStore::OnTextChangeInternal(aIMENotification={ "
+     "mMessage=0x%08X, mTextChangeData={ mStartOffset=%lu, "
+     "mRemovedEndOffset=%lu, mAddedEndOffset=%lu, "
+     "mCausedOnlyByComposition=%s, "
+     "mIncludingChangesDuringComposition=%s, "
+     "mIncludingChangesWithoutComposition=%s }), "
+     "mDestroyed=%s, mSink=0x%p, mSinkMask=%s, "
+     "mComposition.IsComposing()=%s",
+     this, aIMENotification.mMessage,
+     textChangeData.mStartOffset,
+     textChangeData.mRemovedEndOffset,
+     textChangeData.mAddedEndOffset,
+     GetBoolName(textChangeData.mCausedOnlyByComposition),
+     GetBoolName(textChangeData.mIncludingChangesDuringComposition),
+     GetBoolName(textChangeData.mIncludingChangesWithoutComposition),
+     GetBoolName(mDestroyed),
+     mSink.get(),
+     GetSinkMaskNameStr(mSinkMask).get(),
+     GetBoolName(mComposition.IsComposing())));
 
   if (mDestroyed) {
     // If this instance is already destroyed, we shouldn't notify TSF of any
     // changes.
     return NS_OK;
   }
 
   mDeferNotifyingTSF = false;
@@ -4703,73 +4704,73 @@ TSFTextStore::NotifyTSFOfTextChange()
 
   // First, forget cached selection.
   mSelectionForTSF.MarkDirty();
 
   // For making it safer, we should check if there is a valid sink to receive
   // text change notification.
   if (NS_WARN_IF(!mSink) || NS_WARN_IF(!(mSinkMask & TS_AS_TEXT_CHANGE))) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::NotifyTSFOfTextChange() FAILED due to "
-            "mSink is not ready to call ITextStoreACPSink::OnTextChange()...",
-            this));
+      ("0x%p   TSFTextStore::NotifyTSFOfTextChange() FAILED due to "
+       "mSink is not ready to call ITextStoreACPSink::OnTextChange()...",
+       this));
     mPendingTextChangeData.Clear();
     return;
   }
 
   if (NS_WARN_IF(!mPendingTextChangeData.IsInInt32Range())) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::NotifyTSFOfTextChange() FAILED due to "
-            "offset is too big for calling "
-            "ITextStoreACPSink::OnTextChange()...",
-            this));
+      ("0x%p   TSFTextStore::NotifyTSFOfTextChange() FAILED due to "
+       "offset is too big for calling "
+       "ITextStoreACPSink::OnTextChange()...",
+       this));
     mPendingTextChangeData.Clear();
     return;
    }
 
   TS_TEXTCHANGE textChange;
   textChange.acpStart =
     static_cast<LONG>(mPendingTextChangeData.mStartOffset);
   textChange.acpOldEnd =
     static_cast<LONG>(mPendingTextChangeData.mRemovedEndOffset);
   textChange.acpNewEnd =
     static_cast<LONG>(mPendingTextChangeData.mAddedEndOffset);
   mPendingTextChangeData.Clear();
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::NotifyTSFOfTextChange(), calling "
-          "ITextStoreACPSink::OnTextChange(0, { acpStart=%ld, acpOldEnd=%ld, "
-          "acpNewEnd=%ld })...", this, textChange.acpStart,
-          textChange.acpOldEnd, textChange.acpNewEnd));
+    ("0x%p   TSFTextStore::NotifyTSFOfTextChange(), calling "
+     "ITextStoreACPSink::OnTextChange(0, { acpStart=%ld, acpOldEnd=%ld, "
+     "acpNewEnd=%ld })...", this, textChange.acpStart,
+     textChange.acpOldEnd, textChange.acpNewEnd));
   mSink->OnTextChange(0, &textChange);
 }
 
 nsresult
 TSFTextStore::OnSelectionChangeInternal(const IMENotification& aIMENotification)
 {
   const SelectionChangeDataBase& selectionChangeData =
     aIMENotification.mSelectionChangeData;
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::OnSelectionChangeInternal("
-          "aIMENotification={ mSelectionChangeData={ mOffset=%lu, "
-          "Length()=%lu, mReversed=%s, mWritingMode=%s, "
-          "mCausedByComposition=%s, mCausedBySelectionEvent=%s, "
-          "mOccurredDuringComposition=%s } }), mDestroyed=%s, "
-          "mSink=0x%p, mSinkMask=%s, mIsRecordingActionsWithoutLock=%s, "
-          "mComposition.IsComposing()=%s",
-          this, selectionChangeData.mOffset, selectionChangeData.Length(),
-          GetBoolName(selectionChangeData.mReversed),
-          GetWritingModeName(selectionChangeData.GetWritingMode()).get(),
-          GetBoolName(selectionChangeData.mCausedByComposition),
-          GetBoolName(selectionChangeData.mCausedBySelectionEvent),
-          GetBoolName(selectionChangeData.mOccurredDuringComposition),
-          GetBoolName(mDestroyed),
-          mSink.get(), GetSinkMaskNameStr(mSinkMask).get(),
-          GetBoolName(mIsRecordingActionsWithoutLock),
-          GetBoolName(mComposition.IsComposing())));
+    ("0x%p   TSFTextStore::OnSelectionChangeInternal("
+     "aIMENotification={ mSelectionChangeData={ mOffset=%lu, "
+     "Length()=%lu, mReversed=%s, mWritingMode=%s, "
+     "mCausedByComposition=%s, mCausedBySelectionEvent=%s, "
+     "mOccurredDuringComposition=%s } }), mDestroyed=%s, "
+     "mSink=0x%p, mSinkMask=%s, mIsRecordingActionsWithoutLock=%s, "
+     "mComposition.IsComposing()=%s",
+     this, selectionChangeData.mOffset, selectionChangeData.Length(),
+     GetBoolName(selectionChangeData.mReversed),
+     GetWritingModeName(selectionChangeData.GetWritingMode()).get(),
+     GetBoolName(selectionChangeData.mCausedByComposition),
+     GetBoolName(selectionChangeData.mCausedBySelectionEvent),
+     GetBoolName(selectionChangeData.mOccurredDuringComposition),
+     GetBoolName(mDestroyed),
+     mSink.get(), GetSinkMaskNameStr(mSinkMask).get(),
+     GetBoolName(mIsRecordingActionsWithoutLock),
+     GetBoolName(mComposition.IsComposing())));
 
   if (mDestroyed) {
     // If this instance is already destroyed, we shouldn't notify TSF of any
     // changes.
     return NS_OK;
   }
 
   mDeferNotifyingTSF = false;
@@ -4799,30 +4800,30 @@ TSFTextStore::NotifyTSFOfSelectionChange
   // of this selection change.
   if (!mSelectionForTSF.SetSelection(
                           mPendingSelectionChangeData.mOffset,
                           mPendingSelectionChangeData.Length(),
                           mPendingSelectionChangeData.mReversed,
                           mPendingSelectionChangeData.GetWritingMode())) {
     mPendingSelectionChangeData.Clear();
     MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-           ("0x%p   TSFTextStore::NotifyTSFOfSelectionChange(), "
-            "selection isn't actually changed.", this));
+      ("0x%p   TSFTextStore::NotifyTSFOfSelectionChange(), "
+       "selection isn't actually changed.", this));
     return;
   }
 
   mPendingSelectionChangeData.Clear();
 
   if (!mSink || !(mSinkMask & TS_AS_SEL_CHANGE)) {
     return;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::NotifyTSFOfSelectionChange(), calling "
-          "ITextStoreACPSink::OnSelectionChange()...", this));
+    ("0x%p   TSFTextStore::NotifyTSFOfSelectionChange(), calling "
+     "ITextStoreACPSink::OnSelectionChange()...", this));
   mSink->OnSelectionChange();
 }
 
 nsresult
 TSFTextStore::OnLayoutChangeInternal()
 {
   if (mDestroyed) {
     // If this instance is already destroyed, we shouldn't notify TSF of any
@@ -4836,25 +4837,25 @@ TSFTextStore::OnLayoutChangeInternal()
   mDeferNotifyingTSF = false;
 
   nsresult rv = NS_OK;
 
   // We need to notify TSF of layout change even if the document is locked.
   // So, don't use MaybeFlushPendingNotifications() for flushing pending
   // layout change.
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::OnLayoutChangeInternal(), calling "
-          "NotifyTSFOfLayoutChange()...", this));
+    ("0x%p   TSFTextStore::OnLayoutChangeInternal(), calling "
+     "NotifyTSFOfLayoutChange()...", this));
   if (NS_WARN_IF(!NotifyTSFOfLayoutChange())) {
     rv = NS_ERROR_FAILURE;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::OnLayoutChangeInternal(), calling "
-          "MaybeFlushPendingNotifications()...", this));
+    ("0x%p   TSFTextStore::OnLayoutChangeInternal(), calling "
+     "MaybeFlushPendingNotifications()...", this));
   MaybeFlushPendingNotifications();
 
   return rv;
 }
 
 bool
 TSFTextStore::NotifyTSFOfLayoutChange()
 {
@@ -4882,84 +4883,84 @@ TSFTextStore::NotifyTSFOfLayoutChange()
   // if there is.
   MaybeDestroyNativeCaret();
 
   // This method should return true if either way succeeds.
   bool ret = true;
 
   if (mSink) {
     MOZ_LOG(sTextStoreLog, LogLevel::Info,
-           ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
-            "calling ITextStoreACPSink::OnLayoutChange()...",
-            this));
+      ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
+       "calling ITextStoreACPSink::OnLayoutChange()...",
+       this));
     HRESULT hr = mSink->OnLayoutChange(TS_LC_CHANGE, TEXTSTORE_DEFAULT_VIEW);
     MOZ_LOG(sTextStoreLog, LogLevel::Info,
-           ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
-            "called ITextStoreACPSink::OnLayoutChange()",
-            this));
+      ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
+       "called ITextStoreACPSink::OnLayoutChange()",
+       this));
     ret = SUCCEEDED(hr);
   }
 
   // The layout change caused by composition string change should cause
   // calling ITfContextOwnerServices::OnLayoutChange() too.
   if (returnedNoLayoutError && mContext) {
     RefPtr<ITfContextOwnerServices> service;
     mContext->QueryInterface(IID_ITfContextOwnerServices,
                              getter_AddRefs(service));
     if (service) {
       MOZ_LOG(sTextStoreLog, LogLevel::Info,
-             ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
-              "calling ITfContextOwnerServices::OnLayoutChange()...",
-              this));
+        ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
+         "calling ITfContextOwnerServices::OnLayoutChange()...",
+         this));
       HRESULT hr = service->OnLayoutChange();
       ret = ret && SUCCEEDED(hr);
       MOZ_LOG(sTextStoreLog, LogLevel::Info,
-             ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
-              "called ITfContextOwnerServices::OnLayoutChange()",
-              this));
+        ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
+         "called ITfContextOwnerServices::OnLayoutChange()",
+         this));
     }
   }
 
   if (!mWidget || mWidget->Destroyed()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Info,
-           ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
-            "the widget is destroyed during calling OnLayoutChange()",
-            this));
+      ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
+       "the widget is destroyed during calling OnLayoutChange()",
+       this));
     return ret;
   }
 
   if (mDestroyed) {
     MOZ_LOG(sTextStoreLog, LogLevel::Info,
-           ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
-            "the TSFTextStore instance is destroyed during calling "
-            "OnLayoutChange()",
-            this));
+      ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
+       "the TSFTextStore instance is destroyed during calling "
+       "OnLayoutChange()",
+       this));
     return ret;
   }
 
   // If we returned TS_E_NOLAYOUT again, we need another call of
   // OnLayoutChange() later.  So, let's wait a query from TIP.
   if (mHasReturnedNoLayoutError) {
     mWaitingQueryLayout = true;
   }
 
   if (!mWaitingQueryLayout) {
     MOZ_LOG(sTextStoreLog, LogLevel::Info,
-           ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
-            "succeeded notifying TIP of our layout change",
-            this));
+      ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
+       "succeeded notifying TIP of our layout change",
+       this));
     return ret;
   }
 
   // If we believe that TIP needs to retry to retrieve our layout information
   // later, we should call it with ::PostMessage() hack.
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-          ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
-           "posing  MOZ_WM_NOTIY_TSF_OF_LAYOUT_CHANGE for calling "
-           "OnLayoutChange() again...", this));
+    ("0x%p   TSFTextStore::NotifyTSFOfLayoutChange(), "
+     "posing  MOZ_WM_NOTIY_TSF_OF_LAYOUT_CHANGE for calling "
+     "OnLayoutChange() again...", this));
   ::PostMessage(mWidget->GetWindowHandle(),
                 MOZ_WM_NOTIY_TSF_OF_LAYOUT_CHANGE,
                 reinterpret_cast<WPARAM>(this), 0);
 
   return true;
 }
 
 void
@@ -4973,35 +4974,35 @@ TSFTextStore::NotifyTSFOfLayoutChangeAga
 
   // Before preforming this method, TIP has accessed our layout information by
   // itself.  In such case, we don't need to call OnLayoutChange() anymore.
   if (!mWaitingQueryLayout) {
     return;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
-         ("0x%p   TSFTextStore::NotifyTSFOfLayoutChangeAgain(), "
-          "calling NotifyTSFOfLayoutChange()...", this));
+    ("0x%p   TSFTextStore::NotifyTSFOfLayoutChangeAgain(), "
+     "calling NotifyTSFOfLayoutChange()...", this));
   NotifyTSFOfLayoutChange();
 
   // If TIP didn't retrieved our layout information during a call of
   // NotifyTSFOfLayoutChange(), it means that the TIP already gave up to
   // retry to retrieve layout information or doesn't necessary it anymore.
   // But don't forget that the call may have caused returning TS_E_NOLAYOUT
   // error again.  In such case we still need to call OnLayoutChange() later.
   if (!mHasReturnedNoLayoutError && mWaitingQueryLayout) {
     mWaitingQueryLayout = false;
     MOZ_LOG(sTextStoreLog, LogLevel::Warning,
-            ("0x%p   TSFTextStore::NotifyTSFOfLayoutChangeAgain(), "
-             "called NotifyTSFOfLayoutChange() but TIP didn't retry to "
-             "retrieve the layout information", this));
+      ("0x%p   TSFTextStore::NotifyTSFOfLayoutChangeAgain(), "
+       "called NotifyTSFOfLayoutChange() but TIP didn't retry to "
+       "retrieve the layout information", this));
   } else {
     MOZ_LOG(sTextStoreLog, LogLevel::Info,
-            ("0x%p   TSFTextStore::NotifyTSFOfLayoutChangeAgain(), "
-             "called NotifyTSFOfLayoutChange()", this));
+      ("0x%p   TSFTextStore::NotifyTSFOfLayoutChangeAgain(), "
+       "called NotifyTSFOfLayoutChange()", this));
   }
 }
 
 nsresult
 TSFTextStore::OnUpdateCompositionInternal()
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
     ("0x%p   TSFTextStore::OnUpdateCompositionInternal(), "
@@ -5034,33 +5035,33 @@ TSFTextStore::OnMouseButtonEventInternal
     return NS_OK;
   }
 
   if (mMouseTrackers.IsEmpty()) {
     return NS_OK;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::OnMouseButtonEventInternal("
-          "aIMENotification={ mEventMessage=%s, mOffset=%u, mCursorPos={ "
-          "mX=%d, mY=%d }, mCharRect={ mX=%d, mY=%d, mWidth=%d, mHeight=%d }, "
-          "mButton=%s, mButtons=%s, mModifiers=%s })",
-          this, ToChar(aIMENotification.mMouseButtonEventData.mEventMessage),
-          aIMENotification.mMouseButtonEventData.mOffset,
-          aIMENotification.mMouseButtonEventData.mCursorPos.mX,
-          aIMENotification.mMouseButtonEventData.mCursorPos.mY,
-          aIMENotification.mMouseButtonEventData.mCharRect.mX,
-          aIMENotification.mMouseButtonEventData.mCharRect.mY,
-          aIMENotification.mMouseButtonEventData.mCharRect.mWidth,
-          aIMENotification.mMouseButtonEventData.mCharRect.mHeight,
-          GetMouseButtonName(aIMENotification.mMouseButtonEventData.mButton),
-          GetMouseButtonsName(
-            aIMENotification.mMouseButtonEventData.mButtons).get(),
-          GetModifiersName(
-            aIMENotification.mMouseButtonEventData.mModifiers).get()));
+    ("0x%p   TSFTextStore::OnMouseButtonEventInternal("
+     "aIMENotification={ mEventMessage=%s, mOffset=%u, mCursorPos={ "
+     "mX=%d, mY=%d }, mCharRect={ mX=%d, mY=%d, mWidth=%d, mHeight=%d }, "
+     "mButton=%s, mButtons=%s, mModifiers=%s })",
+     this, ToChar(aIMENotification.mMouseButtonEventData.mEventMessage),
+     aIMENotification.mMouseButtonEventData.mOffset,
+     aIMENotification.mMouseButtonEventData.mCursorPos.mX,
+     aIMENotification.mMouseButtonEventData.mCursorPos.mY,
+     aIMENotification.mMouseButtonEventData.mCharRect.mX,
+     aIMENotification.mMouseButtonEventData.mCharRect.mY,
+     aIMENotification.mMouseButtonEventData.mCharRect.mWidth,
+     aIMENotification.mMouseButtonEventData.mCharRect.mHeight,
+     GetMouseButtonName(aIMENotification.mMouseButtonEventData.mButton),
+     GetMouseButtonsName(
+       aIMENotification.mMouseButtonEventData.mButtons).get(),
+     GetModifiersName(
+       aIMENotification.mMouseButtonEventData.mModifiers).get()));
 
   uint32_t offset = aIMENotification.mMouseButtonEventData.mOffset;
   nsIntRect charRect =
     aIMENotification.mMouseButtonEventData.mCharRect.AsIntRect();
   nsIntPoint cursorPos =
     aIMENotification.mMouseButtonEventData.mCursorPos.AsIntPoint();
   ULONG quadrant = 1;
   if (charRect.width > 0) {
@@ -5110,25 +5111,25 @@ TSFTextStore::CreateNativeCaret()
   MaybeDestroyNativeCaret();
 
   // Don't create native caret after destroyed.
   if (mDestroyed) {
     return;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::CreateNativeCaret(), "
-          "mComposition.IsComposing()=%s",
-          this, GetBoolName(mComposition.IsComposing())));
+    ("0x%p   TSFTextStore::CreateNativeCaret(), "
+     "mComposition.IsComposing()=%s",
+     this, GetBoolName(mComposition.IsComposing())));
 
   Selection& selectionForTSF = SelectionForTSFRef();
   if (selectionForTSF.IsDirty()) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::CreateNativeCaret() FAILED due to "
-            "SelectionForTSFRef() failure", this));
+      ("0x%p   TSFTextStore::CreateNativeCaret() FAILED due to "
+       "SelectionForTSFRef() failure", this));
     return;
   }
 
   WidgetQueryContentEvent queryCaretRect(true, eQueryCaretRect, mWidget);
   mWidget->InitEvent(queryCaretRect);
 
   WidgetQueryContentEvent::Options options;
   // XXX If this is called without composition and the selection isn't
@@ -5141,37 +5142,37 @@ TSFTextStore::CreateNativeCaret()
     options.mRelativeToInsertionPoint = true;
     caretOffset -= mComposition.mStart;
   }
   queryCaretRect.InitForQueryCaretRect(caretOffset, options);
 
   DispatchEvent(queryCaretRect);
   if (NS_WARN_IF(!queryCaretRect.mSucceeded)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::CreateNativeCaret() FAILED due to "
-            "eQueryCaretRect failure (offset=%d)", this, caretOffset));
+      ("0x%p   TSFTextStore::CreateNativeCaret() FAILED due to "
+       "eQueryCaretRect failure (offset=%d)", this, caretOffset));
     return;
   }
 
   LayoutDeviceIntRect& caretRect = queryCaretRect.mReply.mRect;
   mNativeCaretIsCreated = ::CreateCaret(mWidget->GetWindowHandle(), nullptr,
                                         caretRect.width, caretRect.height);
   if (!mNativeCaretIsCreated) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::CreateNativeCaret() FAILED due to "
-            "CreateCaret() failure", this));
+      ("0x%p   TSFTextStore::CreateNativeCaret() FAILED due to "
+       "CreateCaret() failure", this));
     return;
   }
 
   nsWindow* window = static_cast<nsWindow*>(mWidget.get());
   nsWindow* toplevelWindow = window->GetTopLevelWindow(false);
   if (!toplevelWindow) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::CreateNativeCaret() FAILED due to "
-            "no top level window", this));
+      ("0x%p   TSFTextStore::CreateNativeCaret() FAILED due to "
+       "no top level window", this));
     return;
   }
 
   if (toplevelWindow != window) {
     caretRect.MoveBy(toplevelWindow->WidgetToScreenOffset());
     caretRect.MoveBy(-window->WidgetToScreenOffset());
   }
 
@@ -5181,33 +5182,33 @@ TSFTextStore::CreateNativeCaret()
 void
 TSFTextStore::MaybeDestroyNativeCaret()
 {
   if (!mNativeCaretIsCreated) {
     return;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::MaybeDestroyNativeCaret(), "
-          "destroying native caret", this));
+    ("0x%p   TSFTextStore::MaybeDestroyNativeCaret(), "
+     "destroying native caret", this));
 
   ::DestroyCaret();
   mNativeCaretIsCreated = false;
 }
 
 void
 TSFTextStore::CommitCompositionInternal(bool aDiscard)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::CommitCompositionInternal(aDiscard=%s), "
-          "mSink=0x%p, mContext=0x%p, mComposition.mView=0x%p, "
-          "mComposition.mString=\"%s\"",
-          this, GetBoolName(aDiscard), mSink.get(), mContext.get(),
-          mComposition.mView.get(),
-          NS_ConvertUTF16toUTF8(mComposition.mString).get()));
+    ("0x%p   TSFTextStore::CommitCompositionInternal(aDiscard=%s), "
+     "mSink=0x%p, mContext=0x%p, mComposition.mView=0x%p, "
+     "mComposition.mString=\"%s\"",
+     this, GetBoolName(aDiscard), mSink.get(), mContext.get(),
+     mComposition.mView.get(),
+     NS_ConvertUTF16toUTF8(mComposition.mString).get()));
 
   // If the document is locked, TSF will fail to commit composition since
   // TSF needs another document lock.  So, let's put off the request.
   // Note that TextComposition will commit composition in the focused editor
   // with the latest composition string for web apps and waits asynchronous
   // committing messages.  Therefore, we can and need to perform this
   // asynchronously.
   if (IsReadLocked()) {
@@ -5234,36 +5235,36 @@ TSFTextStore::CommitCompositionInternal(
     mComposition.mString.Truncate(0);
     // Note that don't notify TSF of text change after this is destroyed.
     if (mSink && !mDestroyed) {
       TS_TEXTCHANGE textChange;
       textChange.acpStart = mComposition.mStart;
       textChange.acpOldEnd = endOffset;
       textChange.acpNewEnd = mComposition.mStart;
       MOZ_LOG(sTextStoreLog, LogLevel::Info,
-             ("0x%p   TSFTextStore::CommitCompositionInternal(), calling"
-              "mSink->OnTextChange(0, { acpStart=%ld, acpOldEnd=%ld, "
-              "acpNewEnd=%ld })...", this, textChange.acpStart,
-              textChange.acpOldEnd, textChange.acpNewEnd));
+        ("0x%p   TSFTextStore::CommitCompositionInternal(), calling"
+         "mSink->OnTextChange(0, { acpStart=%ld, acpOldEnd=%ld, "
+         "acpNewEnd=%ld })...", this, textChange.acpStart,
+         textChange.acpOldEnd, textChange.acpNewEnd));
       mSink->OnTextChange(0, &textChange);
     }
   }
   // Terminate two contexts, the base context (mContext) and the top
   // if the top context is not the same as the base context
   RefPtr<ITfContext> context = mContext;
   do {
     if (context) {
       RefPtr<ITfContextOwnerCompositionServices> services;
       context->QueryInterface(IID_ITfContextOwnerCompositionServices,
                               getter_AddRefs(services));
       if (services) {
         MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-               ("0x%p   TSFTextStore::CommitCompositionInternal(), "
-                "requesting TerminateComposition() for the context 0x%p...",
-                this, context.get()));
+          ("0x%p   TSFTextStore::CommitCompositionInternal(), "
+           "requesting TerminateComposition() for the context 0x%p...",
+           this, context.get()));
         services->TerminateComposition(nullptr);
       }
     }
     if (context != mContext)
       break;
     if (mDocumentMgr)
       mDocumentMgr->GetTop(getter_AddRefs(context));
   } while (context != mContext);
@@ -5285,36 +5286,36 @@ GetCompartment(IUnknown* pUnk,
          (*aCompartment) != nullptr;
 }
 
 // static
 void
 TSFTextStore::SetIMEOpenState(bool aState)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("TSFTextStore::SetIMEOpenState(aState=%s)",
-          GetBoolName(aState)));
+    ("TSFTextStore::SetIMEOpenState(aState=%s)",
+     GetBoolName(aState)));
 
   RefPtr<ITfCompartment> comp;
   if (!GetCompartment(sThreadMgr,
                       GUID_COMPARTMENT_KEYBOARD_OPENCLOSE,
                       getter_AddRefs(comp))) {
     MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-           ("  TSFTextStore::SetIMEOpenState() FAILED due to"
-            "no compartment available"));
+      ("  TSFTextStore::SetIMEOpenState() FAILED due to"
+       "no compartment available"));
     return;
   }
 
   VARIANT variant;
   variant.vt = VT_I4;
   variant.lVal = aState;
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("  TSFTextStore::SetIMEOpenState(), setting "
-          "0x%04X to GUID_COMPARTMENT_KEYBOARD_OPENCLOSE...",
-          variant.lVal));
+    ("  TSFTextStore::SetIMEOpenState(), setting "
+     "0x%04X to GUID_COMPARTMENT_KEYBOARD_OPENCLOSE...",
+     variant.lVal));
   comp->SetValue(sClientId, &variant);
 }
 
 // static
 bool
 TSFTextStore::GetIMEOpenState()
 {
   RefPtr<ITfCompartment> comp;
@@ -5334,22 +5335,22 @@ TSFTextStore::GetIMEOpenState()
 
 // static
 void
 TSFTextStore::SetInputContext(nsWindowBase* aWidget,
                              const InputContext& aContext,
                              const InputContextAction& aAction)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("TSFTextStore::SetInputContext(aWidget=%p, "
-          "aContext.mIMEState.mEnabled=%s, aAction.mFocusChange=%s), "
-          "sEnabledTextStore=0x%p, ThinksHavingFocus()=%s",
-          aWidget, GetIMEEnabledName(aContext.mIMEState.mEnabled),
-          GetFocusChangeName(aAction.mFocusChange), sEnabledTextStore.get(),
-          GetBoolName(ThinksHavingFocus())));
+    ("TSFTextStore::SetInputContext(aWidget=%p, "
+     "aContext.mIMEState.mEnabled=%s, aAction.mFocusChange=%s), "
+     "sEnabledTextStore=0x%p, ThinksHavingFocus()=%s",
+     aWidget, GetIMEEnabledName(aContext.mIMEState.mEnabled),
+     GetFocusChangeName(aAction.mFocusChange), sEnabledTextStore.get(),
+     GetBoolName(ThinksHavingFocus())));
 
   NS_ENSURE_TRUE_VOID(IsInTSFMode());
 
   if (aAction.mFocusChange != InputContextAction::FOCUS_NOT_CHANGED) {
     if (sEnabledTextStore) {
       RefPtr<TSFTextStore> textStore(sEnabledTextStore);
       textStore->SetInputScope(aContext.mHTMLInputType,
                                aContext.mHTMLInputInputmode);
@@ -5374,49 +5375,49 @@ TSFTextStore::MarkContextAsKeyboardDisab
   variant_int4_value1.vt = VT_I4;
   variant_int4_value1.lVal = 1;
 
   RefPtr<ITfCompartment> comp;
   if (!GetCompartment(aContext,
                       GUID_COMPARTMENT_KEYBOARD_DISABLED,
                       getter_AddRefs(comp))) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("TSFTextStore::MarkContextAsKeyboardDisabled() failed"
-            "aContext=0x%p...", aContext));
+      ("TSFTextStore::MarkContextAsKeyboardDisabled() failed"
+       "aContext=0x%p...", aContext));
     return;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("TSFTextStore::MarkContextAsKeyboardDisabled(), setting "
-          "to disable context 0x%p...",
-          aContext));
+    ("TSFTextStore::MarkContextAsKeyboardDisabled(), setting "
+     "to disable context 0x%p...",
+     aContext));
   comp->SetValue(sClientId, &variant_int4_value1);
 }
 
 // static
 void
 TSFTextStore::MarkContextAsEmpty(ITfContext* aContext)
 {
   VARIANT variant_int4_value1;
   variant_int4_value1.vt = VT_I4;
   variant_int4_value1.lVal = 1;
 
   RefPtr<ITfCompartment> comp;
   if (!GetCompartment(aContext,
                       GUID_COMPARTMENT_EMPTYCONTEXT,
                       getter_AddRefs(comp))) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("TSFTextStore::MarkContextAsEmpty() failed"
-            "aContext=0x%p...", aContext));
+      ("TSFTextStore::MarkContextAsEmpty() failed"
+       "aContext=0x%p...", aContext));
     return;
   }
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("TSFTextStore::MarkContextAsEmpty(), setting "
-          "to mark empty context 0x%p...", aContext));
+    ("TSFTextStore::MarkContextAsEmpty(), setting "
+     "to mark empty context 0x%p...", aContext));
   comp->SetValue(sClientId, &variant_int4_value1);
 }
 
 // static
 void
 TSFTextStore::Initialize()
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Info,
@@ -5912,124 +5913,124 @@ TSFTextStore::MouseTracker::MouseTracker
   , mCookie(kInvalidCookie)
 {
 }
 
 HRESULT
 TSFTextStore::MouseTracker::Init(TSFTextStore* aTextStore)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::MouseTracker::Init(aTextStore=0x%p), "
-          "aTextStore->mMouseTrackers.Length()=%d",
-          this, aTextStore->mMouseTrackers.Length()));
+    ("0x%p   TSFTextStore::MouseTracker::Init(aTextStore=0x%p), "
+     "aTextStore->mMouseTrackers.Length()=%d",
+     this, aTextStore->mMouseTrackers.Length()));
 
   if (&aTextStore->mMouseTrackers.LastElement() != this) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::MouseTracker::Init() FAILED due to "
-            "this is not the last element of mMouseTrackers", this));
+      ("0x%p   TSFTextStore::MouseTracker::Init() FAILED due to "
+       "this is not the last element of mMouseTrackers", this));
     return E_FAIL;
   }
   if (aTextStore->mMouseTrackers.Length() > kInvalidCookie) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::MouseTracker::Init() FAILED due to "
-            "no new cookie available", this));
+      ("0x%p   TSFTextStore::MouseTracker::Init() FAILED due to "
+       "no new cookie available", this));
     return E_FAIL;
   }
   MOZ_ASSERT(!aTextStore->mMouseTrackers.IsEmpty(),
              "This instance must be in TSFTextStore::mMouseTrackers");
   mCookie = static_cast<DWORD>(aTextStore->mMouseTrackers.Length() - 1);
   return S_OK;
 }
 
 HRESULT
 TSFTextStore::MouseTracker::AdviseSink(TSFTextStore* aTextStore,
                                        ITfRangeACP* aTextRange,
                                        ITfMouseSink* aMouseSink)
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::MouseTracker::AdviseSink(aTextStore=0x%p, "
-          "aTextRange=0x%p, aMouseSink=0x%p), mCookie=%d, mSink=0x%p",
-          this, aTextStore, aTextRange, aMouseSink, mCookie, mSink.get()));
+    ("0x%p   TSFTextStore::MouseTracker::AdviseSink(aTextStore=0x%p, "
+     "aTextRange=0x%p, aMouseSink=0x%p), mCookie=%d, mSink=0x%p",
+     this, aTextStore, aTextRange, aMouseSink, mCookie, mSink.get()));
   MOZ_ASSERT(mCookie != kInvalidCookie, "This hasn't been initalized?");
 
   if (mSink) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::MouseTracker::AdviseMouseSink() FAILED "
-            "due to already being used", this));
+      ("0x%p   TSFTextStore::MouseTracker::AdviseMouseSink() FAILED "
+       "due to already being used", this));
     return E_FAIL;
   }
 
   HRESULT hr = aTextRange->GetExtent(&mStart, &mLength);
   if (FAILED(hr)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::MouseTracker::AdviseMouseSink() FAILED "
-            "due to failure of ITfRangeACP::GetExtent()", this));
+      ("0x%p   TSFTextStore::MouseTracker::AdviseMouseSink() FAILED "
+       "due to failure of ITfRangeACP::GetExtent()", this));
     return hr;
   }
 
   if (mStart < 0 || mLength <= 0) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::MouseTracker::AdviseMouseSink() FAILED "
-            "due to odd result of ITfRangeACP::GetExtent(), "
-            "mStart=%d, mLength=%d", this, mStart, mLength));
+      ("0x%p   TSFTextStore::MouseTracker::AdviseMouseSink() FAILED "
+       "due to odd result of ITfRangeACP::GetExtent(), "
+       "mStart=%d, mLength=%d", this, mStart, mLength));
     return E_INVALIDARG;
   }
 
   nsAutoString textContent;
   if (NS_WARN_IF(!aTextStore->GetCurrentText(textContent))) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::MouseTracker::AdviseMouseSink() FAILED "
-            "due to failure of TSFTextStore::GetCurrentText()", this));
+      ("0x%p   TSFTextStore::MouseTracker::AdviseMouseSink() FAILED "
+       "due to failure of TSFTextStore::GetCurrentText()", this));
     return E_FAIL;
   }
 
   if (textContent.Length() <= static_cast<uint32_t>(mStart) ||
       textContent.Length() < static_cast<uint32_t>(mStart + mLength)) {
     MOZ_LOG(sTextStoreLog, LogLevel::Error,
-           ("0x%p   TSFTextStore::MouseTracker::AdviseMouseSink() FAILED "
-            "due to out of range, mStart=%d, mLength=%d, "
-            "textContent.Length()=%d",
-            this, mStart, mLength, textContent.Length()));
+      ("0x%p   TSFTextStore::MouseTracker::AdviseMouseSink() FAILED "
+       "due to out of range, mStart=%d, mLength=%d, "
+       "textContent.Length()=%d",
+       this, mStart, mLength, textContent.Length()));
     return E_INVALIDARG;
   }
 
   mSink = aMouseSink;
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::MouseTracker::AdviseMouseSink(), "
-          "succeeded, mStart=%d, mLength=%d, textContent.Length()=%d",
-          this, mStart, mLength, textContent.Length()));
+    ("0x%p   TSFTextStore::MouseTracker::AdviseMouseSink(), "
+     "succeeded, mStart=%d, mLength=%d, textContent.Length()=%d",
+     this, mStart, mLength, textContent.Length()));
   return S_OK;
 }
 
 void
 TSFTextStore::MouseTracker::UnadviseSink()
 {
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::MouseTracker::UnadviseSink(), "
-          "mCookie=%d, mSink=0x%p, mStart=%d, mLength=%d",
-          this, mCookie, mSink.get(), mStart, mLength));
+    ("0x%p   TSFTextStore::MouseTracker::UnadviseSink(), "
+     "mCookie=%d, mSink=0x%p, mStart=%d, mLength=%d",
+     this, mCookie, mSink.get(), mStart, mLength));
   mSink = nullptr;
   mStart = mLength = -1;
 }
 
 bool
 TSFTextStore::MouseTracker::OnMouseButtonEvent(ULONG aEdge,
                                                ULONG aQuadrant,
                                                DWORD aButtonStatus)
 {
   MOZ_ASSERT(IsUsing(), "The caller must check before calling OnMouseEvent()");
 
   BOOL eaten = FALSE;
   HRESULT hr = mSink->OnMouseEvent(aEdge, aQuadrant, aButtonStatus, &eaten);
 
   MOZ_LOG(sTextStoreLog, LogLevel::Debug,
-         ("0x%p   TSFTextStore::MouseTracker::OnMouseEvent(aEdge=%d, "
-          "aQuadrant=%d, aButtonStatus=0x%08X), hr=0x%08X, eaten=%s",
-          this, aEdge, aQuadrant, aButtonStatus, hr, GetBoolName(!!eaten)));
+    ("0x%p   TSFTextStore::MouseTracker::OnMouseEvent(aEdge=%d, "
+     "aQuadrant=%d, aButtonStatus=0x%08X), hr=0x%08X, eaten=%s",
+     this, aEdge, aQuadrant, aButtonStatus, hr, GetBoolName(!!eaten)));
 
   return SUCCEEDED(hr) && eaten;
 }
 
 #ifdef DEBUG
 // static
 bool
 TSFTextStore::CurrentKeyboardLayoutHasIME()