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

Unified Diff: components/data_reduction_proxy/browser/data_reduction_proxy_settings_unittest.cc

Issue 538243003: proposed interface change at //net for webrtc (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Split address_tracker_linux into a different CL. Created 6 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: components/data_reduction_proxy/browser/data_reduction_proxy_settings_unittest.cc
diff --git a/components/data_reduction_proxy/browser/data_reduction_proxy_settings_unittest.cc b/components/data_reduction_proxy/browser/data_reduction_proxy_settings_unittest.cc
index d40cadccb9521d4f4670db58db7aa071d24206d2..f45e529aeb96ccce7e6c9beb2086b60c540089f8 100644
--- a/components/data_reduction_proxy/browser/data_reduction_proxy_settings_unittest.cc
+++ b/components/data_reduction_proxy/browser/data_reduction_proxy_settings_unittest.cc
@@ -312,28 +312,30 @@ TEST_F(DataReductionProxySettingsTest, TestOnIPAddressChanged) {
// Simulate a VPN connection. The proxy should be disabled.
MockSettings* settings = static_cast<MockSettings*>(settings_.get());
settings->network_interfaces_.reset(new net::NetworkInterfaceList());
- settings->network_interfaces_->push_back(
- net::NetworkInterface("tun0", /* network interface name */
- "tun0", /* network interface friendly name */
- 0, /* interface index */
- net::NetworkChangeNotifier::CONNECTION_WIFI,
- net::IPAddressNumber(), /* IP address */
- 0 /* network prefix */
- ));
+ settings->network_interfaces_->push_back(net::NetworkInterface(
+ "tun0", /* network interface name */
+ "tun0", /* network interface friendly name */
+ 0, /* interface index */
+ net::NetworkChangeNotifier::CONNECTION_WIFI,
+ net::IPAddressNumber(), /* IP address */
+ 0, /* network prefix */
+ net::IP_ADDRESS_ATTRIBUTE_NONE /* ip address attribute */
+ ));
settings_->OnIPAddressChanged();
base::MessageLoop::current()->RunUntilIdle();
CheckProxyConfigs(false, false, false);
// Check that the proxy is re-enabled if a non-VPN connection is later used.
settings->network_interfaces_.reset(new net::NetworkInterfaceList());
- settings->network_interfaces_->push_back(
- net::NetworkInterface("eth0", /* network interface name */
- "eth0", /* network interface friendly name */
- 0, /* interface index */
- net::NetworkChangeNotifier::CONNECTION_WIFI,
- net::IPAddressNumber(),
- 0 /* network prefix */
- ));
+ settings->network_interfaces_->push_back(net::NetworkInterface(
+ "eth0", /* network interface name */
+ "eth0", /* network interface friendly name */
+ 0, /* interface index */
+ net::NetworkChangeNotifier::CONNECTION_WIFI,
+ net::IPAddressNumber(),
+ 0, /* network prefix */
+ net::IP_ADDRESS_ATTRIBUTE_NONE /* ip address attribute */
+ ));
CheckProbeOnIPChange(kProbeURLWithOKResponse,
"OK",
true,

Powered by Google App Engine
This is Rietveld 408576698