Bug 1792387 - part 10: Make editor/libeditor/tests/test_inline_style_cache.html be aware of both split node directions r=m_kato

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

Author: masayuki-nakano

editor/libeditor/tests/test_inline_style_cache.html

@@ -18,6 +18,10 @@
 <script class="testbody" type="application/javascript">
 SimpleTest.waitForExplicitFinish();
 SimpleTest.waitForFocus(function() {
+  const isTraditionalSplitDirection =!SpecialPowers.getBoolPref(
+    "editor.join_split_direction.compatible_with_the_other_browsers"
+  );
+
   var editor = document.getElementById("editor");
   editor.focus();
 
@@ -37,7 +41,6 @@
      "#01-01 At typing something after setting some styles, should cause inserting some nodes to apply the style");
 
   // #01-02 Typing something after removing some characters after setting some styles should work as without removing some character.
-  // XXX This behavior is different from Chromium.
   editor.innerHTML = "beforeafter";
   selection.collapse(editor.firstChild, "before".length);
   document.execCommand("bold");
@@ -46,8 +49,16 @@
   synthesizeKey("KEY_Delete");
   sendString("test");
 
-  is(editor.innerHTML, "before<strike><i><b>test</b></i></strike>fter",
-     "#01-02-1 At typing something after Delete after setting style, should cause inserting some nodes to apply the style");
+  todo_is(
+    editor.innerHTML,
+    "beforetestfter",
+    "#01-02-1 At typing something after Delete after setting style, the style should not be preserved for compatibility with the other browsers"
+  );
+  is(
+    editor.innerHTML,
+    "before<strike><i><b>test</b></i></strike>fter",
+    "#01-02-1 At typing something after Delete after setting style, the style should not be preserved, but it's okay to do it for backward compatibility"
+  );
 
   editor.innerHTML = "beforeafter";
   selection.collapse(editor.firstChild, "before".length);
@@ -84,7 +95,6 @@
      "#01-03-2 Typing Enter after setting style should not ignore the styles");
 
   // #02-01 Replacing text with typing some text after setting some styles should work as just inserting text.
-  // XXX Chromium works as expected.
   editor.innerHTML = "beforeselectionafter";
   selection.collapse(editor.firstChild, "before".length);
   selection.extend(editor.firstChild, "beforeselection".length);
@@ -93,11 +103,18 @@
   document.execCommand("strikethrough");
   sendString("test");
 
-  is(editor.innerHTML, "before<strike><i><b>test</b></i></strike>after",
-     "#02-01 At replacing \"selection\" after setting some styles, should keep the styles at inserting text");
+  todo_is(
+    editor.innerHTML,
+    "before<b><i><strike>test</strike></i></b>after",
+    "#02-01 At replacing \"selection\" after setting some styles, should keep the styles at inserting text for compatibility with the other browsers"
+  );
+  is(
+    editor.innerHTML,
+    "before<strike><i><b>test</b></i></strike>after",
+    "#02-01 At replacing \"selection\" after setting some styles, should keep the styles at inserting text, but it's okay to do it with reverted order for the backward compatibility"
+  );
 
   // #02-02 Inserting text after removing selected text after setting some styles should not keep the styles.
-  // XXX Chromium keeps the style.
   editor.innerHTML = "beforeselectionafter";
   selection.collapse(editor.firstChild, "before".length);
   selection.extend(editor.firstChild, "beforeselection".length);
@@ -107,8 +124,24 @@
   synthesizeKey("KEY_Backspace");
   sendString("test");
 
-  is(editor.innerHTML, "beforetestafter",
-     "#02-02 After removing \"selection\" after setting some styles, should not keep the styles");
+  todo_is(
+    editor.innerHTML,
+    "before<b><i><strike>test</strike></i></b>after",
+    "#02-02 After removing \"selection\" after setting some styles, should preserve the style for compatibility with the other browsers"
+  );
+  if (isTraditionalSplitDirection) {
+    is(
+      editor.innerHTML,
+      "beforetestafter",
+      "#02-02 After removing \"selection\" after setting some styles, should preserve the style, but it's okay not to do it for backward compatibility"
+    );
+  } else {
+    is(
+      editor.innerHTML,
+      "before<strike><i><b>test</b></i></strike>after",
+      "#02-02 After removing \"selection\" after setting some styles, should preserve the style, but it's okay to do it with reverted order for the backward compatibility"
+    );
+  }
 
   // #02-03 Inserting text after replacing selected text after setting some styles should keep the styles.
   editor.innerHTML = "beforeselectionafter";
@@ -120,8 +153,24 @@
   synthesizeKey("KEY_Enter");
   sendString("test");
 
-  is(editor.innerHTML, "<div>before</div><div><strike><i><b>test</b></i></strike>after</div>",
-     "#02-03-1 Typing Enter after setting style to selected text should keep the styles");
+  todo_is(
+    editor.innerHTML,
+    "before<div><b><i><strike>test</strike></i></b>after</div>",
+    "#02-03-1 Typing Enter after setting style to selected text should keep the styles and insert text into the new paragraph"
+  );
+  if (isTraditionalSplitDirection) {
+    is(
+      editor.innerHTML,
+      "<div>before</div><div><strike><i><b>test</b></i></strike>after</div>",
+      "#02-03-1 Typing Enter after setting style to selected text should keep the styles, it's okay to create left paragraph for backward compatibility"
+    );
+  } else {
+    is(
+      editor.innerHTML,
+      "<div>before</div><div><strike><i><b>test</b></i></strike>after</div>",
+      "#02-03-1 Typing Enter after setting style to selected text should keep the styles, it's okay to create left paragraph for backward compatibility"
+    );
+  }
 
   editor.innerHTML = "<p>beforeselectionafter</p>";
   selection.collapse(editor.firstChild.firstChild, "before".length);
@@ -132,8 +181,11 @@
   synthesizeKey("KEY_Enter");
   sendString("test");
 
-  is(editor.innerHTML, "<p>before</p><p><strike><i><b>test</b></i></strike>after</p>",
-     "#02-03-2 Typing Enter after setting style to selected text should keep the styles");
+  is(
+    editor.innerHTML,
+    "<p>before</p><p><strike><i><b>test</b></i></strike>after</p>",
+    "#02-03-2 Typing Enter after setting style to selected text should keep the styles"
+  );
 
   // #03-01 Replacing in <b style="font-weight: normal;"> shouldn't cause new <b>.
   editor.innerHTML = "<b style=\"font-weight: normal;\">beforeselectionafter</b>";