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

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

Issue 793503003: Fix metrics in actions.xml (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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
« no previous file with comments | « no previous file | tools/metrics/actions/actions.xml » ('j') | 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) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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.exportPath('options'); 5 cr.exportPath('options');
6 6
7 /** 7 /**
8 * @typedef {{actionLinkText: (string|undefined), 8 * @typedef {{actionLinkText: (string|undefined),
9 * childUser: (boolean|undefined), 9 * childUser: (boolean|undefined),
10 * hasError: (boolean|undefined), 10 * hasError: (boolean|undefined),
(...skipping 550 matching lines...) Expand 10 before | Expand all | Expand 10 after
561 if (loadTimeData.getBoolean('websiteSettingsManagerEnabled')) { 561 if (loadTimeData.getBoolean('websiteSettingsManagerEnabled')) {
562 $('website-settings-section').hidden = false; 562 $('website-settings-section').hidden = false;
563 $('website-management-button').onclick = function(event) { 563 $('website-management-button').onclick = function(event) {
564 PageManager.showPageByName('websiteSettings'); 564 PageManager.showPageByName('websiteSettings');
565 }; 565 };
566 } 566 }
567 567
568 // Web Content section. 568 // Web Content section.
569 $('fontSettingsCustomizeFontsButton').onclick = function(event) { 569 $('fontSettingsCustomizeFontsButton').onclick = function(event) {
570 PageManager.showPageByName('fonts'); 570 PageManager.showPageByName('fonts');
571 chrome.send('coreOptionsUserMetricsAction', ['Options_FontSettings']); 571 chrome.send('coreOptionsUserMetricsAction',
572 ['Options_ShowFontSettings']);
572 }; 573 };
573 $('defaultFontSize').onchange = function(event) { 574 $('defaultFontSize').onchange = function(event) {
574 var value = event.target.options[event.target.selectedIndex].value; 575 var value = event.target.options[event.target.selectedIndex].value;
575 Preferences.setIntegerPref( 576 Preferences.setIntegerPref(
576 'webkit.webprefs.default_fixed_font_size', 577 'webkit.webprefs.default_fixed_font_size',
577 value - OptionsPage.SIZE_DIFFERENCE_FIXED_STANDARD, true); 578 value - OptionsPage.SIZE_DIFFERENCE_FIXED_STANDARD, true);
578 chrome.send('defaultFontSizeAction', [String(value)]); 579 chrome.send('defaultFontSizeAction', [String(value)]);
579 }; 580 };
580 $('defaultZoomFactor').onchange = function(event) { 581 $('defaultZoomFactor').onchange = function(event) {
581 chrome.send('defaultZoomFactorAction', 582 chrome.send('defaultZoomFactorAction',
(...skipping 1622 matching lines...) Expand 10 before | Expand all | Expand 10 after
2204 } 2205 }
2205 button.textContent = loadTimeData.getString(strId); 2206 button.textContent = loadTimeData.getString(strId);
2206 }; 2207 };
2207 } 2208 }
2208 2209
2209 // Export 2210 // Export
2210 return { 2211 return {
2211 BrowserOptions: BrowserOptions 2212 BrowserOptions: BrowserOptions
2212 }; 2213 };
2213 }); 2214 });
OLDNEW
« no previous file with comments | « no previous file | tools/metrics/actions/actions.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698