Changeset 246821 in webkit

Timestamp:

Jun 25, 2019 4:45:37 PM (5 years ago)

Author:

Matt Baker

Message:

Web Inspector: Elements: show shadow DOM by default
​https://bugs.webkit.org/show_bug.cgi?id=199128

Reviewed by Devin Rousso.

• UserInterface/Base/Setting.js:

• UserInterface/Test/Test.js:

(WI.loaded):
Overriding the default value isn't necessary since the default is now true.

Location:

trunk/Source/WebInspectorUI

Files:

• ChangeLog (modified) (1 diff)
• UserInterface/Base/Setting.js (modified) (1 diff)
• UserInterface/Test/Test.js (modified) (1 diff)

trunk/Source/WebInspectorUI/ChangeLog

@@ +1 @@
+2019-06-25  Matt Baker  <mattbaker@apple.com>
+
+        Web Inspector: Elements: show shadow DOM by default
+        https://bugs.webkit.org/show_bug.cgi?id=199128
+
+        Reviewed by Devin Rousso.
+
+        * UserInterface/Base/Setting.js:
+
+        * UserInterface/Test/Test.js:
+        (WI.loaded):
+        Overriding the default value isn't necessary since the default is now true.
+
 2019-06-25  Nikita Vasilyev  <nvasilyev@apple.com>
 

trunk/Source/WebInspectorUI/UserInterface/Base/Setting.js

@@ -169 +169 @@
     showRulersDuringElementSelection: new WI.Setting("show-rulers-during-element-selection", true),
     showScopeChainOnPause: new WI.Setting("show-scope-chain-sidebar", true),
-    showShadowDOM: new WI.Setting("show-shadow-dom", false),
+    showShadowDOM: new WI.Setting("show-shadow-dom", true),
     showWhitespaceCharacters: new WI.Setting("show-whitespace-characters", false),
     tabSize: new WI.Setting("tab-size", 4),

trunk/Source/WebInspectorUI/UserInterface/Test/Test.js

@@ -69 +69 @@
     // Register for events.
     document.addEventListener("DOMContentLoaded", WI.contentLoaded);
-
-    // Non-default global setting values for tests.
-    WI.settings.showShadowDOM.value = true;
 
     // Targets.