Backed out changeset c132cd7a5920 (bug 1827541) for causing Bug 1832746. CLOSED TREE

taskcluster/ci/config.yml

@@ -84,7 +84,6 @@ treeherder:
         'M-f': 'Mochitest failures'
         'M-dt-no-eft-nofis': 'DevTools Mochitests with EFT without fission enabled'
         'M-msix': 'Mochitests from MSIX builds'
-        'M-editor-legacy-dir': 'Mochitests within the legacy join/split node direction mode of the editor'
         'MSI': 'Repack installers into MSIs'
         'MSIs': 'Signing of Repacked installers of MSIs'
         'MSIX': 'Repack into MSIX package'
@@ -100,7 +99,6 @@ treeherder:
         'R-wr-dc1-p': 'Reftests with dcomp present but not overlays'
         'R-wr-dc2-o': 'Reftests with dcomp overlays but not compositing'
         'R-wr-dc3-c': 'Reftests with dcomp compositing'
-        'R-editor-legacy-dir': 'Reftests within the legacy join/split node direction mode of the editor'
         'Rap': 'Raptor performance tests on Firefox'
         'Rap-live': 'Raptor performance tests on Firefox with live sites'
         'Rap-fis': 'Raptor performance tests on Firefox with fission enabled'
@@ -133,7 +131,6 @@ treeherder:
         'W-swr': 'Web platform tests with software webrender enabled'
         'W-b': 'Web platform tests (backlog)'
         'W-b-nofis': 'Web platform tests (backlog) without fission enabled'
-        'W-editor-legacy-dir': 'Web platform tests within the legacy join/split node direction mode of the editor'
         'X': 'Xpcshell tests'
         'X-condprof': 'Xpcshell tests with a conditioned profile'
         'X-nofis': 'Xpcshell tests without fission enabled'

taskcluster/ci/test/mochitest.yml

@@ -63,7 +63,6 @@ mochitest-plain:
         - headless
         - headless+socketprocess_networking
         - http3
-        - editor-legacy-direction
     run-on-projects:
         by-variant:
             conditioned_profile:
@@ -120,10 +119,6 @@ mochitest-plain:
                 by-test-platform:
                     linux1804-64-qr/debug: built-projects
                     default: []
-            editor-legacy-direction:
-                by-test-platform:
-                    linux1804-64-qr/debug: ['mozilla-central']
-                    default: []
             default:
                 by-test-platform:
                     android-em-7.0-x86_64-qr/debug-isolated-process: []
@@ -194,7 +189,6 @@ mochitest-a11y:
         - socketprocess_networking+1proc
         - webrender-sw+1proc
         - a11y-no-cache+1proc
-        - editor-legacy-direction
     run-on-projects:
         by-variant:
             socketprocess_networking+1proc:
@@ -208,10 +202,6 @@ mochitest-a11y:
                     default: []
             a11y-no-cache+1proc: ['trunk']
             1proc: built-projects
-            editor-legacy-direction:
-                by-test-platform:
-                    linux1804-64-qr/debug: ['mozilla-central']
-                    default: []
     mozharness:
         mochitest-flavor: a11y
 
@@ -227,7 +217,6 @@ mochitest-browser-chrome:
         - webrender-sw+a11y-checks
         - socketprocess_networking
         - msix
-        - editor-legacy-direction
     run-on-projects:
         by-variant:
             webrender-sw:
@@ -254,10 +243,6 @@ mochitest-browser-chrome:
                     windows11-64-2009-qr/debug: built-projects
                     windows11-64-2009(-shippable)?-qr/opt: built-projects
                     default: []
-            editor-legacy-direction:
-                by-test-platform:
-                    linux1804-64-qr/debug: ['mozilla-central']
-                    default: []
             default:
                 by-test-platform:
                     android.*: []
@@ -405,7 +390,6 @@ mochitest-chrome:
         - 1proc
         - socketprocess_networking+1proc
         - webrender-sw+1proc
-        - editor-legacy-direction
     run-on-projects:
         by-variant:
             socketprocess_networking+1proc:
@@ -419,10 +403,6 @@ mochitest-chrome:
                     android-em-7.0-x86_64-qr/debug: built-projects
                     linux.*64-qr/debug: built-projects
                     default: []
