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

Unified Diff: components/offline_pages/background/request_coordinator.h

Issue 2431193003: [Offline Pages] Defines longer processing budget for immediate bg loads. (Closed)
Patch Set: Fixes tests wrt setting processing state Created 4 years, 2 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/offline_pages/background/request_coordinator.h
diff --git a/components/offline_pages/background/request_coordinator.h b/components/offline_pages/background/request_coordinator.h
index 371fd23c319dac2200575e2fdbc700c98867a499..ec15d06556b48c34529b67e92c55705c86901403 100644
--- a/components/offline_pages/background/request_coordinator.h
+++ b/components/offline_pages/background/request_coordinator.h
@@ -145,6 +145,8 @@ class RequestCoordinator : public KeyedService,
immediate_schedule_callback_ = callback;
}
+ void StartImmediatelyForTest() { StartImmediatelyIfConnected(); }
+
// Observers implementing the RequestCoordinator::Observer interface can
// register here to get notifications of changes to request state. This
// pointer is not owned, and it is the callers responsibility to remove the
@@ -327,6 +329,9 @@ class RequestCoordinator : public KeyedService,
friend class RequestCoordinatorTest;
+ // Cached value of whether low end device. Overwritable for testing.
+ bool is_low_end_device_;
+
// The offliner can only handle one request at a time - if the offliner is
// busy, prevent other requests. This flag marks whether the offliner is in
// use.

Powered by Google App Engine
This is Rietveld 408576698