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

Unified Diff: components/keyed_service/ios/browser_state_dependency_manager.cc

Issue 2749823002: Restore KeyedServiceFactory diagnostics for context use-after-destroy. (Closed)
Patch Set: Refactor SiteEngagementService tests. Created 3 years, 9 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: components/keyed_service/ios/browser_state_dependency_manager.cc
diff --git a/components/keyed_service/ios/browser_state_dependency_manager.cc b/components/keyed_service/ios/browser_state_dependency_manager.cc
index fb313ae3d997f3350062a10c3b3e76bfeb8d2a18..7dcc8d9a72087177d61adee1525bddee2b166bf3 100644
--- a/components/keyed_service/ios/browser_state_dependency_manager.cc
+++ b/components/keyed_service/ios/browser_state_dependency_manager.cc
@@ -34,17 +34,15 @@ void BrowserStateDependencyManager::DestroyBrowserStateServices(
DependencyManager::DestroyContextServices(context);
}
-#ifndef NDEBUG
void BrowserStateDependencyManager::AssertBrowserStateWasntDestroyed(
- web::BrowserState* context) {
+ web::BrowserState* context) const {
DependencyManager::AssertContextWasntDestroyed(context);
}
-void BrowserStateDependencyManager::MarkBrowserStateLiveForTesting(
+void BrowserStateDependencyManager::MarkBrowserStateLive(
web::BrowserState* context) {
- DependencyManager::MarkContextLiveForTesting(context);
+ DependencyManager::MarkContextLive(context);
}
-#endif // NDEBUG
BrowserStateDependencyManager::BrowserStateDependencyManager() {
}

Powered by Google App Engine
This is Rietveld 408576698