Implemented DeviceStatusCollector::GetDeviceSessionStatus()

chrome/browser/chromeos/settings/device_settings_provider.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/settings/device_settings_provider.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/logging.h"
@@ skipping 46 matching lines @@
     kDeviceAttestationEnabled,
     kDeviceOwner,
     kPolicyMissingMitigationMode,
     kReleaseChannel,
     kReleaseChannelDelegated,
     kReportDeviceActivityTimes,
     kReportDeviceBootMode,
     kReportDeviceLocation,
     kReportDeviceNetworkInterfaces,
     kReportDeviceUsers,
-  kReportDeviceHardwareStatus,
+    kReportDeviceHardwareStatus,
+    kReportDeviceSessionStatus,
     kReportDeviceVersionInfo,
     kServiceAccountIdentity,
     kSignedDataRoamingEnabled,
     kStartUpFlags,
     kStatsReportingPref,
     kSystemTimezonePolicy,
     kSystemUse24HourClock,
     kUpdateDisabled,
     kVariationsRestrictParameter,
     kDeviceDisabled,
@@ skipping 226 matching lines @@
     if (reporting_policy.has_report_users()) {
       new_values_cache->SetBoolean(
           kReportDeviceUsers,
           reporting_policy.report_users());
     }
     if (reporting_policy.has_report_hardware_status()) {
       new_values_cache->SetBoolean(
           kReportDeviceHardwareStatus,
           reporting_policy.report_hardware_status());
     }
+    if (reporting_policy.has_report_session_status()) {

[bartfab (slow), 2015/02/11 17:28:43]

You need to add this new policy policy_templates.json. You also have to wire up
decoding in device_policy_decoder_chromeos.cc so that the value shows up in
chrome:// policy for debugging.

While you are at it, please add |report_hardware_status| in those two places as
well - it seems to have been missed.

[Andrew T Wilson (Slow), 2015/02/12 15:15:39]

This is part of the other CL which I will land before this.

+      new_values_cache->SetBoolean(
+          kReportDeviceSessionStatus,
+          reporting_policy.report_session_status());
+    }
   }
 }
 
 void DecodeGenericPolicies(
     const em::ChromeDeviceSettingsProto& policy,
     PrefValueMap* new_values_cache) {
   if (policy.has_metrics_enabled()) {
     new_values_cache->SetBoolean(kStatsReportingPref,
                                  policy.metrics_enabled().metrics_enabled());
   } else {
@@ skipping 437 matching lines @@
 void DeviceSettingsProvider::AttemptMigration() {
   if (device_settings_service_->HasPrivateOwnerKey()) {
     PrefValueMap::const_iterator i;
     for (i = migration_values_.begin(); i != migration_values_.end(); ++i)
       DoSet(i->first, *i->second);
     migration_values_.Clear();
   }
 }
 
 }  // namespace chromeos