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

Side by Side Diff: chrome/browser/extensions/extension_proxy_api_helpers_unittest.cc

Issue 7649006: more changes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix another typo Created 9 years, 4 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Unit tests for helper functions for the Chrome Extensions Proxy Settings API. 5 // Unit tests for helper functions for the Chrome Extensions Proxy Settings API.
6 6
7 #include "base/memory/scoped_ptr.h" 7 #include "base/memory/scoped_ptr.h"
8 #include "base/values.h" 8 #include "base/values.h"
9 #include "chrome/browser/extensions/extension_proxy_api_constants.h" 9 #include "chrome/browser/extensions/extension_proxy_api_constants.h"
10 #include "chrome/browser/extensions/extension_proxy_api_helpers.h" 10 #include "chrome/browser/extensions/extension_proxy_api_helpers.h"
(...skipping 177 matching lines...) Expand 10 before | Expand all | Expand 10 after
188 // Currently we are still missing a proxy config entry. 188 // Currently we are still missing a proxy config entry.
189 // This is silently ignored. 189 // This is silently ignored.
190 ASSERT_TRUE( 190 ASSERT_TRUE(
191 GetBypassListFromExtensionPref(&proxy_config, &out, &error, 191 GetBypassListFromExtensionPref(&proxy_config, &out, &error,
192 &bad_message)); 192 &bad_message));
193 EXPECT_EQ(std::string(), out); 193 EXPECT_EQ(std::string(), out);
194 EXPECT_EQ(std::string(), error); 194 EXPECT_EQ(std::string(), error);
195 EXPECT_FALSE(bad_message); 195 EXPECT_FALSE(bad_message);
196 196
197 ListValue* bypass_list = new ListValue; 197 ListValue* bypass_list = new ListValue;
198 bypass_list->Append(Value::CreateStringValue("host1")); 198 bypass_list->Append(base::StringValue::New("host1"));
199 bypass_list->Append(Value::CreateStringValue("host2")); 199 bypass_list->Append(base::StringValue::New("host2"));
200 DictionaryValue* proxy_rules = new DictionaryValue; 200 DictionaryValue* proxy_rules = new DictionaryValue;
201 proxy_rules->Set(keys::kProxyConfigBypassList, bypass_list); 201 proxy_rules->Set(keys::kProxyConfigBypassList, bypass_list);
202 proxy_config.Set(keys::kProxyConfigRules, proxy_rules); 202 proxy_config.Set(keys::kProxyConfigRules, proxy_rules);
203 203
204 ASSERT_TRUE( 204 ASSERT_TRUE(
205 GetBypassListFromExtensionPref(&proxy_config, &out, &error, 205 GetBypassListFromExtensionPref(&proxy_config, &out, &error,
206 &bad_message)); 206 &bad_message));
207 EXPECT_EQ("host1,host2", out); 207 EXPECT_EQ("host1,host2", out);
208 EXPECT_EQ(std::string(), error); 208 EXPECT_EQ(std::string(), error);
209 EXPECT_FALSE(bad_message); 209 EXPECT_FALSE(bad_message);
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
275 ASSERT_TRUE( 275 ASSERT_TRUE(
276 GetProxyServer(&proxy_server_dict, net::ProxyServer::SCHEME_HTTP, 276 GetProxyServer(&proxy_server_dict, net::ProxyServer::SCHEME_HTTP,
277 &created, &error, &bad_message)); 277 &created, &error, &bad_message));
278 EXPECT_EQ("SOCKS proxy_server:1234", created.ToPacString()); 278 EXPECT_EQ("SOCKS proxy_server:1234", created.ToPacString());
279 EXPECT_FALSE(bad_message); 279 EXPECT_FALSE(bad_message);
280 } 280 }
281 281
282 TEST(ExtensionProxyApiHelpers, JoinUrlList) { 282 TEST(ExtensionProxyApiHelpers, JoinUrlList) {
283 bool bad_message = false; 283 bool bad_message = false;
284 ListValue list; 284 ListValue list;
285 list.Append(Value::CreateStringValue("s1")); 285 list.Append(base::StringValue::New("s1"));
286 list.Append(Value::CreateStringValue("s2")); 286 list.Append(base::StringValue::New("s2"));
287 list.Append(Value::CreateStringValue("s3")); 287 list.Append(base::StringValue::New("s3"));
288 288
289 std::string out; 289 std::string out;
290 std::string error; 290 std::string error;
291 ASSERT_TRUE(JoinUrlList(&list, ";", &out, &error, &bad_message)); 291 ASSERT_TRUE(JoinUrlList(&list, ";", &out, &error, &bad_message));
292 EXPECT_EQ("s1;s2;s3", out); 292 EXPECT_EQ("s1;s2;s3", out);
293 EXPECT_FALSE(bad_message); 293 EXPECT_FALSE(bad_message);
294 } 294 }
295 295
296 // This tests CreateProxyServerDict as well. 296 // This tests CreateProxyServerDict as well.
297 TEST(ExtensionProxyApiHelpers, CreateProxyRulesDict) { 297 TEST(ExtensionProxyApiHelpers, CreateProxyRulesDict) {
298 scoped_ptr<DictionaryValue> browser_pref( 298 scoped_ptr<DictionaryValue> browser_pref(
299 ProxyConfigDictionary::CreateFixedServers( 299 ProxyConfigDictionary::CreateFixedServers(
300 "http=proxy1:80;https=proxy2:80;ftp=proxy3:80;socks=proxy4:80", 300 "http=proxy1:80;https=proxy2:80;ftp=proxy3:80;socks=proxy4:80",
301 "localhost")); 301 "localhost"));
302 ProxyConfigDictionary config(browser_pref.get()); 302 ProxyConfigDictionary config(browser_pref.get());
303 scoped_ptr<DictionaryValue> extension_pref(CreateProxyRulesDict(config)); 303 scoped_ptr<DictionaryValue> extension_pref(CreateProxyRulesDict(config));
304 ASSERT_TRUE(extension_pref.get()); 304 ASSERT_TRUE(extension_pref.get());
305 305
306 scoped_ptr<DictionaryValue> expected(new DictionaryValue); 306 scoped_ptr<DictionaryValue> expected(new DictionaryValue);
307 expected->Set("proxyForHttp", 307 expected->Set("proxyForHttp",
308 CreateTestProxyServerDict("http", "proxy1", 80)); 308 CreateTestProxyServerDict("http", "proxy1", 80));
309 expected->Set("proxyForHttps", 309 expected->Set("proxyForHttps",
310 CreateTestProxyServerDict("http", "proxy2", 80)); 310 CreateTestProxyServerDict("http", "proxy2", 80));
311 expected->Set("proxyForFtp", 311 expected->Set("proxyForFtp",
312 CreateTestProxyServerDict("http", "proxy3", 80)); 312 CreateTestProxyServerDict("http", "proxy3", 80));
313 expected->Set("fallbackProxy", 313 expected->Set("fallbackProxy",
314 CreateTestProxyServerDict("socks4", "proxy4", 80)); 314 CreateTestProxyServerDict("socks4", "proxy4", 80));
315 ListValue* bypass_list = new ListValue; 315 ListValue* bypass_list = new ListValue;
316 bypass_list->Append(Value::CreateStringValue("localhost")); 316 bypass_list->Append(base::StringValue::New("localhost"));
317 expected->Set(keys::kProxyConfigBypassList, bypass_list); 317 expected->Set(keys::kProxyConfigBypassList, bypass_list);
318 318
319 EXPECT_TRUE(Value::Equals(expected.get(), extension_pref.get())); 319 EXPECT_TRUE(Value::Equals(expected.get(), extension_pref.get()));
320 } 320 }
321 321
322 // Test if a PAC script URL is specified. 322 // Test if a PAC script URL is specified.
323 TEST(ExtensionProxyApiHelpers, CreatePacScriptDictWithUrl) { 323 TEST(ExtensionProxyApiHelpers, CreatePacScriptDictWithUrl) {
324 scoped_ptr<DictionaryValue> browser_pref( 324 scoped_ptr<DictionaryValue> browser_pref(
325 ProxyConfigDictionary::CreatePacScript(kSamplePacScriptUrl, false)); 325 ProxyConfigDictionary::CreatePacScript(kSamplePacScriptUrl, false));
326 ProxyConfigDictionary config(browser_pref.get()); 326 ProxyConfigDictionary config(browser_pref.get());
(...skipping 17 matching lines...) Expand all
344 344
345 scoped_ptr<DictionaryValue> expected(new DictionaryValue); 345 scoped_ptr<DictionaryValue> expected(new DictionaryValue);
346 expected->SetString(keys::kProxyConfigPacScriptData, kSamplePacScript); 346 expected->SetString(keys::kProxyConfigPacScriptData, kSamplePacScript);
347 expected->SetBoolean(keys::kProxyConfigPacScriptMandatory, false); 347 expected->SetBoolean(keys::kProxyConfigPacScriptMandatory, false);
348 348
349 EXPECT_TRUE(Value::Equals(expected.get(), extension_pref.get())); 349 EXPECT_TRUE(Value::Equals(expected.get(), extension_pref.get()));
350 } 350 }
351 351
352 TEST(ExtensionProxyApiHelpers, TokenizeToStringList) { 352 TEST(ExtensionProxyApiHelpers, TokenizeToStringList) {
353 ListValue expected; 353 ListValue expected;
354 expected.Append(Value::CreateStringValue("s1")); 354 expected.Append(base::StringValue::New("s1"));
355 expected.Append(Value::CreateStringValue("s2")); 355 expected.Append(base::StringValue::New("s2"));
356 expected.Append(Value::CreateStringValue("s3")); 356 expected.Append(base::StringValue::New("s3"));
357 357
358 scoped_ptr<ListValue> out(TokenizeToStringList("s1;s2;s3", ";")); 358 scoped_ptr<ListValue> out(TokenizeToStringList("s1;s2;s3", ";"));
359 EXPECT_TRUE(Value::Equals(&expected, out.get())); 359 EXPECT_TRUE(Value::Equals(&expected, out.get()));
360 } 360 }
361 361
362 } // namespace extension_proxy_api_helpers 362 } // namespace extension_proxy_api_helpers
OLDNEW
« no previous file with comments | « chrome/browser/extensions/extension_proxy_api_helpers.cc ('k') | chrome/browser/extensions/extension_rlz_module.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698