Make update over cellular an option for user

chrome/browser/resources/settings/about_page/about_page.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 'settings-about-page' contains version and OS related
  * information.
  */
 Polymer({
   is: 'settings-about-page',
 
   behaviors: [WebUIListenerBehavior, MainPageBehavior, I18nBehavior],
 
   properties: {
@@ skipping 62 matching lines @@
     /** @private {!Map<string, string>} */
     focusConfig_: {
       type: Object,
       value: function() {
         var map = new Map();
         map.set(
             settings.Route.DETAILED_BUILD_INFO.path,
             '#detailed-build-info-trigger');
         return map;
       },
-    }
+    },
+
+    /** @private */
+    showUpdateWarningDialog_: {
+      type: Boolean,
+      value: false,
+    },
 // </if>
   },
 
   observers: [
 // <if expr="not chromeos">
     'updateShowUpdateStatus_(' +
         'obsoleteSystemInfo_, currentUpdateStatusEvent_)',
     'updateShowRelaunch_(currentUpdateStatusEvent_)',
     'updateShowButtonContainer_(showRelaunch_)',
 // </if>
@@ skipping 58 matching lines @@
   },
 
   /**
    * @param {!UpdateStatusChangedEvent} event
    * @private
    */
   onUpdateStatusChanged_: function(event) {
 // <if expr="chromeos">
     if (event.status == UpdateStatus.CHECKING)
       this.hasCheckedForUpdates_ = true;
+    else if (event.status == UpdateStatus.NEED_PERMISSION_TO_UPDATE) {
+      this.showUpdateWarningDialog_ = true;
+      // Async to wait for dialog to appear in the DOM.
+      this.async(function() {
+        var dialog = this.$$('settings-update-warning-dialog');
+        // event includes the update version and size in bytes.
+        dialog.setUpdateWarningMessage(event.version, event.size);

[stevenjb, 2017/05/10 23:29:05]

This can be done a bit more cleanly by using data binding to set something like
this.updateInfo_ = { version: event.version, size: event.size } then using data
binding to bind updateInfo_ to the dialog, e.g. update-info="[[updateInfo_]] as
an attribute of the dialog.

[weidongg, 2017/05/11 16:34:05]

Done.

+        // Register listener to detect when the dialog is closed. Flip the
+        // boolean once closed to force a restamp next time it is shown such
+        // that the previous dialog's contents are cleared.
+        dialog.addEventListener('close', function() {
+          this.showUpdateWarningDialog_ = false;
+        }.bind(this));

[stevenjb, 2017/05/10 23:29:05]

This can be done by adding on-close="onUpdateWarningDialogClose_" to the HTML
and moving this to the named function. This is generally preferred to injecting
the event in JS like this.

[weidongg, 2017/05/11 16:34:05]

Done.

+      }.bind(this));
+    }
 // </if>
     this.currentUpdateStatusEvent_ = event;
   },
 
 // <if expr="_google_chrome and is_macosx">
   /**
    * @param {!PromoteUpdaterStatus} status
    * @private
    */
   onPromoteUpdaterStatusChanged_: function(status) {
@@ skipping 75 matching lines @@
   },
 
   /**
    * @return {string}
    * @private
    */
   getUpdateStatusMessage_: function() {
     switch (this.currentUpdateStatusEvent_.status) {
       case UpdateStatus.CHECKING:
         return this.i18n('aboutUpgradeCheckStarted');
+      case UpdateStatus.NEED_PERMISSION_TO_UPDATE:
+        return "";
       case UpdateStatus.NEARLY_UPDATED:
 // <if expr="chromeos">
         if (this.currentChannel_ != this.targetChannel_)
           return this.i18n('aboutUpgradeSuccessChannelSwitch');
 // </if>
         return this.i18n('aboutUpgradeRelaunch');
       case UpdateStatus.UPDATED:
         return this.i18n('aboutUpgradeUpToDate');
       case UpdateStatus.UPDATING:
         assert(typeof this.currentUpdateStatusEvent_.progress == 'number');
@@ skipping 121 matching lines @@
   /**
    * @return {boolean}
    * @private
    */
   computeShowCheckUpdates_: function() {
     // Enable the update button if we are in a stale 'updated' status or
     // update has failed. Disable it otherwise.
     var staleUpdatedStatus = !this.hasCheckedForUpdates_ &&
         this.checkStatus_(UpdateStatus.UPDATED);
 
-    return staleUpdatedStatus || this.checkStatus_(UpdateStatus.FAILED);
+    return staleUpdatedStatus || this.checkStatus_(UpdateStatus.FAILED) ||
+        this.checkStatus_(UpdateStatus.NEED_PERMISSION_TO_UPDATE);
   },
 
   /**
    * @return {boolean}
    * @private
    */
   shouldShowRegulatoryInfo_: function() {
     return this.regulatoryInfo_ !== null;
   },
 // </if>
 
   /** @private */
   onProductLogoTap_: function() {
     this.$['product-logo'].animate({
       transform: ['none', 'rotate(-10turn)'],
     }, {
       duration: 500,
       easing: 'cubic-bezier(1, 0, 0, 1)',
     });
   },
 
 // <if expr="_google_chrome">
   /** @private */
   onReportIssueTap_: function() {
     this.aboutBrowserProxy_.openFeedbackDialog();
   },
 // </if>
 });