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

Side by Side Diff: chrome/browser/resources/settings/settings_page/settings_page_visibility.js

Issue 1477773003: Use dom-if to hide settings pages and show explicitly. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 /**
6 * @fileoverview
7 * Behavior common to all Settings pages
8 *
9 * Example:
10 * behaviors: [SettingsPageVisibility],
11 *
12 * @group Chrome UI Behavior
13 */
14
15 /**
16 * Set this to override page visibility, e.g. in tests.
17 * @type !Object|undefined
18 */
19 var SettingPageDefaultVisibility;
20
21 /** @polymerBehavior */
22 var SettingsPageVisibility = {
23 properties: {
24 /** Dictionary of pages to hide. Includes 'basic' and 'advanced'. */
25 pageVisible: {
26 type: Object,
27 value: SettingPageDefaultVisibility,
28 notify: true,
29 },
30 },
31
32 /**
33 * Sets the visibility for each page in |pages| to true, causing it to load.
34 * @param {!Array<string>} pages
35 */
36 setVisiblePages(pages) {
37 var visible = this.pageVisible || {};
38 for (var p of pages) {
39 visible[p] = true;
40 }
41 this.set('pageVisible', visible);
42 }
43 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698