Add reporting notification to the enterprise banner on the CrOS login screen.

chrome/browser/chromeos/login/version_info_updater.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/login/version_info_updater.h"
 
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/chromeos/chromeos_version.h"
 #include "base/string16.h"
 #include "base/string_util.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chromeos/cros/cros_library.h"
+#include "chrome/browser/chromeos/cros_settings.h"
+#include "chrome/browser/chromeos/cros_settings_names.h"
 #include "chrome/browser/policy/browser_policy_connector.h"
 #include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/chrome_version_info.h"
 #include "googleurl/src/gurl.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "third_party/cros_system_api/window_manager/chromeos_wm_ipc_enums.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 
 namespace chromeos {
 
+namespace {
+
+const char* kReportingFlags[] = {
+  chromeos::kReportDeviceVersionInfo,
+  chromeos::kReportDeviceActivityTimes,
+  chromeos::kReportDeviceBootMode,
+};
+
+}
+
 ///////////////////////////////////////////////////////////////////////////////
 // VersionInfoUpdater public:
 
 VersionInfoUpdater::VersionInfoUpdater(Delegate* delegate)
-    : delegate_(delegate) {
+    : enterprise_reporting_hint_(false),
+      delegate_(delegate) {
+  cros_settings_ = chromeos::CrosSettings::Get();
+
+  // Watch for changes to the reporting flags.
+  for (unsigned int i = 0; i < arraysize(kReportingFlags); ++i)
+    cros_settings_->AddSettingsObserver(kReportingFlags[i], this);
 }
 
 VersionInfoUpdater::~VersionInfoUpdater() {
 }
 
 void VersionInfoUpdater::StartUpdate(bool is_official_build) {
   if (base::chromeos::IsRunningOnChromeOS()) {
     version_loader_.GetVersion(
         &version_consumer_,
         base::Bind(&VersionInfoUpdater::OnVersion, base::Unretained(this)),
@@ skipping 90 matching lines @@
       case policy::CloudPolicySubsystem::NETWORK_ERROR:
         status_text = l10n_util::GetStringUTF8(
             IDS_LOGIN_MANAGED_BY_STATUS_NETWORK_ERROR);
         break;
       case policy::CloudPolicySubsystem::TOKEN_FETCHED:
       case policy::CloudPolicySubsystem::SUCCESS:
         break;
     }
   }
 
-  SetEnterpriseInfo(policy_connector->GetEnterpriseDomain(), status_text);
+  bool reporting_hint = false;
+  for (unsigned int i = 0; i < arraysize(kReportingFlags); ++i) {
+    bool enabled = false;
+    if (cros_settings_->GetBoolean(kReportingFlags[i], &enabled))

[Patrick Dubroy, 2012/03/26 08:28:20]

Why not just "="? Using bitwise OR on a boolean seems a little funny. Also,
might as well break as soon as one flag is found to be true. 

[Mattias Nissler (ping if slow), 2012/03/26 13:08:26]

Done.

+      reporting_hint |= enabled;
+  }
+
+  SetEnterpriseInfo(policy_connector->GetEnterpriseDomain(),
+                    status_text,
+                    reporting_hint);
 }
 
 void VersionInfoUpdater::SetEnterpriseInfo(const std::string& domain_name,
-                                           const std::string& status_text) {
+                                           const std::string& status_text,
+                                           bool reporting_hint) {
   if (domain_name != enterprise_domain_text_ ||
-      status_text != enterprise_status_text_) {
+      status_text != enterprise_status_text_ ||
+      reporting_hint != enterprise_reporting_hint_) {
     enterprise_domain_text_ = domain_name;
     enterprise_status_text_ = status_text;
+    enterprise_reporting_hint_ = enterprise_reporting_hint_;
     UpdateVersionLabel();
 
     // Update the notification about device status reporting.
     if (delegate_) {
       std::string enterprise_info;
       if (!domain_name.empty()) {
         enterprise_info = l10n_util::GetStringFUTF8(
             IDS_LOGIN_MANAGED_BY_NOTICE,
             UTF8ToUTF16(enterprise_domain_text_));
-        delegate_->OnEnterpriseInfoUpdated(enterprise_info);
+        delegate_->OnEnterpriseInfoUpdated(enterprise_info, reporting_hint);
       }
     }
   }
 }
 
 void VersionInfoUpdater::OnVersion(
     VersionLoader::Handle handle, std::string version) {
   version_text_.swap(version);
   UpdateVersionLabel();
 }
@@ skipping 31 matching lines @@
   if (delegate_)
     delegate_->OnBootTimesLabelTextUpdated(boot_times_text);
 }
 
 void VersionInfoUpdater::OnPolicyStateChanged(
     policy::CloudPolicySubsystem::PolicySubsystemState state,
     policy::CloudPolicySubsystem::ErrorDetails error_details) {
   UpdateEnterpriseInfo();
 }
 
+void VersionInfoUpdater::Observe(
+    int type,
+    const content::NotificationSource& source,
+    const content::NotificationDetails& details) {
+  if (type == chrome::NOTIFICATION_SYSTEM_SETTING_CHANGED)
+    UpdateEnterpriseInfo();
+  else
+    NOTREACHED();
+}
+
 }  // namespace chromeos