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

Side by Side Diff: components/data_reduction_proxy/browser/data_reduction_proxy_params_unittest.cc

Issue 464023002: Fixed DataReductionProxyParams::AreProxiesBypassed logic (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed bengr comments Created 6 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "components/data_reduction_proxy/browser/data_reduction_proxy_params.h" 5 #include "components/data_reduction_proxy/browser/data_reduction_proxy_params.h"
6 6
7 #include <map> 7 #include <map>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 750 matching lines...) Expand 10 before | Expand all | Expand 10 after
761 // expected result 761 // expected result
762 true, 762 true,
763 }, 763 },
764 { // proxy flags 764 { // proxy flags
765 true, 765 true,
766 true, 766 true,
767 false, 767 false,
768 // is https request 768 // is https request
769 false, 769 false,
770 // proxies in retry map 770 // proxies in retry map
771 false,
772 true,
773 false,
774 false,
775 false,
776 // expected result
777 true,
778 },
779 { // proxy flags
780 false,
781 true,
782 true,
783 // is https request
784 false,
785 // proxies in retry map
786 false,
787 false,
788 false,
789 true,
790 false,
791 // expected result
792 true,
793 },
794 { // proxy flags
795 true,
796 true,
797 false,
798 // is https request
799 false,
800 // proxies in retry map
771 true, 801 true,
772 false, 802 false,
773 false, 803 false,
774 false, 804 false,
775 false, 805 false,
776 // expected result 806 // expected result
777 false, 807 false,
778 }, 808 },
779 { // proxy flags 809 { // proxy flags
780 true, 810 true,
(...skipping 187 matching lines...) Expand 10 before | Expand all | Expand 10 after
968 retry_map[ssl_origin] = retry_info; 998 retry_map[ssl_origin] = retry_info;
969 999
970 bool was_bypassed = params.AreProxiesBypassed(retry_map, 1000 bool was_bypassed = params.AreProxiesBypassed(retry_map,
971 tests[i].is_https, 1001 tests[i].is_https,
972 NULL); 1002 NULL);
973 1003
974 EXPECT_EQ(tests[i].expected_result, was_bypassed); 1004 EXPECT_EQ(tests[i].expected_result, was_bypassed);
975 } 1005 }
976 } 1006 }
977 } // namespace data_reduction_proxy 1007 } // namespace data_reduction_proxy
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698