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

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

Issue 8135001: Fixed behavior of the bookmark bar visibility. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: DCHECK_EQ instead of DCHECK Created 9 years, 2 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
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 const OptionsPage = options.OptionsPage; 6 const OptionsPage = options.OptionsPage;
7 const ArrayDataModel = cr.ui.ArrayDataModel; 7 const ArrayDataModel = cr.ui.ArrayDataModel;
8 8
9 // 9 //
10 // BrowserOptions class 10 // BrowserOptions class
(...skipping 30 matching lines...) Expand all
41 * Initialize BrowserOptions page. 41 * Initialize BrowserOptions page.
42 */ 42 */
43 initializePage: function() { 43 initializePage: function() {
44 // Call base class implementation to start preference initialization. 44 // Call base class implementation to start preference initialization.
45 OptionsPage.prototype.initializePage.call(this); 45 OptionsPage.prototype.initializePage.call(this);
46 46
47 // Wire up controls. 47 // Wire up controls.
48 $('startupUseCurrentButton').onclick = function(event) { 48 $('startupUseCurrentButton').onclick = function(event) {
49 chrome.send('setStartupPagesToCurrentPages'); 49 chrome.send('setStartupPagesToCurrentPages');
50 }; 50 };
51 $('toolbarShowBookmarksBar').onchange = function() {
52 chrome.send('toggleShowBookmarksBar');
53 };
54 $('defaultSearchManageEnginesButton').onclick = function(event) { 51 $('defaultSearchManageEnginesButton').onclick = function(event) {
55 OptionsPage.navigateToPage('searchEngines'); 52 OptionsPage.navigateToPage('searchEngines');
56 chrome.send('coreOptionsUserMetricsAction', 53 chrome.send('coreOptionsUserMetricsAction',
57 ['Options_ManageSearchEngines']); 54 ['Options_ManageSearchEngines']);
58 }; 55 };
59 $('defaultSearchEngine').onchange = this.setDefaultSearchEngine_; 56 $('defaultSearchEngine').onchange = this.setDefaultSearchEngine_;
60 57
61 var self = this; 58 var self = this;
62 $('instantEnabledCheckbox').customChangeHandler = function(event) { 59 $('instantEnabledCheckbox').customChangeHandler = function(event) {
63 if (this.checked) { 60 if (this.checked) {
(...skipping 293 matching lines...) Expand 10 before | Expand all | Expand 10 after
357 BrowserOptions.setInstantFieldTrialStatus = function(enabled) { 354 BrowserOptions.setInstantFieldTrialStatus = function(enabled) {
358 BrowserOptions.getInstance().setInstantFieldTrialStatus_(enabled); 355 BrowserOptions.getInstance().setInstantFieldTrialStatus_(enabled);
359 }; 356 };
360 357
361 // Export 358 // Export
362 return { 359 return {
363 BrowserOptions: BrowserOptions 360 BrowserOptions: BrowserOptions
364 }; 361 };
365 362
366 }); 363 });
OLDNEW
« no previous file with comments | « chrome/browser/prefs/pref_model_associator.cc ('k') | chrome/browser/ui/bookmarks/bookmark_tab_helper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698