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

Side by Side Diff: components/ssl_errors/error_classification_unittest.cc

Issue 2446273004: Reduce buggy usage of the registry controlled domain service. (Closed)
Patch Set: Review comment Created 4 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
« no previous file with comments | « components/ssl_errors/error_classification.cc ('k') | components/url_formatter/url_fixer.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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/ssl_errors/error_classification.h" 5 #include "components/ssl_errors/error_classification.h"
6 6
7 #include "base/files/file_path.h" 7 #include "base/files/file_path.h"
8 #include "base/memory/ptr_util.h" 8 #include "base/memory/ptr_util.h"
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 #include "base/strings/string_split.h" 10 #include "base/strings/string_split.h"
(...skipping 158 matching lines...) Expand 10 before | Expand all | Expand 10 after
169 EXPECT_FALSE(ssl_errors::AnyNamesUnderName(dns_name_tokens_webkit, 169 EXPECT_FALSE(ssl_errors::AnyNamesUnderName(dns_name_tokens_webkit,
170 host_name_tokens)); 170 host_name_tokens));
171 EXPECT_TRUE(ssl_errors::IsSubDomainOutsideWildcard(origin, *webkit_cert)); 171 EXPECT_TRUE(ssl_errors::IsSubDomainOutsideWildcard(origin, *webkit_cert));
172 EXPECT_FALSE( 172 EXPECT_FALSE(
173 ssl_errors::IsCertLikelyFromMultiTenantHosting(origin, *webkit_cert)); 173 ssl_errors::IsCertLikelyFromMultiTenantHosting(origin, *webkit_cert));
174 EXPECT_TRUE(ssl_errors::IsCertLikelyFromSameDomain(origin, *webkit_cert)); 174 EXPECT_TRUE(ssl_errors::IsCertLikelyFromSameDomain(origin, *webkit_cert));
175 } 175 }
176 } 176 }
177 177
178 TEST_F(SSLErrorClassificationTest, TestHostNameHasKnownTLD) { 178 TEST_F(SSLErrorClassificationTest, TestHostNameHasKnownTLD) {
179 EXPECT_TRUE(ssl_errors::IsHostNameKnownTLD("www.google.com")); 179 EXPECT_TRUE(ssl_errors::HostNameHasKnownTLD("www.google.com"));
180 EXPECT_TRUE(ssl_errors::IsHostNameKnownTLD("b.appspot.com")); 180 EXPECT_TRUE(ssl_errors::HostNameHasKnownTLD("b.appspot.com"));
181 EXPECT_FALSE(ssl_errors::IsHostNameKnownTLD("a.private")); 181 EXPECT_FALSE(ssl_errors::HostNameHasKnownTLD("a.private"));
182 } 182 }
183 183
184 TEST_F(SSLErrorClassificationTest, TestPrivateURL) { 184 TEST_F(SSLErrorClassificationTest, TestPrivateURL) {
185 EXPECT_FALSE(ssl_errors::IsHostnameNonUniqueOrDotless("www.foogoogle.com.")); 185 EXPECT_FALSE(ssl_errors::IsHostnameNonUniqueOrDotless("www.foogoogle.com."));
186 EXPECT_TRUE(ssl_errors::IsHostnameNonUniqueOrDotless("go")); 186 EXPECT_TRUE(ssl_errors::IsHostnameNonUniqueOrDotless("go"));
187 EXPECT_TRUE(ssl_errors::IsHostnameNonUniqueOrDotless("172.17.108.108")); 187 EXPECT_TRUE(ssl_errors::IsHostnameNonUniqueOrDotless("172.17.108.108"));
188 EXPECT_TRUE(ssl_errors::IsHostnameNonUniqueOrDotless("foo.blah")); 188 EXPECT_TRUE(ssl_errors::IsHostnameNonUniqueOrDotless("foo.blah"));
189 } 189 }
190 190
191 TEST(ErrorClassification, LevenshteinDistance) { 191 TEST(ErrorClassification, LevenshteinDistance) {
(...skipping 250 matching lines...) Expand 10 before | Expand all | Expand 10 after
442 clock->Advance(base::TimeDelta::FromDays(1)); 442 clock->Advance(base::TimeDelta::FromDays(1));
443 // GetClockState() will fall back to the build time heuristic. 443 // GetClockState() will fall back to the build time heuristic.
444 ssl_errors::GetClockState(clock->Now(), &network_time_tracker); 444 ssl_errors::GetClockState(clock->Now(), &network_time_tracker);
445 histograms.ExpectTotalCount(kNetworkTimeHistogram, 8); 445 histograms.ExpectTotalCount(kNetworkTimeHistogram, 8);
446 histograms.ExpectBucketCount( 446 histograms.ExpectBucketCount(
447 kNetworkTimeHistogram, ssl_errors::NETWORK_CLOCK_STATE_UNKNOWN_SYNC_LOST, 447 kNetworkTimeHistogram, ssl_errors::NETWORK_CLOCK_STATE_UNKNOWN_SYNC_LOST,
448 1); 448 1);
449 449
450 io_thread.Stop(); 450 io_thread.Stop();
451 } 451 }
OLDNEW
« no previous file with comments | « components/ssl_errors/error_classification.cc ('k') | components/url_formatter/url_fixer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698