Checks the age of device before initializing HaTS Notification

chrome/browser/chromeos/hats/hats_notification_controller.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_HATS_HATS_NOTIFICATION_CONTROLLER_H_
 #define CHROME_BROWSER_CHROMEOS_HATS_HATS_NOTIFICATION_CONTROLLER_H_
 
 #include "base/macros.h"
+#include "base/memory/weak_ptr.h"
 #include "base/time/time.h"
 #include "chrome/browser/notifications/notification.h"
 #include "chrome/browser/notifications/notification_delegate.h"
 #include "chromeos/network/portal_detector/network_portal_detector.h"
 
 class Profile;
 class NetworkState;
 
 namespace chromeos {
 
 // Happiness tracking survey (HaTS) notification controller is responsible for
 // managing the HaTS notification that is displayed to the user.
 class HatsNotificationController : public NotificationDelegate,
                                    public NetworkPortalDetector::Observer {
  public:
   // Minimum amount of time before the notification is displayed again after a
   // user has interacted with it.
   static const base::TimeDelta kHatsThresholdTime;
+  // Minimum amount of time after initial login or oobe after which we can show
+  // the HaTS notification.
+  static const base::TimeDelta kHatsNewDeviceThresholdTime;
   static const char kDelegateId[];
   static const char kNotificationId[];

[stevenjb, 2016/06/29 23:35:55]

Any of these that are not used outside of hats_notification_controller.cc should
be defined as file local variables inside an anonymous namespace.

[malaykeshav, 2016/06/30 01:30:16]

These are used in the unittests. Would have to move them back here again when
writing the test.

 
   explicit HatsNotificationController(Profile* profile);
 
   // Returns true if the survey needs to be displayed for the given |profile|.
   static bool ShouldShowSurveyToProfile(Profile* profile);
 
  private:
   ~HatsNotificationController() override;
 
   // NotificationDelegate overrides:
+  void Initialize(bool is_new_device);
   void ButtonClick(int button_index) override;
   void Close(bool by_user) override;
   std::string id() const override;
 
   // NetworkPortalDetector::Observer override:
   void OnPortalDetectionCompleted(
       const NetworkState* network,
       const NetworkPortalDetector::CaptivePortalState& state) override;
 
   Notification* CreateNotification();
   void UpdateLastInteractionTime();
 
   Profile* profile_;
+  base::WeakPtrFactory<HatsNotificationController> weak_pointer_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(HatsNotificationController);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_HATS_HATS_NOTIFICATION_CONTROLLER_H_