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

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

Issue 410843003: Disable all the tests that are flaking more than 5% on Windows builders. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase 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 697 matching lines...) Expand 10 before | Expand all | Expand 10 after
708 GenUrlFullhashResult(badbin_url, safe_browsing_util::BINURL, 708 GenUrlFullhashResult(badbin_url, safe_browsing_util::BINURL,
709 &full_hash_result); 709 &full_hash_result);
710 SetupResponseForUrl(badbin_url, full_hash_result); 710 SetupResponseForUrl(badbin_url, full_hash_result);
711 711
712 client->CheckDownloadUrl(badbin_urls); 712 client->CheckDownloadUrl(badbin_urls);
713 713
714 // Now, the badbin_url is not safe since it is added to download database. 714 // Now, the badbin_url is not safe since it is added to download database.
715 EXPECT_EQ(SB_THREAT_TYPE_BINARY_MALWARE_URL, client->GetThreatType()); 715 EXPECT_EQ(SB_THREAT_TYPE_BINARY_MALWARE_URL, client->GetThreatType());
716 } 716 }
717 717
718 IN_PROC_BROWSER_TEST_F(SafeBrowsingServiceTest, CheckDownloadUrlTimedOut) { 718 #if defined(OS_WIN)
719 // http://crbug.com/396409
720 #define MAYBE_CheckDownloadUrlTimedOut DISABLED_CheckDownloadUrlTimedOut
721 #else
722 #define MAYBE_CheckDownloadUrlTimedOut CheckDownloadUrlTimedOut
723 #endif
724 IN_PROC_BROWSER_TEST_F(SafeBrowsingServiceTest,
725 MAYBE_CheckDownloadUrlTimedOut) {
719 GURL badbin_url = test_server()->GetURL(kMalwareFile); 726 GURL badbin_url = test_server()->GetURL(kMalwareFile);
720 std::vector<GURL> badbin_urls(1, badbin_url); 727 std::vector<GURL> badbin_urls(1, badbin_url);
721 728
722 scoped_refptr<TestSBClient> client(new TestSBClient); 729 scoped_refptr<TestSBClient> client(new TestSBClient);
723 SBFullHashResult full_hash_result; 730 SBFullHashResult full_hash_result;
724 GenUrlFullhashResult(badbin_url, safe_browsing_util::BINURL, 731 GenUrlFullhashResult(badbin_url, safe_browsing_util::BINURL,
725 &full_hash_result); 732 &full_hash_result);
726 SetupResponseForUrl(badbin_url, full_hash_result); 733 SetupResponseForUrl(badbin_url, full_hash_result);
727 client->CheckDownloadUrl(badbin_urls); 734 client->CheckDownloadUrl(badbin_urls);
728 735
(...skipping 268 matching lines...) Expand 10 before | Expand all | Expand 10 after
997 content::WindowedNotificationObserver observer( 1004 content::WindowedNotificationObserver observer(
998 chrome::NOTIFICATION_SAFE_BROWSING_UPDATE_COMPLETE, 1005 chrome::NOTIFICATION_SAFE_BROWSING_UPDATE_COMPLETE,
999 content::Source<SafeBrowsingDatabaseManager>( 1006 content::Source<SafeBrowsingDatabaseManager>(
1000 sb_service_->database_manager().get())); 1007 sb_service_->database_manager().get()));
1001 BrowserThread::PostTask( 1008 BrowserThread::PostTask(
1002 BrowserThread::IO, 1009 BrowserThread::IO,
1003 FROM_HERE, 1010 FROM_HERE,
1004 base::Bind(&SafeBrowsingDatabaseManagerCookieTest::ForceUpdate, this)); 1011 base::Bind(&SafeBrowsingDatabaseManagerCookieTest::ForceUpdate, this));
1005 observer.Wait(); 1012 observer.Wait();
1006 } 1013 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698