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

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

Issue 7016047: Fix TestURLRequestContext leaks in MalwareDetailsTest.HTTPCache. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/safe_browsing/malware_details_unittest.cc
diff --git a/chrome/browser/safe_browsing/malware_details_unittest.cc b/chrome/browser/safe_browsing/malware_details_unittest.cc
index cc9802021694e864829fb3ed6f42ecb7f518ede9..73ccfdde74b871e2c5ac4664bbb91c279e3ee532 100644
--- a/chrome/browser/safe_browsing/malware_details_unittest.cc
+++ b/chrome/browser/safe_browsing/malware_details_unittest.cc
@@ -105,7 +105,7 @@ void FillCache(net::URLRequestContextGetter* context_getter) {
disk_cache::Backend* cache;
int rv =
context_getter->GetURLRequestContext()->http_transaction_factory()->
- GetCache()-> GetBackend(&cache, &cb);
+ GetCache()->GetBackend(&cache, &cb);
ASSERT_EQ(net::OK, cb.GetResult(rv));
std::string empty;
@@ -559,6 +559,7 @@ TEST_F(MalwareDetailsTest, HTTPCache) {
expected.set_complete(true);
VerifyResults(actual, expected);
+ profile()->ResetRequestContext();
}
// Tests the interaction with the HTTP cache (where the cache is empty).
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698