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

Unified Diff: chrome/browser/android/offline_pages/offline_page_utils.cc

Issue 2522073002: [OfflinePages] RequestCoordinatorFactory return null for incognito (Closed)
Patch Set: Merge Created 4 years 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: chrome/browser/android/offline_pages/offline_page_utils.cc
diff --git a/chrome/browser/android/offline_pages/offline_page_utils.cc b/chrome/browser/android/offline_pages/offline_page_utils.cc
index 63b321f009c01366a3422ad73189bc397d60333f..485ed4939f813964111499a3fb764fa03d2666d9 100644
--- a/chrome/browser/android/offline_pages/offline_page_utils.cc
+++ b/chrome/browser/android/offline_pages/offline_page_utils.cc
@@ -185,6 +185,8 @@ void OfflinePageUtils::CheckExistenceOfRequestsWithURL(
const PagesExistCallback& callback) {
RequestCoordinator* request_coordinator =
RequestCoordinatorFactory::GetForBrowserContext(browser_context);
+ if (!request_coordinator)
+ return;
auto request_coordinator_continuation = [](
const std::string& name_space, const GURL& offline_page_url,

Powered by Google App Engine
This is Rietveld 408576698