-            editor-legacy-direction:
-                by-test-platform:
-                    linux1804-64-qr/debug: ['mozilla-central']
-                    default: []
             1proc: built-projects
     chunks:
         by-test-platform:

taskcluster/ci/test/reftest.yml

@@ -47,7 +47,6 @@ crashtest:
         - no-fission
         - webrender-sw+no-fission
         - webrender-sw
-        - editor-legacy-direction
     run-on-projects:
         by-variant:
             webrender-sw+no-fission:
@@ -71,10 +70,6 @@ crashtest:
                     linux1804-64(-shippable|-asan)?-qr/opt: ['trunk']
                     android.*: built-projects
                     default: []
-            editor-legacy-direction:
-                by-test-platform:
-                    linux1804-64-qr/debug: ['mozilla-central']
-                    default: []
             default:
                 by-test-platform:
                     android.*: []
@@ -170,7 +165,6 @@ reftest:
         - webrender-dcomp-compositing  # Should match default
         - webrender-sw+no-fission
         - webrender-sw
-        - editor-legacy-direction
     run-on-projects:
         by-variant:
             webrender-sw+no-fission:
@@ -194,10 +188,6 @@ reftest:
                     linux1804-64(-shippable|-asan)?-qr/opt: ['trunk']
                     android.*: built-projects
                     default: []
-            editor-legacy-direction:
-                by-test-platform:
-                    linux1804-64-qr/debug: ['mozilla-central']
-                    default: []
             default:
                 by-test-platform:
                     android.*/.*: []

taskcluster/ci/test/variants.yml

@@ -104,18 +104,6 @@ xorigin:
                 - "--enable-xorigin-tests"
                 - "--setpref=privacy.partition.always_partition_third_party_non_cookie_storage=false"
 
-editor-legacy-direction:
-    description: "{description} with compatible join/split node direction of editor disabled"
-    component: "Core::DOM: Editor"
-    expiration: "2023-09-01"
-    suffix: "editor-legacy-dir"
-    replace:
-        tier: 2
-    merge:
-        mozharness:
-            extra-options:
-                - "--setpref=editor.join_split_direction.compatible_with_the_other_browsers=false"
-
 conditioned_profile:
     description: "{description} with conditioned profile"
     component: "Testing::General"

taskcluster/ci/test/web-platform.yml

@@ -65,7 +65,6 @@ web-platform-tests:
         - no-fission
         - webrender-sw+no-fission
         - webrender-sw
-        - editor-legacy-direction
     run-on-projects:
         by-variant:
             webrender-sw:
@@ -88,10 +87,6 @@ web-platform-tests:
                     linux1804-64-qr/debug: ['trunk']
                     linux1804-64(-shippable|-asan)?-qr/opt: ['trunk']
                     default: []
-            editor-legacy-direction:
-                by-test-platform:
-                    linux1804-64-qr/debug: ['mozilla-central']
-                    default: []
             default:
                 by-test-platform:
                     android.*: []
@@ -164,7 +159,6 @@ web-platform-tests-reftest:
         - no-fission
         - webrender-sw+no-fission
         - webrender-sw
-        - editor-legacy-direction
     run-on-projects:
         by-variant:
             webrender-sw:
@@ -186,10 +180,6 @@ web-platform-tests-reftest:
                     linux1804-64(-shippable|-asan)?-qr/opt: ['trunk']
                     android.*: built-projects
                     default: []
-            editor-legacy-direction:
-                by-test-platform:
-                    linux1804-64-qr/debug: ['mozilla-central']
-                    default: []
             default:
                 by-test-platform:
                     android.*: []
@@ -274,7 +264,6 @@ web-platform-tests-wdspec:
         - webrender-sw+no-fission
         - webrender-sw
         - headless
-        - editor-legacy-direction
     run-on-projects:
         by-variant:
             webrender-sw+no-fission:
@@ -300,10 +289,6 @@ web-platform-tests-wdspec:
                     linux1804-64(-shippable|-asan)?-qr/opt: ['trunk']
                     android.*: built-projects
                     default: []
