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

Side by Side Diff: chrome/browser/safe_browsing/safe_browsing_service_browsertest.cc

Issue 366813007: Removed InProcessBrowserTest::CleanUpOnMainThread() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: More merge to ToT. Created 6 years, 5 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 // This test creates a safebrowsing service using test safebrowsing database 5 // This test creates a safebrowsing service using test safebrowsing database
6 // and a test protocol manager. It is used to test logics in safebrowsing 6 // and a test protocol manager. It is used to test logics in safebrowsing
7 // service. 7 // service.
8 8
9 #include <algorithm> 9 #include <algorithm>
10 10
(...skipping 955 matching lines...) Expand 10 before | Expand all | Expand 10 after
966 ASSERT_EQ("c", smt.ColumnString(0)); 966 ASSERT_EQ("c", smt.ColumnString(0));
967 ASSERT_EQ("d", smt.ColumnString(1)); 967 ASSERT_EQ("d", smt.ColumnString(1));
968 EXPECT_FALSE(smt.Step()); 968 EXPECT_FALSE(smt.Step());
969 } 969 }
970 970
971 virtual void SetUpOnMainThread() OVERRIDE { 971 virtual void SetUpOnMainThread() OVERRIDE {
972 sb_service_ = g_browser_process->safe_browsing_service(); 972 sb_service_ = g_browser_process->safe_browsing_service();
973 ASSERT_TRUE(sb_service_.get() != NULL); 973 ASSERT_TRUE(sb_service_.get() != NULL);
974 } 974 }
975 975
976 virtual void CleanUpOnMainThread() OVERRIDE { 976 virtual void TearDownOnMainThread() OVERRIDE {
977 sb_service_ = NULL; 977 sb_service_ = NULL;
978 } 978 }
979 979
980 void ForceUpdate() { 980 void ForceUpdate() {
981 sb_service_->protocol_manager()->ForceScheduleNextUpdate( 981 sb_service_->protocol_manager()->ForceScheduleNextUpdate(
982 base::TimeDelta::FromSeconds(0)); 982 base::TimeDelta::FromSeconds(0));
983 } 983 }
984 984
985 scoped_refptr<SafeBrowsingService> sb_service_; 985 scoped_refptr<SafeBrowsingService> sb_service_;
986 986
(...skipping 10 matching lines...) Expand all
997 content::WindowedNotificationObserver observer( 997 content::WindowedNotificationObserver observer(
998 chrome::NOTIFICATION_SAFE_BROWSING_UPDATE_COMPLETE, 998 chrome::NOTIFICATION_SAFE_BROWSING_UPDATE_COMPLETE,
999 content::Source<SafeBrowsingDatabaseManager>( 999 content::Source<SafeBrowsingDatabaseManager>(
1000 sb_service_->database_manager().get())); 1000 sb_service_->database_manager().get()));
1001 BrowserThread::PostTask( 1001 BrowserThread::PostTask(
1002 BrowserThread::IO, 1002 BrowserThread::IO,
1003 FROM_HERE, 1003 FROM_HERE,
1004 base::Bind(&SafeBrowsingDatabaseManagerCookieTest::ForceUpdate, this)); 1004 base::Bind(&SafeBrowsingDatabaseManagerCookieTest::ForceUpdate, this));
1005 observer.Wait(); 1005 observer.Wait();
1006 } 1006 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698