Title: [254773] trunk/Source/WebInspectorUI
- Revision
- 254773
- Author
- nvasil...@apple.com
- Date
- 2020-01-17 16:16:09 -0800 (Fri, 17 Jan 2020)
Log Message
Web Inspector: Unchecking Enable Preview Features on Engineering and Preview builds does not affect WI.arePreviewFeaturesEnabled()
https://bugs.webkit.org/show_bug.cgi?id=204026
Reviewed by Brian Burg.
Make `WI.arePreviewFeaturesEnabled()` return false when Enable Preview Features is unchecked.
* UserInterface/Base/Setting.js:
(WI.canShowPreviewFeatures):
(WI.arePreviewFeaturesEnabled):
(WI.isTechnologyPreviewBuild): Deleted.
(WI.canShowPreviewFeatures): Added.
* UserInterface/Views/SettingsTabContentView.js:
Modified Paths
Diff
Modified: trunk/Source/WebInspectorUI/ChangeLog (254772 => 254773)
--- trunk/Source/WebInspectorUI/ChangeLog 2020-01-17 23:57:35 UTC (rev 254772)
+++ trunk/Source/WebInspectorUI/ChangeLog 2020-01-18 00:16:09 UTC (rev 254773)
@@ -1,3 +1,19 @@
+2020-01-17 Nikita Vasilyev <nvasil...@apple.com>
+
+ Web Inspector: Unchecking Enable Preview Features on Engineering and Preview builds does not affect WI.arePreviewFeaturesEnabled()
+ https://bugs.webkit.org/show_bug.cgi?id=204026
+
+ Reviewed by Brian Burg.
+
+ Make `WI.arePreviewFeaturesEnabled()` return false when Enable Preview Features is unchecked.
+
+ * UserInterface/Base/Setting.js:
+ (WI.canShowPreviewFeatures):
+ (WI.arePreviewFeaturesEnabled):
+ (WI.isTechnologyPreviewBuild): Deleted.
+ (WI.canShowPreviewFeatures): Added.
+ * UserInterface/Views/SettingsTabContentView.js:
+
2020-01-16 Devin Rousso <drou...@apple.com>
Web Inspector: add links to reference pages
Modified: trunk/Source/WebInspectorUI/UserInterface/Base/Setting.js (254772 => 254773)
--- trunk/Source/WebInspectorUI/UserInterface/Base/Setting.js 2020-01-17 23:57:35 UTC (rev 254772)
+++ trunk/Source/WebInspectorUI/UserInterface/Base/Setting.js 2020-01-18 00:16:09 UTC (rev 254773)
@@ -216,7 +216,7 @@
zoomFactor: new WI.Setting("zoom-factor", 1),
// Experimental
- experimentalEnablePreviewFeatures: new WI.Setting("experimental-enable-preview-features", false),
+ experimentalEnablePreviewFeatures: new WI.Setting("experimental-enable-preview-features", true),
experimentalEnableNewTabBar: new WI.Setting("experimental-enable-new-tab-bar", false),
experimentalEnableStylesJumpToEffective: new WI.Setting("experimental-styles-jump-to-effective", false),
@@ -245,18 +245,15 @@
WI.previewFeatures = [];
-WI.isTechnologyPreviewBuild = function()
+// WebKit may by default enable certain features in a Technology Preview that are not enabled in trunk.
+// Provide a switch that will make non-preview builds behave like an experimental build, for those preview features.
+WI.canShowPreviewFeatures = function()
{
- return WI.isExperimentalBuild && !WI.isEngineeringBuild;
+ let hasPreviewFeatures = WI.previewFeatures.length > 0;
+ return hasPreviewFeatures && WI.isExperimentalBuild;
};
WI.arePreviewFeaturesEnabled = function()
{
- if (WI.isExperimentalBuild)
- return true;
-
- if (WI.settings.experimentalEnablePreviewFeatures.value)
- return true;
-
- return false;
+ return WI.canShowPreviewFeatures() && WI.settings.experimentalEnablePreviewFeatures.value;
};
Modified: trunk/Source/WebInspectorUI/UserInterface/Views/SettingsTabContentView.js (254772 => 254773)
--- trunk/Source/WebInspectorUI/UserInterface/Views/SettingsTabContentView.js 2020-01-17 23:57:35 UTC (rev 254772)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/SettingsTabContentView.js 2020-01-18 00:16:09 UTC (rev 254773)
@@ -355,10 +355,7 @@
let initialValues = new Map;
- // WebKit may by default enable certain features in a Technology Preview that are not enabled in trunk.
- // Provide a switch that will make non-preview builds behave like an experimental build, for those preview features.
- let hasPreviewFeatures = WI.previewFeatures.length > 0;
- if (hasPreviewFeatures && (WI.isTechnologyPreviewBuild() || WI.isEngineeringBuild)) {
+ if (WI.canShowPreviewFeatures()) {
experimentalSettingsView.addSetting(WI.UIString("Staging:"), WI.settings.experimentalEnablePreviewFeatures, WI.UIString("Enable Preview Features"));
experimentalSettingsView.addSeparator();
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
