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

Unified Diff: chrome/browser/resources/settings/about_page/about_page_browser_proxy.js

Issue 2873193002: Make update over cellular an option for user (Closed)
Patch Set: Move update engine related code to another CL Created 3 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/settings/about_page/about_page_browser_proxy.js
diff --git a/chrome/browser/resources/settings/about_page/about_page_browser_proxy.js b/chrome/browser/resources/settings/about_page/about_page_browser_proxy.js
index 2b592acb5631f84a196c887c13c8f205cfcb8abe..b8c3bfbc7462ae7f1810d1e66b00bdd92595833e 100644
--- a/chrome/browser/resources/settings/about_page/about_page_browser_proxy.js
+++ b/chrome/browser/resources/settings/about_page/about_page_browser_proxy.js
@@ -58,6 +58,7 @@ var UpdateStatus = {
FAILED: 'failed',
DISABLED: 'disabled',
DISABLED_BY_ADMIN: 'disabled_by_admin',
+ NEED_PERMISSION_TO_UPDATE: 'need_permission_to_update',
};
// <if expr="_google_chrome and is_macosx">
@@ -146,6 +147,14 @@ cr.define('settings', function() {
*/
requestUpdate: function() {},
+ /**
+ * Checks for the update with specified version and size and applies over
+ * cellular.
stevenjb 2017/05/11 17:20:15 Maybe provide a brief explanation of why/how size
weidongg 2017/05/11 18:09:30 Done.
+ * @param {string} target_version
+ * @param {string} target_size
+ */
+ requestUpdateOverCellular: function(target_version, target_size) {},
+
/**
* @param {!BrowserChannel} channel
* @param {boolean} isPowerwashAllowed
@@ -213,6 +222,11 @@ cr.define('settings', function() {
chrome.send('requestUpdate');
},
+ /** @override */
+ requestUpdateOverCellular: function(target_version, target_size) {
+ chrome.send('requestUpdateOverCellular', [target_version, target_size]);
+ },
+
/** @override */
setChannel: function(channel, isPowerwashAllowed) {
chrome.send('setChannel', [channel, isPowerwashAllowed]);

Powered by Google App Engine
This is Rietveld 408576698