Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1728)

Unified Diff: chrome/browser/resources/settings/page_visibility.js

Issue 2962823002: MD Settings: decouple page visibility from settings_ui.html. (Closed)
Patch Set: moving import out of this CL Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/settings/page_visibility.js
diff --git a/chrome/browser/resources/settings/page_visibility.js b/chrome/browser/resources/settings/page_visibility.js
new file mode 100644
index 0000000000000000000000000000000000000000..0349d6cc83e89a7552abd18a126585cbe031eb08
--- /dev/null
+++ b/chrome/browser/resources/settings/page_visibility.js
@@ -0,0 +1,105 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+/**
+ * Specifies page visibility in guest mode in cr and cros.
+ * @typedef {{
+ * advancedSettings: (boolean|undefined),
+ * appearance: (boolean|undefined|AppearancePageVisibility),
+ * dateTime: (boolean|undefined|DateTimePageVisibility),
+ * defaultBrowser: (boolean|undefined),
+ * downloads: (undefined|DownloadsPageVisibility),
+ * onStartup: (boolean|undefined),
+ * passwordsAndForms: (boolean|undefined),
+ * people: (boolean|undefined),
+ * privacy: (undefined|PrivacyPageVisibility),
+ * reset:(boolean|undefined),
+ * }}
+ */
+var GuestModePageVisibility;
+
+/**
+ * @typedef {{
+ * bookmarksBar: boolean,
+ * homeButton: boolean,
+ * pageZoom: boolean,
+ * setTheme: boolean,
+ * setWallpaper: boolean,
+ * }}
+ */
+var AppearancePageVisibility;
+
+/**
+ * @typedef {{
+ * timeZoneSelector: boolean,
+ * }}
+ */
+var DateTimePageVisibility;
+
+/**
+ * @typedef {{
+ * googleDrive: boolean
+ * }}
+ */
+var DownloadsPageVisibility;
+
+/**
+ * @typedef {{
+ * networkPrediction: boolean,
+ * searchPrediction: boolean,
+ * }}
+ */
+var PrivacyPageVisibility;
+
+cr.define('settings', function() {
+
+ /**
+ * Dictionary defining page visibility.
+ * This is only set when in guest mode. All pages are visible when not set
+ * because polymer only notifies after a property is set.
+ * @type {!GuestModePageVisibility}
+ */
+ var pageVisibility = {};
+
+ if (loadTimeData.getBoolean('isGuest')) {
+ // "if not chromeos" and "if chromeos" in two completely separate blocks
+ // to work around closure compiler.
+ // <if expr="not chromeos">
+ pageVisibility = {
+ passwordsAndForms: false,
+ people: false,
+ onStartup: false,
+ reset: false,
+ appearance: false,
+ defaultBrowser: false,
+ advancedSettings: false,
+ };
+ // </if>
+ // <if expr="chromeos">
+ pageVisibility = {
+ passwordsAndForms: false,
+ people: false,
+ onStartup: false,
+ reset: false,
+ appearance: {
+ setWallpaper: false,
+ setTheme: false,
+ homeButton: false,
+ bookmarksBar: false,
+ pageZoom: false,
+ },
+ advancedSettings: true,
+ privacy: {
+ searchPrediction: false,
+ networkPrediction: false,
+ },
+ downloads: {
+ googleDrive: false,
+ },
+ };
+ // </if>
+ }
+
+ return {pageVisibility: pageVisibility};
+});

Powered by Google App Engine
This is Rietveld 408576698