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

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

Issue 1690593002: Reduce memory use of GetLevenshteinDistance() by almost 50% (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: compile? Created 4 years, 10 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 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/strings/string_split.h" 8 #include "base/strings/string_split.h"
9 #include "net/base/net_errors.h" 9 #include "net/base/net_errors.h"
10 #include "net/base/test_data_directory.h" 10 #include "net/base/test_data_directory.h"
(...skipping 135 matching lines...) Expand 10 before | Expand all | Expand 10 after
146 EXPECT_TRUE(ssl_errors::IsHostNameKnownTLD("b.appspot.com")); 146 EXPECT_TRUE(ssl_errors::IsHostNameKnownTLD("b.appspot.com"));
147 EXPECT_FALSE(ssl_errors::IsHostNameKnownTLD("a.private")); 147 EXPECT_FALSE(ssl_errors::IsHostNameKnownTLD("a.private"));
148 } 148 }
149 149
150 TEST_F(SSLErrorClassificationTest, TestPrivateURL) { 150 TEST_F(SSLErrorClassificationTest, TestPrivateURL) {
151 EXPECT_FALSE(ssl_errors::IsHostnameNonUniqueOrDotless("www.foogoogle.com.")); 151 EXPECT_FALSE(ssl_errors::IsHostnameNonUniqueOrDotless("www.foogoogle.com."));
152 EXPECT_TRUE(ssl_errors::IsHostnameNonUniqueOrDotless("go")); 152 EXPECT_TRUE(ssl_errors::IsHostnameNonUniqueOrDotless("go"));
153 EXPECT_TRUE(ssl_errors::IsHostnameNonUniqueOrDotless("172.17.108.108")); 153 EXPECT_TRUE(ssl_errors::IsHostnameNonUniqueOrDotless("172.17.108.108"));
154 EXPECT_TRUE(ssl_errors::IsHostnameNonUniqueOrDotless("foo.blah")); 154 EXPECT_TRUE(ssl_errors::IsHostnameNonUniqueOrDotless("foo.blah"));
155 } 155 }
156
157 TEST(ErrorClassification, LevenshteinDistance) {
158 EXPECT_EQ(0u, GetLevenshteinDistance("banana", "banana"));
159
160 EXPECT_EQ(2u, GetLevenshteinDistance("ab", "ba"));
161 EXPECT_EQ(2u, GetLevenshteinDistance("ba", "ab"));
162
163 EXPECT_EQ(2u, GetLevenshteinDistance("ananas", "banana"));
164 EXPECT_EQ(2u, GetLevenshteinDistance("banana", "ananas"));
165
166 EXPECT_EQ(2u, GetLevenshteinDistance("unclear", "nuclear"));
167 EXPECT_EQ(2u, GetLevenshteinDistance("nuclear", "unclear"));
168
169 EXPECT_EQ(3u, GetLevenshteinDistance("chrome", "chromium"));
170 EXPECT_EQ(3u, GetLevenshteinDistance("chromium", "chrome"));
171
172 EXPECT_EQ(4u, GetLevenshteinDistance("", "abcd"));
173 EXPECT_EQ(4u, GetLevenshteinDistance("abcd", ""));
174
175 EXPECT_EQ(4u, GetLevenshteinDistance("xxx", "xxxxxxx"));
176 EXPECT_EQ(4u, GetLevenshteinDistance("xxxxxxx", "xxx"));
177
178 EXPECT_EQ(7u, GetLevenshteinDistance("yyy", "xxxxxxx"));
179 EXPECT_EQ(7u, GetLevenshteinDistance("xxxxxxx", "yyy"));
180 }
OLDNEW
« components/ssl_errors/error_classification.h ('K') | « components/ssl_errors/error_classification.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698