author Masayuki Nakano <>
Thu, 02 Feb 2023 05:09:15 +0000
changeset 651452 20512741e7b60f3f6791002d8311d681f9242e68
parent 247556 f1c34cec973b1cc8c6e48e10028d7c2e2d31f6c8
permissions -rw-r--r--
Bug 1812170 - Make `HTMLEditor::ClearStyleAt` should return `pointToPutCaret` if next node of first split is not split r=m_kato The new path added in bug 1807829 hits the odd result case of `ClearStyleAt`. `pointToPutCaret` may be updated if the preceding split occurs. Therefore, it should be returned if it does not split next nodes after the first split instead of returning unset point (because of not splitting the point, `unwrappedSplitNodeResult.AtSplitPoint` may return unset point, therefore, it may return unset point in the case). Additionally, I forgot to split delete `SplitNodeResult::mCaretPoint` in bug 1792654, and it causes `SplitNodeDeepWithTransaction` returns unset caret point. This patch fixes this bug too. Differential Revision:

<meta charset="utf-8">
<style type="text/css">
.grid {
  border: 1px solid gray;
  font-size: 0;
.grid > div {
  display: inline-block;
  margin: 2px;
#a { background: #006; height: 46px; width: 96px; }
#b { background: #009; height: 46px; width: 146px; }
#c { background: #00c; height: 46px; width: 46px; }
#d { background: #00f; height: 46px; width: 16px; }
#e { background: #060; height: 96px; width: 96px; }
#f { background: #090; height: 96px; width: 146px; }
#g { background: #0c0; height: 96px; width: 46px; }
#h { background: #0f0; height: 96px; width: 16px; }
#i { background: #600; height: 16px; width: 96px; }
#j { background: #900; height: 16px; width: 146px; }
#k { background: #c00; height: 16px; width: 46px; }
#l { background: #f00; height: 16px; width: 16px; }

<div class=grid>
  <div id=a></div>
  <div id=b></div>
  <div id=c></div>
  <div id=d></div><br>
  <div id=e></div>
  <div id=f></div>
  <div id=g></div>
  <div id=h></div><br>
  <div id=i></div>
  <div id=j></div>
  <div id=k></div>
  <div id=l></div>