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

Unified Diff: chrome/browser/policy/device_status_collector.h

Issue 10103029: Add device location reporting (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Comments addressed. Created 8 years, 8 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: chrome/browser/policy/device_status_collector.h
diff --git a/chrome/browser/policy/device_status_collector.h b/chrome/browser/policy/device_status_collector.h
index 4afffcb09a12ae3b33f9fd7ac157e2aa46c3ca29..5e1f1819e3af1a149b2c5ef458573ad63b4de5de 100644
--- a/chrome/browser/policy/device_status_collector.h
+++ b/chrome/browser/policy/device_status_collector.h
@@ -23,10 +23,13 @@ namespace enterprise_management {
class DeviceStatusReportRequest;
}
+class GeolocationProvider;
class PrefService;
namespace policy {
+class DeviceStatusLocationHelper;
+
// Collects and summarizes the status of an enterprised-managed ChromeOS device.
class DeviceStatusCollector : public content::NotificationObserver {
public:
@@ -51,6 +54,8 @@ class DeviceStatusCollector : public content::NotificationObserver {
// Callback which receives the results of the idle state check.
void IdleStateCallback(IdleState state);
+ void SetGeolocationProviderForTest(GeolocationProvider* provider);
+
// The number of days in the past to store device activity.
// This is kept in case device status uploads fail for a number of days.
unsigned int max_stored_past_activity_days_;
@@ -80,6 +85,8 @@ class DeviceStatusCollector : public content::NotificationObserver {
enterprise_management::DeviceStatusReportRequest* request);
void GetBootMode(
enterprise_management::DeviceStatusReportRequest* request);
+ void GetLocation(
+ enterprise_management::DeviceStatusReportRequest* request);
// Update the cached values of the reporting settings.
void UpdateReportingSettings();
@@ -108,12 +115,16 @@ class DeviceStatusCollector : public content::NotificationObserver {
chromeos::system::StatisticsProvider* statistics_provider_;
+ GeolocationProvider* geolocation_provider_for_test_;
+ DeviceStatusLocationHelper* location_helper_;
+
chromeos::CrosSettings* cros_settings_;
// Cached values of the reporting settings from the device policy.
bool report_version_info_;
bool report_activity_times_;
bool report_boot_mode_;
+ bool report_location_;
DISALLOW_COPY_AND_ASSIGN(DeviceStatusCollector);
};

Powered by Google App Engine
This is Rietveld 408576698