diff options
author | dcheng@chromium.org <dcheng@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-04-09 06:41:12 +0000 |
---|---|---|
committer | dcheng@chromium.org <dcheng@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-04-09 06:41:12 +0000 |
commit | 69d7f89d6a93e0fbce451960287066195c6d4ea0 (patch) | |
tree | 51c8c1f971d39d652ec38ed76070539bb33e85c7 /net/proxy | |
parent | e3748a79b523a8d365d4a33ef986eebb4186fa78 (diff) | |
download | chromium_src-69d7f89d6a93e0fbce451960287066195c6d4ea0.zip chromium_src-69d7f89d6a93e0fbce451960287066195c6d4ea0.tar.gz chromium_src-69d7f89d6a93e0fbce451960287066195c6d4ea0.tar.bz2 |
Revert "Rewrite std::string("") to std::string(), Linux edition."
This reverts commit e59558b78e8c6a1b0bd916a724724b638c3c91b6.
Revert "Fix build after r193020."
This reverts commit 558a35897f6b3ffbcaefde927c1f150b815d140a.
Revert "Really fix build after r193020."
This reverts commit e3748a79b523a8d365d4a33ef986eebb4186fa78.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193030 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/proxy')
-rw-r--r-- | net/proxy/dhcp_proxy_script_fetcher.cc | 2 | ||||
-rw-r--r-- | net/proxy/proxy_resolver_v8_tracing_unittest.cc | 6 | ||||
-rw-r--r-- | net/proxy/proxy_service_unittest.cc | 12 |
3 files changed, 9 insertions, 11 deletions
diff --git a/net/proxy/dhcp_proxy_script_fetcher.cc b/net/proxy/dhcp_proxy_script_fetcher.cc index 1771be0..4771d9d 100644 --- a/net/proxy/dhcp_proxy_script_fetcher.cc +++ b/net/proxy/dhcp_proxy_script_fetcher.cc @@ -9,7 +9,7 @@ namespace net { std::string DhcpProxyScriptFetcher::GetFetcherName() const { - return std::string(); + return ""; } DhcpProxyScriptFetcher::DhcpProxyScriptFetcher() {} diff --git a/net/proxy/proxy_resolver_v8_tracing_unittest.cc b/net/proxy/proxy_resolver_v8_tracing_unittest.cc index 5e0e108..b5503e5 100644 --- a/net/proxy/proxy_resolver_v8_tracing_unittest.cc +++ b/net/proxy/proxy_resolver_v8_tracing_unittest.cc @@ -282,8 +282,7 @@ TEST_F(ProxyResolverV8TracingTest, Dns) { host_resolver.rules()->AddRuleForAddressFamily( "host1", ADDRESS_FAMILY_IPV4, "166.155.144.44"); - host_resolver.rules() - ->AddIPLiteralRule("host1", "::1,192.168.1.1", std::string()); + host_resolver.rules()->AddIPLiteralRule("host1", "::1,192.168.1.1", ""); host_resolver.rules()->AddSimulatedFailure("host2"); host_resolver.rules()->AddRule("host3", "166.155.144.33"); host_resolver.rules()->AddRule("host5", "166.155.144.55"); @@ -996,8 +995,7 @@ TEST_F(ProxyResolverV8TracingTest, MultipleResolvers) { MockHostResolver host_resolver0; host_resolver0.rules()->AddRuleForAddressFamily( "host1", ADDRESS_FAMILY_IPV4, "166.155.144.44"); - host_resolver0.rules() - ->AddIPLiteralRule("host1", "::1,192.168.1.1", std::string()); + host_resolver0.rules()->AddIPLiteralRule("host1", "::1,192.168.1.1", ""); host_resolver0.rules()->AddSimulatedFailure("host2"); host_resolver0.rules()->AddRule("host3", "166.155.144.33"); host_resolver0.rules()->AddRule("host5", "166.155.144.55"); diff --git a/net/proxy/proxy_service_unittest.cc b/net/proxy/proxy_service_unittest.cc index 53cc326..ec4a0fa 100644 --- a/net/proxy/proxy_service_unittest.cc +++ b/net/proxy/proxy_service_unittest.cc @@ -1723,7 +1723,7 @@ TEST_F(ProxyServiceTest, FallbackFromAutodetectToCustomPac) { // the script download. EXPECT_TRUE(fetcher->has_pending_request()); EXPECT_EQ(GURL("http://wpad/wpad.dat"), fetcher->pending_request_url()); - fetcher->NotifyFetchCompletion(ERR_FAILED, std::string()); + fetcher->NotifyFetchCompletion(ERR_FAILED, ""); // Next it should be trying the custom PAC url. EXPECT_TRUE(fetcher->has_pending_request()); @@ -1872,12 +1872,12 @@ TEST_F(ProxyServiceTest, FallbackFromAutodetectToCustomToManual) { // It should be trying to auto-detect first -- fail the download. EXPECT_TRUE(fetcher->has_pending_request()); EXPECT_EQ(GURL("http://wpad/wpad.dat"), fetcher->pending_request_url()); - fetcher->NotifyFetchCompletion(ERR_FAILED, std::string()); + fetcher->NotifyFetchCompletion(ERR_FAILED, ""); // Next it should be trying the custom PAC url -- fail the download. EXPECT_TRUE(fetcher->has_pending_request()); EXPECT_EQ(GURL("http://foopy/proxy.pac"), fetcher->pending_request_url()); - fetcher->NotifyFetchCompletion(ERR_FAILED, std::string()); + fetcher->NotifyFetchCompletion(ERR_FAILED, ""); // Since we never managed to initialize a ProxyResolver, nothing should have // been sent to it. @@ -2251,7 +2251,7 @@ TEST_F(ProxyServiceTest, PACScriptRefetchAfterFailure) { // // We simulate a failed download attempt, the proxy service should now // fall-back to DIRECT connections. - fetcher->NotifyFetchCompletion(ERR_FAILED, std::string()); + fetcher->NotifyFetchCompletion(ERR_FAILED, ""); ASSERT_TRUE(resolver->pending_requests().empty()); @@ -2604,7 +2604,7 @@ TEST_F(ProxyServiceTest, PACScriptRefetchAfterSuccess) { // to download the script. EXPECT_TRUE(fetcher->has_pending_request()); EXPECT_EQ(GURL("http://foopy/proxy.pac"), fetcher->pending_request_url()); - fetcher->NotifyFetchCompletion(ERR_FAILED, std::string()); + fetcher->NotifyFetchCompletion(ERR_FAILED, ""); MessageLoop::current()->RunUntilIdle(); @@ -2771,7 +2771,7 @@ TEST_F(ProxyServiceTest, PACScriptRefetchAfterActivity) { EXPECT_EQ(GURL("http://foopy/proxy.pac"), fetcher->pending_request_url()); // This time we will fail the download, to simulate a PAC script change. - fetcher->NotifyFetchCompletion(ERR_FAILED, std::string()); + fetcher->NotifyFetchCompletion(ERR_FAILED, ""); // Drain the message loop, so ProxyService is notified of the change // and has a chance to re-configure itself. |