Make update over cellular an option for user

chrome/browser/resources/settings/about_page/about_page.html

 <link rel="import" href="chrome://resources/html/polymer.html">
 <link rel="import" href="about_page_browser_proxy.html">
+<link rel="import" href="update_warning_dialog.html">
 <link rel="import" href="../i18n_setup.html">
 <link rel="import" href="../icons.html">
 <link rel="import" href="../lifetime_browser_proxy.html">
 <link rel="import" href="../route.html">
 <link rel="import" href="../settings_page/main_page_behavior.html">
 <link rel="import" href="../settings_page/settings_animated_pages.html">
 <link rel="import" href="../settings_page/settings_section.html">
 <link rel="import" href="../settings_page_css.html">
 <link rel="import" href="../settings_shared_css.html">
 <link rel="import" href="chrome://resources/cr_elements/icons.html">
@@ skipping 202 matching lines @@
 <if expr="chromeos">
           <template is="dom-if" route-path="/help/details">
             <settings-subpage page-title="$i18n{aboutDetailedBuildInfo}">
               <settings-detailed-build-info></settings-detailed-build-info>
             </settings-subpage>
           </template>
 </if>
         </settings-animated-pages>
       </settings-section>
     </div>
+    <template is="dom-if" if="[[showUpdateWarningDialog_]]" restamp>

[xiyuan, 2017/05/11 20:37:25]

Should this block under <if expr="chromeos"> ?

[stevenjb, 2017/05/11 21:03:45]

Good catch. Yes it should.

[weidongg, 2017/05/11 23:50:44]

Yes, you are right.

+      <settings-update-warning-dialog update-info="[[updateInfo_]]"
+          on-close="onUpdateWarningDialogClose_">
+      </settings-update-warning-dialog>
+    </template>
   </template>
   <script src="about_page.js"></script>
 </dom-module>