-            editor-legacy-direction:
-                by-test-platform:
-                    linux1804-64-qr/debug: ['mozilla-central']
-                    default: []
             default:
                 by-test-platform:
                     android.*: []
@@ -356,7 +341,6 @@ web-platform-tests-crashtest:
         - no-fission
         - webrender-sw
         - webrender-sw+no-fission
-        - editor-legacy-direction
     run-on-projects:
         by-variant:
             webrender-sw:
@@ -380,10 +364,6 @@ web-platform-tests-crashtest:
                     linux1804-64(-shippable|-asan)?-qr/opt: ['trunk']
                     android.*: built-projects
                     default: []
-            editor-legacy-direction:
-                by-test-platform:
-                    linux1804-64-qr/debug: ['mozilla-central']
-                    default: []
             default:
                 by-test-platform:
                     android.*: []

testing/web-platform/meta/editing/other/cloning-attributes-at-splitting-element.tentative.html.ini

@@ -1,11 +1,11 @@
 [cloning-attributes-at-splitting-element.tentative.html]
   [Cloning attributes at splitting an element in contenteditable]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [Cloning attributes at splitting an element in contenteditable 6]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [Cloning attributes at splitting an element in contenteditable 7]
     expected: FAIL
@@ -15,25 +15,25 @@
 
   [Cloning attributes at splitting an element in contenteditable 18]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [Cloning attributes at splitting an element in contenteditable 19]
     expected: FAIL
 
   [Cloning attributes at splitting an element in contenteditable 24]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [Cloning attributes at splitting an element in contenteditable 25]
     expected: FAIL
 
   [Cloning attributes at splitting an element in contenteditable 30]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [Cloning attributes at splitting an element in contenteditable 31]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [Cloning attributes at splitting an element in contenteditable 32]
     expected: FAIL

testing/web-platform/meta/editing/other/undo-insertparagraph-after-moving-split-nodes.html.ini

@@ -1,15 +1,15 @@
 [undo-insertparagraph-after-moving-split-nodes.html]
   [Undo insertParagraph after moving right node to different paragraph]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [Undo insertParagraph after moving left node to different paragraph]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [Undo insertParagraph after moving right node into the left node]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [Undo insertParagraph after moving left node into the right node]
     expected: FAIL

testing/web-platform/meta/editing/other/white-spaces-after-execCommand-insertlinebreak.tentative.html.ini

@@ -2,29 +2,29 @@
   prefs: [editor.white_space_normalization.blink_compatible:true]
   [execCommand("insertlinebreak", false, "") at "a      [\]   b"]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [execCommand("insertlinebreak", false, "") at "a       [\]  b"]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [execCommand("insertlinebreak", false, "") at "a        [\] b"]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [execCommand("insertlinebreak", false, "") at "a   [\]      b"]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [execCommand("insertlinebreak", false, "") at "a    [\]     b"]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [execCommand("insertlinebreak", false, "") at "a         [\]b"]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [execCommand("insertlinebreak", false, "") at "a     [\]    b"]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 

testing/web-platform/meta/editing/other/white-spaces-after-execCommand-insertparagraph.tentative.html.ini

@@ -4,7 +4,7 @@
     if (os == "android") and fission: [OK, TIMEOUT]
   [execCommand("insertparagraph", false, "") at "<div>a&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;[\]b</div>"]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [execCommand("insertparagraph", false, "") at "<div>a&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;[\]&nbsp;&nbsp;b</div>"]
     expected: FAIL
@@ -32,4 +32,4 @@
 
   [execCommand("insertparagraph", false, "") at "<div>a&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;[\]&nbsp;b</div>"]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL

testing/web-platform/meta/editing/run/delete.html.ini

