Added completion notification to Profile's ClearNetworkingHistorySince.

chrome/browser/profiles/profile_impl_io_data.h

Index: chrome/browser/profiles/profile_impl_io_data.h
diff --git a/chrome/browser/profiles/profile_impl_io_data.h b/chrome/browser/profiles/profile_impl_io_data.h
index 55740c584ec2b64583cae80c5778056da2909e35..62ee3ca8be86b174cd89bbb2e3d921e416db5d51 100644
--- a/chrome/browser/profiles/profile_impl_io_data.h
+++ b/chrome/browser/profiles/profile_impl_io_data.h
@@ -71,7 +71,8 @@ class ProfileImplIOData : public ProfileIOData {
         GetIsolatedMediaRequestContextGetter(
             const std::string& app_id) const;
 
-    void ClearNetworkingHistorySince(base::Time time);
+    void ClearNetworkingHistorySince(base::Time time,
+                                     const base::Closure& completion);

[mmenke, 2012/10/25 20:15:52]

Think you should mention that |completion| will be called on the UI thread here.
 This behavior is non-obvious from looking at this class.  Admittedly, it's
mentioned in the other two classes, and not a whole lot of classes actually have
access to this class, but I'm paranoid.

[engedy, 2012/10/25 21:47:43]

I agree. I copied the comment from profile.h and also added it on line 171.

 
    private:
     typedef base::hash_map<std::string,
@@ -168,7 +169,8 @@ class ProfileImplIOData : public ProfileIOData {
                        net::FtpAuthCache* ftp_auth_cache) const;
 
   // Clears the networking history since |time|.
-  void ClearNetworkingHistorySinceOnIOThread(base::Time time);
+  void ClearNetworkingHistorySinceOnIOThread(base::Time time,
+                                             const base::Closure& completion);
 
   // Lazy initialization params.
   mutable scoped_ptr<LazyParams> lazy_params_;