Show proper message in about Chrome OS page

chrome/browser/ui/webui/help/help_utils_chromeos.h

 // Copyright 2013 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.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_HELP_HELP_UTILS_CHROMEOS_H_
 #define CHROME_BROWSER_UI_WEBUI_HELP_HELP_UTILS_CHROMEOS_H_
 
 #include <string>
 
 #include "base/strings/string16.h"
 
 namespace help_utils_chromeos {
 
-// Returns true if update over cellular networks is enabled.
-bool IsUpdateOverCellularAllowed();
+// Returns true if update over cellular networks is enabled. |interactive|
+// indicates whether the user is actively checking for update. By default, we
+// allow updates in Chrome UI if the user triggers the action (e.g clicking
+// a button) and leave the final decision as to whether to allow such updates
+// up to the update engine.

[stevenjb, 2017/05/22 22:57:32]

This comment confused me until I examined the code. How about something like:

Returns true if updates over cellular networks are allowed. If |interactive| is
true (e.g. the user clicked on a 'check for updates' button), updates over
cellular are allowed unless prohibited by policy. If |interactive| is false,
updates over cellular may be allowed by default for the device type, or by
policy.

[weidongg, 2017/05/23 00:17:00]

Sorry for the confusing comment. Thanks, this is much better.

+bool IsUpdateOverCellularAllowed(bool interactive);
 
 // Returns localized name for the connection |type|.
 base::string16 GetConnectionTypeAsUTF16(const std::string& type);
 
 }  // namespace help_utils_chromeos
 
 #endif  // CHROME_BROWSER_UI_WEBUI_HELP_HELP_UTILS_CHROMEOS_H_