ClearHostCacheCallback should be in IsOffTheRecord block

cdb0144#diff-df5c427fd693f9ebb2e5952cf0e9c8c7 Auditor: @bridiver
brave · Aug 31, 2018 · 232da3e · 232da3e
1 parent 7050961
commit 232da3e
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/brave/browser/brave_browser_context.cc b/brave/browser/brave_browser_context.cc
@@ -266,11 +266,6 @@ BraveBrowserContext::~BraveBrowserContext() {
     }
   }
 
-  // Clears any data the network stack contains that may be related to the
-  // OTR session. Must be done before DestroyBrowserContextServices, since
-  // the NetworkContext is managed by one such service.
-  GetDefaultStoragePartition(this)->GetNetworkContext()->ClearHostCache(
-      nullptr, network::mojom::NetworkContext::ClearHostCacheCallback());
 
   BrowserContextDependencyManager::GetInstance()->
       DestroyBrowserContextServices(this);
@@ -282,6 +277,11 @@ BraveBrowserContext::~BraveBrowserContext() {
         base::Bind(&NotifyOTRProfileDestroyedOnIOThread,
             base::Unretained(original_context_), base::Unretained(this)));
 #endif
+    // Clears any data the network stack contains that may be related to the
+    // OTR session. Must be done before DestroyBrowserContextServices, since
+    // the NetworkContext is managed by one such service.
+    GetDefaultStoragePartition(this)->GetNetworkContext()->ClearHostCache(
+        nullptr, network::mojom::NetworkContext::ClearHostCacheCallback());
   }
 
   ShutdownStoragePartitions();