OLD | NEW |
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 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 #include <vector> | 5 #include <vector> |
6 | 6 |
7 #include "base/bind.h" | 7 #include "base/bind.h" |
8 #include "base/memory/weak_ptr.h" | 8 #include "base/memory/weak_ptr.h" |
9 #include "base/message_loop/message_loop.h" | 9 #include "base/message_loop/message_loop.h" |
10 #include "base/run_loop.h" | 10 #include "base/run_loop.h" |
(...skipping 26 matching lines...) Expand all Loading... |
37 public: | 37 public: |
38 struct Rule { | 38 struct Rule { |
39 Rule(const GURL& url, int fetch_error, bool is_valid_script) | 39 Rule(const GURL& url, int fetch_error, bool is_valid_script) |
40 : url(url), | 40 : url(url), |
41 fetch_error(fetch_error), | 41 fetch_error(fetch_error), |
42 is_valid_script(is_valid_script) { | 42 is_valid_script(is_valid_script) { |
43 } | 43 } |
44 | 44 |
45 base::string16 text() const { | 45 base::string16 text() const { |
46 if (is_valid_script) | 46 if (is_valid_script) |
47 return UTF8ToUTF16(url.spec() + "!FindProxyForURL"); | 47 return base::UTF8ToUTF16(url.spec() + "!FindProxyForURL"); |
48 if (fetch_error == OK) | 48 if (fetch_error == OK) |
49 return UTF8ToUTF16(url.spec() + "!invalid-script"); | 49 return base::UTF8ToUTF16(url.spec() + "!invalid-script"); |
50 return base::string16(); | 50 return base::string16(); |
51 } | 51 } |
52 | 52 |
53 GURL url; | 53 GURL url; |
54 int fetch_error; | 54 int fetch_error; |
55 bool is_valid_script; | 55 bool is_valid_script; |
56 }; | 56 }; |
57 | 57 |
58 Rule AddSuccessRule(const char* url) { | 58 Rule AddSuccessRule(const char* url) { |
59 Rule rule(GURL(url), OK /*fetch_error*/, true); | 59 Rule rule(GURL(url), OK /*fetch_error*/, true); |
(...skipping 608 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
668 | 668 |
669 // All of the tests above that use ProxyScriptDecider have tested | 669 // All of the tests above that use ProxyScriptDecider have tested |
670 // failure to fetch a PAC file via DHCP configuration, so we now test | 670 // failure to fetch a PAC file via DHCP configuration, so we now test |
671 // success at downloading and parsing, and then success at downloading, | 671 // success at downloading and parsing, and then success at downloading, |
672 // failure at parsing. | 672 // failure at parsing. |
673 | 673 |
674 TEST(ProxyScriptDeciderTest, AutodetectDhcpSuccess) { | 674 TEST(ProxyScriptDeciderTest, AutodetectDhcpSuccess) { |
675 Rules rules; | 675 Rules rules; |
676 RuleBasedProxyScriptFetcher fetcher(&rules); | 676 RuleBasedProxyScriptFetcher fetcher(&rules); |
677 SynchronousSuccessDhcpFetcher dhcp_fetcher( | 677 SynchronousSuccessDhcpFetcher dhcp_fetcher( |
678 WideToUTF16(L"http://bingo/!FindProxyForURL")); | 678 base::WideToUTF16(L"http://bingo/!FindProxyForURL")); |
679 | 679 |
680 ProxyConfig config; | 680 ProxyConfig config; |
681 config.set_auto_detect(true); | 681 config.set_auto_detect(true); |
682 | 682 |
683 rules.AddSuccessRule("http://bingo/"); | 683 rules.AddSuccessRule("http://bingo/"); |
684 rules.AddFailDownloadRule("http://wpad/wpad.dat"); | 684 rules.AddFailDownloadRule("http://wpad/wpad.dat"); |
685 | 685 |
686 TestCompletionCallback callback; | 686 TestCompletionCallback callback; |
687 ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, NULL); | 687 ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, NULL); |
688 EXPECT_EQ(OK, decider.Start( | 688 EXPECT_EQ(OK, decider.Start( |
689 config, base::TimeDelta(), true, callback.callback())); | 689 config, base::TimeDelta(), true, callback.callback())); |
690 EXPECT_EQ(dhcp_fetcher.expected_text(), | 690 EXPECT_EQ(dhcp_fetcher.expected_text(), |
691 decider.script_data()->utf16()); | 691 decider.script_data()->utf16()); |
692 | 692 |
693 EXPECT_TRUE(decider.effective_config().has_pac_url()); | 693 EXPECT_TRUE(decider.effective_config().has_pac_url()); |
694 EXPECT_EQ(GURL("http://dhcppac/"), decider.effective_config().pac_url()); | 694 EXPECT_EQ(GURL("http://dhcppac/"), decider.effective_config().pac_url()); |
695 } | 695 } |
696 | 696 |
697 TEST(ProxyScriptDeciderTest, AutodetectDhcpFailParse) { | 697 TEST(ProxyScriptDeciderTest, AutodetectDhcpFailParse) { |
698 Rules rules; | 698 Rules rules; |
699 RuleBasedProxyScriptFetcher fetcher(&rules); | 699 RuleBasedProxyScriptFetcher fetcher(&rules); |
700 SynchronousSuccessDhcpFetcher dhcp_fetcher( | 700 SynchronousSuccessDhcpFetcher dhcp_fetcher( |
701 WideToUTF16(L"http://bingo/!invalid-script")); | 701 base::WideToUTF16(L"http://bingo/!invalid-script")); |
702 | 702 |
703 ProxyConfig config; | 703 ProxyConfig config; |
704 config.set_auto_detect(true); | 704 config.set_auto_detect(true); |
705 | 705 |
706 rules.AddFailParsingRule("http://bingo/"); | 706 rules.AddFailParsingRule("http://bingo/"); |
707 rules.AddFailDownloadRule("http://wpad/wpad.dat"); | 707 rules.AddFailDownloadRule("http://wpad/wpad.dat"); |
708 | 708 |
709 TestCompletionCallback callback; | 709 TestCompletionCallback callback; |
710 ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, NULL); | 710 ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, NULL); |
711 // Since there is fallback to DNS-based WPAD, the final error will be that | 711 // Since there is fallback to DNS-based WPAD, the final error will be that |
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
775 | 775 |
776 // Run the message loop to let the DHCP fetch complete and post the results | 776 // Run the message loop to let the DHCP fetch complete and post the results |
777 // back. Before the fix linked to above, this would try to invoke on | 777 // back. Before the fix linked to above, this would try to invoke on |
778 // the callback object provided by ProxyScriptDecider after it was | 778 // the callback object provided by ProxyScriptDecider after it was |
779 // no longer valid. | 779 // no longer valid. |
780 base::MessageLoop::current()->RunUntilIdle(); | 780 base::MessageLoop::current()->RunUntilIdle(); |
781 } | 781 } |
782 | 782 |
783 } // namespace | 783 } // namespace |
784 } // namespace net | 784 } // namespace net |
OLD | NEW |