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

Unified Diff: chrome/browser/resources/options/advanced_options.js

Issue 6954001: Add "Keep chrome running in background" preference. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Review feedback + added tests. Created 9 years, 7 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/options/advanced_options.js
diff --git a/chrome/browser/resources/options/advanced_options.js b/chrome/browser/resources/options/advanced_options.js
index fa845a86da0b158683cc286aaf0a273fd5cb5afe..467cbaa3c5abcdce67dde22f893fb6cb243e22ed 100644
--- a/chrome/browser/resources/options/advanced_options.js
+++ b/chrome/browser/resources/options/advanced_options.js
@@ -113,6 +113,12 @@ var OptionsPage = options.OptionsPage;
chrome.send('useTLS1CheckboxAction',
[String($('sslUseTLS1').checked)]);
};
+ if (!cr.isChromeOS && !cr.isMac) {
Evan Stade 2011/05/12 18:36:32 can you make this a check for the existence of bac
Andrew T Wilson (Slow) 2011/05/12 23:34:31 Done.
+ $('backgroundModeCheckbox').onclick = function(event) {
+ chrome.send('backgroundModeAction',
+ [String($('backgroundModeCheckbox').checked)]);
+ };
+ }
// 'cloudPrintProxyEnabled' is true for Chrome branded builds on
// certain platforms, or could be enabled by a lab.
@@ -236,6 +242,11 @@ var OptionsPage = options.OptionsPage;
$('sslUseTLS1').disabled = disabled;
};
+ // Set the checked state for the backgroundModeCheckbox element.
+ AdvancedOptions.SetBackgroundModeCheckboxState = function(checked) {
+ $('backgroundModeCheckbox').checked = checked;
+ };
+
// Set the Cloud Print proxy UI to enabled, disabled, or processing.
AdvancedOptions.SetupCloudPrintProxySection = function(
disabled, label, allowed) {

Powered by Google App Engine
This is Rietveld 408576698