Checks the age of device before initializing HaTS Notification

chrome/browser/chromeos/hats/hats_notification_controller.cc

 // Copyright (c) 2016 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/hats/hats_notification_controller.h"
 
 #include "ash/common/system/system_notifier.h"
 #include "base/feature_list.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chromeos/hats/hats_dialog.h"
+#include "chrome/browser/chromeos/login/startup_utils.h"
 #include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h"
 #include "chrome/browser/chromeos/profiles/profile_helper.h"
 #include "chrome/browser/notifications/notification_ui_manager.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/chrome_features.h"
 #include "chrome/common/pref_names.h"
 #include "chromeos/network/network_state.h"
 #include "components/prefs/pref_service.h"
 #include "content/public/browser/browser_thread.h"
 #include "grit/ash_strings.h"
 #include "grit/theme_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/message_center/message_center.h"
 #include "ui/message_center/notification_types.h"
 #include "ui/strings/grit/ui_strings.h"
 
 namespace {
 
 // Returns true if the given |profile| interacted with HaTS by either
 // dismissing the notification or taking the survey within a given threshold
-// time delta |threshold|.
-bool DidShowSurveyToProfileRecently(Profile* profile,
-                                    base::TimeDelta threshold) {
+// days |threshold|.
+bool DidShowSurveyToProfileRecently(Profile* profile, int threshold) {

[stevenjb, 2016/07/07 23:05:10]

nit: threshold_days (or just days)

   int64_t serialized_timestamp =
       profile->GetPrefs()->GetInt64(prefs::kHatsLastInteractionTimestamp);
 
   base::Time previous_interaction_timestamp =
       base::Time::FromInternalValue(serialized_timestamp);
-  return (previous_interaction_timestamp + threshold) > base::Time::Now();
+  return (previous_interaction_timestamp +
+          base::TimeDelta::FromDays(threshold)) > base::Time::Now();
+}
+
+// Returns true if at least |threshold| days have passed since OOBE. This is an
+// indirect measure of whether the owner has used the device for at least
+// |threshold| days.
+bool IsNewDevice(int threshold) {

[stevenjb, 2016/07/07 23:05:10]

nit: threshold_days (or just days)

+  return chromeos::StartupUtils::GetTimeSinceOobeFlagFileCreation() <=
+         base::TimeDelta::FromDays(threshold);
 }
 
 }  // namespace
 
 namespace chromeos {
 
 // static
 const char HatsNotificationController::kDelegateId[] = "hats_delegate";
 
 // static
 const char HatsNotificationController::kNotificationId[] = "hats_notification";
 
 // static
-const base::TimeDelta HatsNotificationController::kHatsThresholdTime =
-    base::TimeDelta::FromDays(90);
+const int kHatsThresholdDays = 90;
+
+// static
+const int kHatsNewDeviceThresholdDays = 7;
 
 HatsNotificationController::HatsNotificationController(Profile* profile)
-    : profile_(profile) {
-  // Add self as an observer to be notified when an internet connection is
-  // available.
-  network_portal_detector::GetInstance()->AddAndFireObserver(this);
+    : profile_(profile), weak_pointer_factory_(this) {
+  base::PostTaskAndReplyWithResult(
+      content::BrowserThread::GetBlockingPool(), FROM_HERE,
+      base::Bind(&IsNewDevice, kHatsNewDeviceThresholdDays),
+      base::Bind(&HatsNotificationController::Initialize,
+                 weak_pointer_factory_.GetWeakPtr()));
 }
 
 HatsNotificationController::~HatsNotificationController() {
   network_portal_detector::GetInstance()->RemoveObserver(this);
 }
 
+void HatsNotificationController::Initialize(bool is_new_device) {
+  if (is_new_device) {
+    // This device has been chosen for a survey, but it is too new. Instead
+    // of showing the user the survey, just mark it as completed.
+    UpdateLastInteractionTime();
+    return;
+  }
+
+  // Add self as an observer to be notified when an internet connection is
+  // available.
+  network_portal_detector::GetInstance()->AddAndFireObserver(this);
+}
+
 // static
 // TODO(malaykeshav): Add check for @google accounts.
 bool HatsNotificationController::ShouldShowSurveyToProfile(Profile* profile) {
-  // Do not show the survey if the HaTS feature is disabled for the device.
+  // Do not show the survey if the HaTS feature is disabled for the device. This
+  // flag is controlled by finch and is enabled only when the device has been
+  // selected for the survey.
   if (!base::FeatureList::IsEnabled(features::kHappininessTrackingSystem))
     return false;
 
   // Do not show survey if this is a guest session.
   if (profile->IsGuestSession())
     return false;
 
   // Do not show the survey if the current user is not an owner.
   if (!ProfileHelper::IsOwnerProfile(profile))
     return false;
 
   // Do not show survey if this is an enterprise managed device.
   // TODO(malaykeshav): Show survey to google users but not any other enterprise
   // users.
   if (g_browser_process->platform_part()
           ->browser_policy_connector_chromeos()
           ->IsEnterpriseManaged()) {
     return false;
   }
 
   // Do not show survey to user if user has interacted with HaTS within the past
   // |kHatsThresholdTime| time delta.
-  if (DidShowSurveyToProfileRecently(profile, kHatsThresholdTime))
+  if (DidShowSurveyToProfileRecently(profile, kHatsThresholdDays))
     return false;
 
   return true;
 }
 
 // NotificationDelegate override:
 std::string HatsNotificationController::id() const {
   return kDelegateId;
 }
 
@@ skipping 53 matching lines @@
       GURL() /* Send an empty invalid url */, kNotificationId, optional, this);
 }
 
 void HatsNotificationController::UpdateLastInteractionTime() {
   PrefService* pref_service = profile_->GetPrefs();
   pref_service->SetInt64(prefs::kHatsLastInteractionTimestamp,
                          base::Time::Now().ToInternalValue());
 }
 
 }  // namespace chromeos