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

Side by Side Diff: chrome/browser/resources/settings/privacy_page/privacy_page.js

Issue 2193953002: MD Settings: Remove unused settings.main.rendered global variable. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix closure Created 4 years, 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 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 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * @fileoverview 6 * @fileoverview
7 * 'settings-privacy-page' is the settings page containing privacy and 7 * 'settings-privacy-page' is the settings page containing privacy and
8 * security settings. 8 * security settings.
9 */ 9 */
10 Polymer({ 10 Polymer({
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 * @private 49 * @private
50 */ 50 */
51 computeShowClearBrowsingDataDialog_: function() { 51 computeShowClearBrowsingDataDialog_: function() {
52 var route = this.currentRoute; 52 var route = this.currentRoute;
53 return route && route.dialog == 'clear-browsing-data'; 53 return route && route.dialog == 'clear-browsing-data';
54 }, 54 },
55 55
56 /** @private */ 56 /** @private */
57 onManageCertificatesTap_: function() { 57 onManageCertificatesTap_: function() {
58 <if expr="use_nss_certs"> 58 <if expr="use_nss_certs">
59 var pages = /** @type {!SettingsAnimatedPagesElement} */(this.$.pages);
60 settings.navigateTo(settings.Route.CERTIFICATES); 59 settings.navigateTo(settings.Route.CERTIFICATES);
61 </if> 60 </if>
62 <if expr="is_win or is_macosx"> 61 <if expr="is_win or is_macosx">
63 settings.PrivacyPageBrowserProxyImpl.getInstance(). 62 settings.PrivacyPageBrowserProxyImpl.getInstance().
64 showManageSSLCertificates(); 63 showManageSSLCertificates();
Dan Beam 2016/07/29 21:50:17 nit: want to further indent this while you're here
dpapad 2016/07/29 21:52:31 Done.
65 </if> 64 </if>
66 }, 65 },
67 66
68 /** @private */ 67 /** @private */
69 onSiteSettingsTap_: function() { 68 onSiteSettingsTap_: function() {
70 var pages = /** @type {!SettingsAnimatedPagesElement} */(this.$.pages);
71 settings.navigateTo(settings.Route.SITE_SETTINGS); 69 settings.navigateTo(settings.Route.SITE_SETTINGS);
72 }, 70 },
73 71
74 /** @private */ 72 /** @private */
75 onClearBrowsingDataTap_: function() { 73 onClearBrowsingDataTap_: function() {
76 settings.navigateTo(settings.Route.CLEAR_BROWSER_DATA); 74 settings.navigateTo(settings.Route.CLEAR_BROWSER_DATA);
77 }, 75 },
78 76
79 /** @private */ 77 /** @private */
80 onDialogClosed_: function() { 78 onDialogClosed_: function() {
81 settings.navigateTo(settings.Route.PRIVACY); 79 settings.navigateTo(settings.Route.PRIVACY);
82 }, 80 },
83 }); 81 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698