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

Unified Diff: chrome_proxy_resolver_unittest.cc

Issue 5205002: AU: Manual proxy support (Closed) Base URL: http://git.chromium.org/git/update_engine.git@master
Patch Set: missed one fix for review Created 10 years, 1 month 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
« no previous file with comments | « chrome_proxy_resolver.cc ('k') | download_action_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome_proxy_resolver_unittest.cc
diff --git a/chrome_proxy_resolver_unittest.cc b/chrome_proxy_resolver_unittest.cc
index 66b2401b0e269e6487e968505085ed9a60f0b102..1cdd5bf84bfa33bd465a789ea92d6bcb4602617e 100644
--- a/chrome_proxy_resolver_unittest.cc
+++ b/chrome_proxy_resolver_unittest.cc
@@ -2,16 +2,16 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
+#include <deque>
#include <string>
-#include <vector>
#include <gtest/gtest.h>
#include "update_engine/chrome_proxy_resolver.h"
#include "update_engine/mock_dbus_interface.h"
+using std::deque;
using std::string;
-using std::vector;
using ::testing::_;
using ::testing::Return;
using ::testing::SetArgumentPointee;
@@ -38,7 +38,7 @@ TEST(ChromeProxyResolverTest, GetProxiesForUrlWithSettingsTest) {
"\"mode\":4}";
ChromeProxyResolver resolver(NULL);
- vector<string> out;
+ deque<string> out;
string urls[] = {"http://foo.com/update", "https://bar.com/foo.gz"};
string multi_settings[] = {kAll, kHttpHttps};
for (size_t i = 0; i < arraysize(urls); i++) {
@@ -119,7 +119,7 @@ TEST(ChromeProxyResolverTest, DbusInterfaceTest) {
SetArgumentPointee<9>(ret_array),
Return(TRUE)));
- vector<string> proxies;
+ deque<string> proxies;
EXPECT_TRUE(resolver.GetProxiesForUrl("http://user:pass@foo.com:22",
&proxies));
EXPECT_EQ(2, proxies.size());
« no previous file with comments | « chrome_proxy_resolver.cc ('k') | download_action_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698