Added completion notification to Profile's ClearNetworkingHistorySince.

chrome/browser/profiles/off_the_record_profile_impl.cc

Index: chrome/browser/profiles/off_the_record_profile_impl.cc
diff --git a/chrome/browser/profiles/off_the_record_profile_impl.cc b/chrome/browser/profiles/off_the_record_profile_impl.cc
index 75d42a62520d1dfa419c5cc24a9833f88e5261a2..1c2395d1bb49a4d8c9e3850a28c507501c4aa516 100644
--- a/chrome/browser/profiles/off_the_record_profile_impl.cc
+++ b/chrome/browser/profiles/off_the_record_profile_impl.cc
@@ -452,8 +452,14 @@ chrome_browser_net::Predictor* OffTheRecordProfileImpl::GetNetworkPredictor() {
   return NULL;
 }
 
-void OffTheRecordProfileImpl::ClearNetworkingHistorySince(base::Time time) {
+void OffTheRecordProfileImpl::ClearNetworkingHistorySince(
+    base::Time time,
+    const base::Closure& completion) {
   // No need to do anything here, our transport security state is read-only.
+  // Fire the callback though, to make sure we do not cause a hang in any case.

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

nit:  It's actually a leak we're preventing.

I'd suggest mentioning it's async specifically.  Maybe "Run callback
asynchronously to avoid reetrancy issues."

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

Done.

+  if (!completion.is_null()) {
+    BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, completion);
+  }
 }
 
 GURL OffTheRecordProfileImpl::GetHomePage() {