Show proper message in about Chrome OS page

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

Index: chrome/browser/ui/webui/help/help_utils_chromeos.h
diff --git a/chrome/browser/ui/webui/help/help_utils_chromeos.h b/chrome/browser/ui/webui/help/help_utils_chromeos.h
index ab58c49b8096c69a223d1b708d08cfc51d84c9c0..7c6c14e3e2c8d4e500ba2c931b1e490888259f7a 100644
--- a/chrome/browser/ui/webui/help/help_utils_chromeos.h
+++ b/chrome/browser/ui/webui/help/help_utils_chromeos.h
@@ -11,8 +11,12 @@
 
 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);