Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1911)

Unified Diff: chrome/browser/ui/webui/help/help_handler.cc

Issue 2897773002: Show proper message in about Chrome OS page (Closed)
Patch Set: Apply fix to patch set 3 Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/help/help_handler.cc
diff --git a/chrome/browser/ui/webui/help/help_handler.cc b/chrome/browser/ui/webui/help/help_handler.cc
index d0a18970628713416f79d8221903fc78c40b4e44..6f729d1651b14c87d32dfb19c9db0322a699540d 100644
--- a/chrome/browser/ui/webui/help/help_handler.cc
+++ b/chrome/browser/ui/webui/help/help_handler.cc
@@ -98,7 +98,10 @@ struct RegulatoryLabel {
// Returns message that informs user that for update it's better to
// connect to a network of one of the allowed types.
base::string16 GetAllowedConnectionTypesMessage() {
- if (help_utils_chromeos::IsUpdateOverCellularAllowed()) {
+ // Old help page does not support interactive-updates over cellular, so just
+ // sets |interactive| to false to make its behavior the same as before.
+ if (help_utils_chromeos::IsUpdateOverCellularAllowed(
+ false /* interactive */)) {
return l10n_util::GetStringUTF16(IDS_UPGRADE_NETWORK_LIST_CELLULAR_ALLOWED);
} else {
return l10n_util::GetStringUTF16(
« no previous file with comments | « chrome/browser/resources/settings/about_page/about_page.js ('k') | chrome/browser/ui/webui/help/help_utils_chromeos.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698