@@ -336,43 +336,43 @@
 
   [[["defaultparagraphseparator","div"\],["delete",""\]\] "<p style=background-color:aqua>foo<p>[\]bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["defaultparagraphseparator","p"\],["delete",""\]\] "<p style=background-color:aqua>foo<p>[\]bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","true"\],["defaultparagraphseparator","div"\],["delete",""\]\] "<p style=background-color:aqua>foo<p style=background-color:tan>[\]bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","false"\],["defaultparagraphseparator","div"\],["delete",""\]\] "<p style=background-color:aqua>foo<p style=background-color:tan>[\]bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","true"\],["defaultparagraphseparator","p"\],["delete",""\]\] "<p style=background-color:aqua>foo<p style=background-color:tan>[\]bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","false"\],["defaultparagraphseparator","p"\],["delete",""\]\] "<p style=background-color:aqua>foo<p style=background-color:tan>[\]bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","true"\],["defaultparagraphseparator","div"\],["delete",""\]\] "<p>foo<p style=background-color:tan>[\]bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","false"\],["defaultparagraphseparator","div"\],["delete",""\]\] "<p>foo<p style=background-color:tan>[\]bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","true"\],["defaultparagraphseparator","p"\],["delete",""\]\] "<p>foo<p style=background-color:tan>[\]bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","false"\],["defaultparagraphseparator","p"\],["delete",""\]\] "<p>foo<p style=background-color:tan>[\]bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","true"\],["defaultparagraphseparator","div"\],["delete",""\]\] "<p style=text-decoration:underline>foo<p>[\]bar" compare innerHTML]
     expected: FAIL

testing/web-platform/meta/editing/run/forwarddelete.html.ini

@@ -264,43 +264,43 @@
 
   [[["defaultparagraphseparator","div"\],["forwarddelete",""\]\] "<p style=background-color:aqua>foo[\]<p>bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["defaultparagraphseparator","p"\],["forwarddelete",""\]\] "<p style=background-color:aqua>foo[\]<p>bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","true"\],["defaultparagraphseparator","div"\],["forwarddelete",""\]\] "<p style=background-color:aqua>foo[\]<p style=background-color:tan>bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","false"\],["defaultparagraphseparator","div"\],["forwarddelete",""\]\] "<p style=background-color:aqua>foo[\]<p style=background-color:tan>bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","true"\],["defaultparagraphseparator","p"\],["forwarddelete",""\]\] "<p style=background-color:aqua>foo[\]<p style=background-color:tan>bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","false"\],["defaultparagraphseparator","p"\],["forwarddelete",""\]\] "<p style=background-color:aqua>foo[\]<p style=background-color:tan>bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","true"\],["defaultparagraphseparator","div"\],["forwarddelete",""\]\] "<p>foo[\]<p style=background-color:tan>bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","false"\],["defaultparagraphseparator","div"\],["forwarddelete",""\]\] "<p>foo[\]<p style=background-color:tan>bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","true"\],["defaultparagraphseparator","p"\],["forwarddelete",""\]\] "<p>foo[\]<p style=background-color:tan>bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","false"\],["defaultparagraphseparator","p"\],["forwarddelete",""\]\] "<p>foo[\]<p style=background-color:tan>bar" compare innerHTML]
     expected:
-      if (not early_beta_or_earlier) or editorLegacyDirectionMode: FAIL
+      if not early_beta_or_earlier: FAIL
 
   [[["stylewithcss","true"\],["defaultparagraphseparator","div"\],["forwarddelete",""\]\] "<p style=text-decoration:underline>foo[\]<p>bar" compare innerHTML]
     expected: FAIL

testing/web-platform/tests/tools/wptrunner/wptrunner/browsers/firefox.py

@@ -195,8 +195,7 @@ def run_info_extras(**kwargs):
           "fission": not kwargs.get("disable_fission"),
           "sessionHistoryInParent": (not kwargs.get("disable_fission") or
                                      not get_bool_pref("fission.disableSessionHistoryInParent")),
-          "swgl": get_bool_pref("gfx.webrender.software"),
-          "editorLegacyDirectionMode": get_bool_pref_if_exists("editor.join_split_direction.compatible_with_the_other_browsers") is False}
+          "swgl": get_bool_pref("gfx.webrender.software")}
 
     rv.update(run_info_browser_version(**kwargs))
 
@@ -218,7 +217,7 @@ def run_info_browser_version(**kwargs):
 
 
 def update_properties():
-    return (["os", "debug", "fission", "processor", "swgl", "domstreams", "editorLegacyDirectionMode"],
+    return (["os", "debug", "fission", "processor", "swgl", "domstreams"],
             {"os": ["version"], "processor": ["bits"]})