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

Side by Side Diff: chrome/browser/resources/options/certificate_manager.js

Issue 7104079: Fix certificate manager starting with no tab selected if opened by URL. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/browser/resources/options/advanced_options.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 cr.define('options', function() { 5 cr.define('options', function() {
6 6
7 var OptionsPage = options.OptionsPage; 7 var OptionsPage = options.OptionsPage;
8 8
9 ///////////////////////////////////////////////////////////////////////////// 9 /////////////////////////////////////////////////////////////////////////////
10 // CertificateManagerTab class: 10 // CertificateManagerTab class:
(...skipping 184 matching lines...) Expand 10 before | Expand all | Expand 10 after
195 initalized_: false, 195 initalized_: false,
196 196
197 /** 197 /**
198 * Handler for OptionsPage's visible property change event. 198 * Handler for OptionsPage's visible property change event.
199 * @private 199 * @private
200 * @param {Event} e Property change event. 200 * @param {Event} e Property change event.
201 */ 201 */
202 handleVisibleChange_: function(e) { 202 handleVisibleChange_: function(e) {
203 if (!this.initalized_ && this.visible) { 203 if (!this.initalized_ && this.visible) {
204 this.initalized_ = true; 204 this.initalized_ = true;
205 OptionsPage.showTab($('personal-certs-nav-tab'));
205 chrome.send('populateCertificateManager'); 206 chrome.send('populateCertificateManager');
206 } 207 }
207 208
208 if (cr.isChromeOS) { 209 if (cr.isChromeOS) {
209 // Ensure TPM token check on visible and stop polling when hidden. 210 // Ensure TPM token check on visible and stop polling when hidden.
210 if (this.visible) 211 if (this.visible)
211 checkTpmToken(); 212 checkTpmToken();
212 else 213 else
213 stopTpmTokenCheckPolling(); 214 stopTpmTokenCheckPolling();
214 } 215 }
(...skipping 25 matching lines...) Expand all
240 } 241 }
241 }; 242 };
242 243
243 // Export 244 // Export
244 return { 245 return {
245 CertificateManagerTab: CertificateManagerTab, 246 CertificateManagerTab: CertificateManagerTab,
246 CertificateManager: CertificateManager 247 CertificateManager: CertificateManager
247 }; 248 };
248 249
249 }); 250 });
OLDNEW
« no previous file with comments | « chrome/browser/resources/options/advanced_options.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698