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

Unified Diff: chrome/browser/safe_browsing/path_sanitizer_unittest.cc

Issue 1420053005: Move code in components/safe_browsing_db and chrome/browser/s_b/ under the safe_browsing namespace. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@02_components_move
Patch Set: Remove '// namespace safe_browsing' for a small fwd decl block. Created 5 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/safe_browsing/path_sanitizer_unittest.cc
diff --git a/chrome/browser/safe_browsing/path_sanitizer_unittest.cc b/chrome/browser/safe_browsing/path_sanitizer_unittest.cc
index a23ae6b1b0d2c51e7dce432d47cd8fc946d529ea..cdfb093a1955b9b07d4926a094d4fcbbe340e4a4 100644
--- a/chrome/browser/safe_browsing/path_sanitizer_unittest.cc
+++ b/chrome/browser/safe_browsing/path_sanitizer_unittest.cc
@@ -26,8 +26,10 @@ base::FilePath GetRootDirectory() {
} // namespace
+namespace safe_browsing {
+
TEST(SafeBrowsingPathSanitizerTest, HomeDirectoryIsNotEmpty) {
- safe_browsing::PathSanitizer path_sanitizer;
+ PathSanitizer path_sanitizer;
ASSERT_FALSE(path_sanitizer.GetHomeDirectory().empty());
}
@@ -38,7 +40,7 @@ TEST(SafeBrowsingPathSanitizerTest, DontStripHomeDirectoryTest) {
GetRootDirectory().Append(FILE_PATH_LITERAL("not_in_home_directory.ext"));
base::FilePath path_expected = path;
- safe_browsing::PathSanitizer path_sanitizer;
+ PathSanitizer path_sanitizer;
path_sanitizer.StripHomeDirectory(&path);
ASSERT_EQ(path.value(), path_expected.value());
@@ -46,7 +48,7 @@ TEST(SafeBrowsingPathSanitizerTest, DontStripHomeDirectoryTest) {
TEST(SafeBrowsingPathSanitizerTest, DoStripHomeDirectoryTest) {
// Test with path in home directory.
- safe_browsing::PathSanitizer path_sanitizer;
+ PathSanitizer path_sanitizer;
base::FilePath path = path_sanitizer.GetHomeDirectory().Append(
FILE_PATH_LITERAL("in_home_directory.ext"));
@@ -57,3 +59,5 @@ TEST(SafeBrowsingPathSanitizerTest, DoStripHomeDirectoryTest) {
ASSERT_EQ(path.value(), path_expected.value());
}
+
+} // namespace safe_browsing
« no previous file with comments | « chrome/browser/safe_browsing/local_two_phase_testserver.cc ('k') | chrome/browser/safe_browsing/ping_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698