Bug 1322938 - Put <dialog> element behind preference.

Tag #1343

dom/html/HTMLDialogElement.cpp

@@ -6,8 +6,20 @@
 
 #include "mozilla/dom/HTMLDialogElement.h"
 #include "mozilla/dom/HTMLDialogElementBinding.h"
+#include "mozilla/dom/HTMLUnknownElement.h"
+#include "mozilla/Preferences.h"
 
-NS_IMPL_NS_NEW_HTML_ELEMENT(Dialog)
+// Expand NS_IMPL_NS_NEW_HTML_ELEMENT(Dialog) with pref check
+nsGenericHTMLElement*
+NS_NewHTMLDialogElement(already_AddRefed<mozilla::dom::NodeInfo>&& aNodeInfo,
+                         mozilla::dom::FromParser aFromParser)
+{
+  if (!mozilla::dom::HTMLDialogElement::IsDialogEnabled()) {
+    return new mozilla::dom::HTMLUnknownElement(aNodeInfo);
+  }
+
+  return new mozilla::dom::HTMLDialogElement(aNodeInfo);
+}
 
 namespace mozilla {
 namespace dom {
@@ -18,6 +30,21 @@ HTMLDialogElement::~HTMLDialogElement()
 
 NS_IMPL_ELEMENT_CLONE(HTMLDialogElement)
 
+bool
+HTMLDialogElement::IsDialogEnabled()
+{
+  static bool isDialogEnabled = false;
+  static bool added = false;
+
+  if (!added) {
+    Preferences::AddBoolVarCache(&isDialogEnabled,
+                                 "dom.dialog_element.enabled");
+    added = true;
+  }
+
+  return isDialogEnabled;
+}
+
 void
 HTMLDialogElement::Close(const mozilla::dom::Optional<nsAString>& aReturnValue)
 {

dom/html/HTMLDialogElement.h

@@ -26,6 +26,8 @@ public:
 
   virtual nsresult Clone(mozilla::dom::NodeInfo* aNodeInfo, nsINode** aResult) const override;
 
+  static bool IsDialogEnabled();
+
   bool Open() const { return GetBoolAttr(nsGkAtoms::open); }
   void SetOpen(bool aOpen, ErrorResult& aError)
   {

dom/tests/mochitest/general/test_interfaces.html

@@ -443,7 +443,7 @@ var interfaceNamesInGlobalScope =
 // IMPORTANT: Do not change this list without review from a DOM peer!
     "HTMLDetailsElement",
 // IMPORTANT: Do not change this list without review from a DOM peer!
-    "HTMLDialogElement",
+    {name: "HTMLDialogElement", disabled: true},
 // IMPORTANT: Do not change this list without review from a DOM peer!
     "HTMLDirectoryElement",
 // IMPORTANT: Do not change this list without review from a DOM peer!

dom/webidl/HTMLDialogElement.webidl

@@ -11,6 +11,7 @@
  * and create derivative works of this document.
  */
 
+[Pref="dom.dialog_element.enabled"]
 interface HTMLDialogElement : HTMLElement {
   [SetterThrows] attribute boolean open;
   attribute DOMString returnValue;

modules/libpref/init/all.js

@@ -5368,6 +5368,9 @@ pref("narrate.filter-voices", true);
 
 pref("dom.audiochannel.mutedByDefault", false);
 
+// HTML <dialog> element
+pref("dom.dialog_element.enabled", false);
+
 // Enable <details> and <summary> tags.
 pref("dom.details_element.enabled", true);
 

testing/web-platform/meta/html/dom/interfaces.html.ini

@@ -1,6 +1,6 @@
 [interfaces.html]
   type: testharness
-  prefs: [dom.forms.inputmode:true, dom.details_element.enabled:true]
+  prefs: [dom.forms.inputmode:true, dom.details_element.enabled:true, dom.dialog_element.enabled:true]
   [Document interface: attribute domain]
     expected: FAIL
 

testing/web-platform/meta/html/dom/reflection-misc.html.ini

@@ -1,6 +1,6 @@
 [reflection-misc.html]
   type: testharness
-  prefs: [dom.details_element.enabled:true]
+  prefs: [dom.details_element.enabled: true, dom.dialog_element.enabled: true]
   [html.tabIndex: setAttribute() to object "3" followed by getAttribute()]
     expected: FAIL
 

testing/web-platform/meta/html/semantics/interactive-elements/the-dialog-element/dialog-close.html.ini

@@ -0,0 +1,3 @@
+[dialog-close.html]
+  type: testharness
+  prefs: [dom.dialog_element.enabled:true]

testing/web-platform/meta/html/semantics/interactive-elements/the-dialog-element/dialog-open.html.ini

@@ -0,0 +1,3 @@
+[dialog-open.html]
+  type: testharness
+  prefs: [dom.dialog_element.enabled:true]

testing/web-platform/meta/html/semantics/interactive-elements/the-dialog-element/dialog-showModal.html.ini

@@ -1,5 +1,6 @@
 [dialog-showModal.html]
   type: testharness
+  prefs: [dom.dialog_element.enabled:true]
   [dialog element: showModal()]
     expected: FAIL
 

testing/web-platform/meta/html/semantics/interfaces.html.ini

@@ -1,6 +1,6 @@
 [interfaces.html]
   type: testharness
-  prefs: [dom.details_element.enabled:true]
+  prefs: [dom.details_element.enabled: true, dom.dialog_element.enabled: true]
   [Interfaces for image]
     expected: FAIL