Index: components/safe_browsing_db/v4_local_database_manager.h |
diff --git a/components/safe_browsing_db/v4_local_database_manager.h b/components/safe_browsing_db/v4_local_database_manager.h |
index f9510c4af51fceb49f9b354f6f31c2ab34a57558..480feb17fcef6347f5547d7b2f27406313605baa 100644 |
--- a/components/safe_browsing_db/v4_local_database_manager.h |
+++ b/components/safe_browsing_db/v4_local_database_manager.h |
@@ -37,9 +37,9 @@ class V4LocalDatabaseManager : public SafeBrowsingDatabaseManager { |
CHECK_MAX |
}; |
- // Construct V4LocalDatabaseManager. |
- // Must be initialized by calling StartOnIOThread() before using. |
- V4LocalDatabaseManager(const base::FilePath& base_path); |
+ // Create and return an instance of V4LocalDatabaseManager, if Finch trial |
+ // allows it; nullptr otherwise. |
+ static V4LocalDatabaseManager* Create(const base::FilePath& base_path); |
Nathan Parker
2016/09/28 22:25:55
return a unique_ptr
vakh (use Gerrit instead)
2016/09/28 23:00:02
It looks like the destructor of the class needs to
Nathan Parker
2016/09/28 23:50:14
Ah yes, it's a base::RefCountedThreadSafe<> decend
|
// |
// SafeBrowsingDatabaseManager implementation |
@@ -74,6 +74,10 @@ class V4LocalDatabaseManager : public SafeBrowsingDatabaseManager { |
// |
protected: |
+ // Construct V4LocalDatabaseManager. |
+ // Must be initialized by calling StartOnIOThread() before using. |
+ V4LocalDatabaseManager(const base::FilePath& base_path); |
+ |
// The information we need to process a URL safety reputation request and |
// respond to the SafeBrowsing client that asked for it. |
// TODO(vakh): In its current form, it only includes information for |