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

Unified Diff: chrome/browser/safe_browsing/malware_details_cache.h

Issue 10068036: RefCounted types should not have public destructors, chrome/browser/ part 5 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Win fix Created 8 years, 8 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
Index: chrome/browser/safe_browsing/malware_details_cache.h
diff --git a/chrome/browser/safe_browsing/malware_details_cache.h b/chrome/browser/safe_browsing/malware_details_cache.h
index 383e8b64e24d459e9b6074ac3c83dfa033202378..d52188b7138254b4b65c819eaebb21475a1f3223 100644
--- a/chrome/browser/safe_browsing/malware_details_cache.h
+++ b/chrome/browser/safe_browsing/malware_details_cache.h
@@ -38,7 +38,6 @@ class MalwareDetailsCacheCollector
public:
MalwareDetailsCacheCollector();
- virtual ~MalwareDetailsCacheCollector();
// We use |request_context_getter|, we modify |resources| and
// |result|, and we call |callback|, so they must all remain alive
@@ -58,6 +57,10 @@ class MalwareDetailsCacheCollector
virtual void OnURLFetchComplete(const content::URLFetcher* source) OVERRIDE;
private:
+ friend class base::RefCountedThreadSafe<MalwareDetailsCacheCollector>;
+
+ virtual ~MalwareDetailsCacheCollector();
+
// Points to the url for which we are fetching the HTTP cache entry or
// redirect chain.
safe_browsing::ResourceMap::iterator resources_it_;

Powered by Google App Engine
This is Rietveld 408576698