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

Unified Diff: chrome/browser/chromeos/settings/device_settings_provider.cc

Issue 23712002: Cleanup network type matching. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed unit test in Debug. Created 7 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/settings/device_settings_provider.cc
diff --git a/chrome/browser/chromeos/settings/device_settings_provider.cc b/chrome/browser/chromeos/settings/device_settings_provider.cc
index 547854b912ebb4fe6b7e2b5bfb128e54fc529b1e..f2824d2fa271fd96a3fd8fd84e24d5c348137900 100644
--- a/chrome/browser/chromeos/settings/device_settings_provider.cc
+++ b/chrome/browser/chromeos/settings/device_settings_provider.cc
@@ -30,6 +30,7 @@
#include "chromeos/network/network_event_log.h"
#include "chromeos/network/network_handler.h"
#include "chromeos/network/network_state_handler.h"
+#include "chromeos/network/shill_property_util.h"
#include "third_party/cros_system_api/dbus/service_constants.h"
using google::protobuf::RepeatedField;
@@ -825,8 +826,8 @@ void DeviceSettingsProvider::ApplyMetricsSetting(bool use_file,
void DeviceSettingsProvider::ApplyRoamingSetting(bool new_value) {
// TODO(armansito): Look up the device by explicitly using the device path.
const DeviceState* cellular =
- NetworkHandler::Get()->network_state_handler()->
- GetDeviceStateByType(flimflam::kTypeCellular);
+ NetworkHandler::Get()->network_state_handler()->GetDeviceStateByType(
+ NetworkTypePattern::Cellular());
if (!cellular) {
NET_LOG_DEBUG("No cellular device is available",
"Roaming is only supported by cellular devices.");
« no previous file with comments | « chrome/browser/chromeos/prerender_condition_network.cc ('k') | chrome/browser/chromeos/status/network_menu.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698