Make DocumentModifiedEvent and HTMLEditor::OnModifyDocument assume there is no editing host

masayuki-nakano · moz-wptsync-bot · commit 082839113615 · 2025-01-16T15:50:32.000Z
The logic before getting editing host in them are correct, but both `nsIContent::GetEditingHost()` and `HTMLEditor::ComputeEditingHostInternal()` return `Document::GetBody()` result if it's in the design mode. For now, we should just add nullptr checks into the methods since they are required only for some specific web apps. Differential Revision: https://phabricator.services.mozilla.com/D234442 bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1941827 gecko-commit: a177fb513de65e6af5daf405f96baced5702e3fa gecko-reviewers: m_kato
diff --git a/editing/crashtests/remove-all-children-of-documentElement-in-designMode-when-no-body.html b/editing/crashtests/remove-all-children-of-documentElement-in-designMode-when-no-body.html
@@ -0,0 +1,16 @@
+<!doctype html>
+<html>
+<head>
+<meta charset="utf-8">
+<script>
+document.addEventListener("DOMContentLoaded", () => {
+  document.designMode = "on";
+  document.documentElement.innerText = "abc def\n"; // causes removing the implicit <body> element
+  let child;
+  while ((child = document.documentElement.lastChild)) {
+    child.remove();
+  }
+}, {once: true});
+</script>
+</head>
+</html>
