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

Unified Diff: components/offline_pages/core/background/scheduler_stub.h

Issue 2659813006: Always get device conditions from Java for every attempt. (Closed)
Patch Set: Change JNI interface to use 3 small methods instead of an object. Created 3 years, 11 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/core/background/scheduler_stub.h
diff --git a/components/offline_pages/core/background/scheduler_stub.h b/components/offline_pages/core/background/scheduler_stub.h
index 7300468422ecb549ca15f574c3f85aae3de6be10..22d453ab09b353fdcdcb5bdc5d63fb3e88e449bb 100644
--- a/components/offline_pages/core/background/scheduler_stub.h
+++ b/components/offline_pages/core/background/scheduler_stub.h
@@ -24,20 +24,27 @@ class SchedulerStub : public Scheduler {
// Unschedules the currently scheduled task, if any.
void Unschedule() override;
+ // Get the current device conditions from the android APIs.
+ DeviceConditions& GetCurrentDeviceConditions() override;
dewittj 2017/01/30 19:18:03 const
Pete Williamson 2017/01/30 21:35:24 Done.
+
bool schedule_called() const { return schedule_called_; }
bool backup_schedule_called() const { return backup_schedule_called_; }
bool unschedule_called() const { return unschedule_called_; }
- TriggerConditions const* conditions() const { return &conditions_; }
+ TriggerConditions const* trigger_conditions() const {
+ return &trigger_conditions_;
+ }
private:
bool schedule_called_;
bool backup_schedule_called_;
bool unschedule_called_;
+ bool get_current_device_conditions_called_;
long schedule_delay_;
- TriggerConditions conditions_;
+ DeviceConditions device_conditions_;
+ TriggerConditions trigger_conditions_;
};
} // namespace offline_pages

Powered by Google App Engine
This is Rietveld 408576698