Make update over cellular an option for user

chrome/browser/resources/settings/about_page/about_page_browser_proxy.js

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview A helper object used from the "About" section to interact with
  * the browser.
  */
 
 // <if expr="chromeos">
@@ skipping 40 matching lines @@
  * @enum {string}
  */
 var UpdateStatus = {
   CHECKING: 'checking',
   UPDATING: 'updating',
   NEARLY_UPDATED: 'nearly_updated',
   UPDATED: 'updated',
   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">
 /**
  * @typedef {{
  *   hidden: boolean,
  *   disabled: boolean,
  *   actionable: boolean,
  *   text: (string|undefined)
  * }}
  */
 var PromoteUpdaterStatus;
 // </if>
 
 /**
  * @typedef {{
  *   status: !UpdateStatus,
  *   progress: (number|undefined),
  *   message: (string|undefined),
  *   connectionTypes: (string|undefined),

[stevenjb, 2017/05/11 17:20:15]

It looks like this now has 'version' and 'size' properties?

[weidongg, 2017/05/11 18:09:29]

Done.

  * }}
  */
 var UpdateStatusChangedEvent;
 
 cr.define('settings', function() {
   /**
    * @param {!BrowserChannel} channel
    * @return {string}
    */
   function browserChannelToI18nId(channel) {
@@ skipping 49 matching lines @@
     openFeedbackDialog: function() {},
 // </if>
 
 // <if expr="chromeos">
     /**
      * Checks for available update and applies if it exists.
      */
     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 is used and where it comes
from, since it is not obvious.

[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
      */
     setChannel: function(channel, isPowerwashAllowed) {},
 
     /** @return {!Promise<!ChannelInfo>} */
     getChannelInfo: function() {},
 
     /** @return {!Promise<!VersionInfo>} */
     getVersionInfo: function() {},
@@ skipping 47 matching lines @@
     },
 // </if>
 
 // <if expr="chromeos">
     /** @override */
     requestUpdate: 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]);
     },
 
     /** @override */
     getChannelInfo: function() {
       return cr.sendWithPromise('getChannelInfo');
     },
 
     /** @override */
     getVersionInfo: function() {
       return cr.sendWithPromise('getVersionInfo');
     },
 
     /** @override */
     getRegulatoryInfo: function() {
       return cr.sendWithPromise('getRegulatoryInfo');
     }
 // </if>
   };
 
   return {
     AboutPageBrowserProxy: AboutPageBrowserProxy,
     AboutPageBrowserProxyImpl: AboutPageBrowserProxyImpl,
     browserChannelToI18nId: browserChannelToI18nId,
     isTargetChannelMoreStable: isTargetChannelMoreStable,
   };
 });