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

Side by Side Diff: chrome/browser/net/spdyproxy/data_reduction_proxy_settings_unittest_android.cc

Issue 59113012: Removed SpdyProxyAuth.State histogram (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed unneeded variables Created 7 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "chrome/browser/net/spdyproxy/data_reduction_proxy_settings_unittest.h" 5 #include "chrome/browser/net/spdyproxy/data_reduction_proxy_settings_unittest.h"
6 6
7 #include "base/android/jni_android.h" 7 #include "base/android/jni_android.h"
8 #include "base/android/jni_string.h" 8 #include "base/android/jni_string.h"
9 #include "base/android/scoped_java_ref.h" 9 #include "base/android/scoped_java_ref.h"
10 #include "base/base64.h" 10 #include "base/base64.h"
(...skipping 74 matching lines...) Expand 10 before | Expand all | Expand 10 after
85 TEST_F(DataReductionProxySettingsAndroidTest, TestSetProxyPac) { 85 TEST_F(DataReductionProxySettingsAndroidTest, TestSetProxyPac) {
86 AddProxyToCommandLine(); 86 AddProxyToCommandLine();
87 Settings()->AddDefaultProxyBypassRules(); 87 Settings()->AddDefaultProxyBypassRules();
88 std::string raw_pac = Settings()->GetProxyPacScript(); 88 std::string raw_pac = Settings()->GetProxyPacScript();
89 EXPECT_NE(raw_pac.find(kDataReductionProxyOriginPAC), std::string::npos); 89 EXPECT_NE(raw_pac.find(kDataReductionProxyOriginPAC), std::string::npos);
90 EXPECT_NE(raw_pac.find(kDataReductionProxyFallbackPAC), std::string::npos);; 90 EXPECT_NE(raw_pac.find(kDataReductionProxyFallbackPAC), std::string::npos);;
91 std::string pac; 91 std::string pac;
92 base::Base64Encode(raw_pac, &pac); 92 base::Base64Encode(raw_pac, &pac);
93 std::string expected_pac_url = 93 std::string expected_pac_url =
94 "data:application/x-ns-proxy-autoconfig;base64," + pac; 94 "data:application/x-ns-proxy-autoconfig;base64," + pac;
95 // Test setting the PAC, without generating histograms.
96 Settings()->SetHasTurnedOn();
97 Settings()->SetProxyConfigs(true, false); 95 Settings()->SetProxyConfigs(true, false);
98 CheckProxyPacPref(expected_pac_url, 96 CheckProxyPacPref(expected_pac_url,
99 ProxyModeToString(ProxyPrefs::MODE_PAC_SCRIPT)); 97 ProxyModeToString(ProxyPrefs::MODE_PAC_SCRIPT));
100 98
101 // Test disabling the PAC, without generating histograms.
102 Settings()->SetHasTurnedOff();
103 Settings()->SetProxyConfigs(false, false); 99 Settings()->SetProxyConfigs(false, false);
104 CheckProxyPacPref(std::string(), ProxyModeToString(ProxyPrefs::MODE_SYSTEM)); 100 CheckProxyPacPref(std::string(), ProxyModeToString(ProxyPrefs::MODE_SYSTEM));
105 } 101 }
106 102
107 TEST_F(DataReductionProxySettingsAndroidTest, TestGetDailyContentLengths) { 103 TEST_F(DataReductionProxySettingsAndroidTest, TestGetDailyContentLengths) {
108 ScopedJavaLocalRef<jlongArray> result = Settings()->GetDailyContentLengths( 104 ScopedJavaLocalRef<jlongArray> result = Settings()->GetDailyContentLengths(
109 env_, prefs::kDailyHttpOriginalContentLength); 105 env_, prefs::kDailyHttpOriginalContentLength);
110 ASSERT_TRUE(result.obj()); 106 ASSERT_TRUE(result.obj());
111 107
112 jsize java_array_len = env_->GetArrayLength(result.obj()); 108 jsize java_array_len = env_->GetArrayLength(result.obj());
113 ASSERT_EQ(static_cast<jsize>(spdyproxy::kNumDaysInHistory), java_array_len); 109 ASSERT_EQ(static_cast<jsize>(spdyproxy::kNumDaysInHistory), java_array_len);
114 110
115 jlong value; 111 jlong value;
116 for (size_t i = 0; i < spdyproxy::kNumDaysInHistory; ++i) { 112 for (size_t i = 0; i < spdyproxy::kNumDaysInHistory; ++i) {
117 env_->GetLongArrayRegion(result.obj(), i, 1, &value); 113 env_->GetLongArrayRegion(result.obj(), i, 1, &value);
118 ASSERT_EQ( 114 ASSERT_EQ(
119 static_cast<long>((spdyproxy::kNumDaysInHistory - 1 - i) * 2), value); 115 static_cast<long>((spdyproxy::kNumDaysInHistory - 1 - i) * 2), value);
120 } 116 }
121 } 117 }
122 118
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698