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

Unified Diff: chrome/browser/safe_browsing/two_phase_uploader_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: Other minor fixes incl. using "using safe_browsing::ClassName" instead of safe_browsing::ClassName everywhere. 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/two_phase_uploader_unittest.cc
diff --git a/chrome/browser/safe_browsing/two_phase_uploader_unittest.cc b/chrome/browser/safe_browsing/two_phase_uploader_unittest.cc
index 08d9abbce5a72188294568821740f62f72130dcb..61a0dd90c11aa4e55ba77581162d290797739fee 100644
--- a/chrome/browser/safe_browsing/two_phase_uploader_unittest.cc
+++ b/chrome/browser/safe_browsing/two_phase_uploader_unittest.cc
@@ -17,7 +17,7 @@
using content::BrowserThread;
using content::MessageLoopRunner;
-namespace {
+namespace safe_browsing {
class Delegate {
public:
@@ -54,8 +54,6 @@ base::FilePath GetTestFilePath() {
return file_path;
}
-} // namespace
mattm 2015/11/11 01:10:16 Keep the anon namespace inside the new namespace.
vakh (old account. dont use) 2015/11/11 18:59:53 Done.
-
class TwoPhaseUploaderTest : public testing::Test {
public:
TwoPhaseUploaderTest()
@@ -187,3 +185,5 @@ TEST_F(TwoPhaseUploaderTest, PhaseTwoConnectionClosed) {
EXPECT_EQ(net::URLFetcher::RESPONSE_CODE_INVALID, delegate.response_code_);
EXPECT_EQ("", delegate.response_);
}
+
+} // namespace safe_browsing

Powered by Google App Engine
This is Rietveld 408576698