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

Unified Diff: chrome/browser/chromeos/status/network_menu_button.cc

Issue 7778043: [cros] Support global carrier config. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixes Created 9 years, 3 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/chromeos/status/network_menu_button.cc
diff --git a/chrome/browser/chromeos/status/network_menu_button.cc b/chrome/browser/chromeos/status/network_menu_button.cc
index 5d34934888a8c83e6cbafe33186964dabf4110ad..0be21128a293abace754d7aa5464f5df35196dd2 100644
--- a/chrome/browser/chromeos/status/network_menu_button.cc
+++ b/chrome/browser/chromeos/status/network_menu_button.cc
@@ -277,8 +277,7 @@ void NetworkMenuButton::OnLinkActivated(size_t index) {
////////////////////////////////////////////////////////////////////////////////
// NetworkMenuButton, private methods:
-const ServicesCustomizationDocument::CarrierDeal*
-NetworkMenuButton::GetCarrierDeal(
+const MobileConfig::Carrier* NetworkMenuButton::GetCarrier(
NetworkLibrary* cros) {
std::string carrier_id = cros->GetCellularHomeCarrierId();
if (carrier_id.empty()) {
@@ -286,13 +285,16 @@ NetworkMenuButton::GetCarrierDeal(
return NULL;
}
- ServicesCustomizationDocument* customization =
- ServicesCustomizationDocument::GetInstance();
- if (!customization->IsReady())
+ MobileConfig* config = MobileConfig::GetInstance();
+ if (!config->IsReady())
return NULL;
- const ServicesCustomizationDocument::CarrierDeal* deal =
- customization->GetCarrierDeal(carrier_id, true);
+ return config->GetCarrier(carrier_id);
+}
+
+const MobileConfig::CarrierDeal* NetworkMenuButton::GetCarrierDeal(
+ const MobileConfig::Carrier* carrier) {
+ const MobileConfig::CarrierDeal* deal = carrier->GetDefaultDeal();
if (deal) {
// Check deal for validity.
int carrier_deal_promo_pref = GetCarrierDealPromoShown();
@@ -354,16 +356,16 @@ void NetworkMenuButton::ShowOptionalMobileDataPromoNotification(
check_for_promo_ && BrowserList::GetLastActive() &&
cros->cellular_connected() && !cros->ethernet_connected() &&
!cros->wifi_connected()) {
- const ServicesCustomizationDocument::CarrierDeal* deal =
- GetCarrierDeal(cros);
+ const MobileConfig::Carrier* carrier = GetCarrier(cros);
std::string deal_text;
int carrier_deal_promo_pref = -1;
- if (deal) {
+ if (carrier) {
+ const MobileConfig::CarrierDeal* deal = GetCarrierDeal(carrier);
carrier_deal_promo_pref = GetCarrierDealPromoShown();
const std::string locale = g_browser_process->GetApplicationLocale();
deal_text = deal->GetLocalizedString(locale, "notification_text");
deal_info_url_ = deal->info_url();
- deal_topup_url_ = deal->top_up_url();
+ deal_topup_url_ = carrier->top_up_url();
} else if (!ShouldShow3gPromoNotification()) {
check_for_promo_ = false;
return;

Powered by Google App Engine
This is Rietveld 408576698