Add a delay to Site Engagement tracking after navigation.

chrome/browser/engagement/site_engagement_helper.h

Index: chrome/browser/engagement/site_engagement_helper.h
diff --git a/chrome/browser/engagement/site_engagement_helper.h b/chrome/browser/engagement/site_engagement_helper.h
index bd4565009ed646d6cbf99d83d9a5eeef7fdc551e..d0e52873baec16a44a2bf81d198055873784da98 100644
--- a/chrome/browser/engagement/site_engagement_helper.h
+++ b/chrome/browser/engagement/site_engagement_helper.h
@@ -26,7 +26,9 @@ class SiteEngagementHelper
  public:
   ~SiteEngagementHelper() override;
 
-  static void SetSecondsBetweenUserInputCheck(double seconds);
+  static void SetSecondsBetweenUserInputCheck(int seconds);
+  static void SetSecondsTrackingDelayAfterNavigation(int seconds);
+  static void SetSecondsTrackingDelayAfterShow(int seconds);
 
  private:
   // Class to encapsulate the user input listening.
@@ -52,28 +54,41 @@ class SiteEngagementHelper
     bool HandleMouseEvent(const blink::WebMouseEvent& event);
 
     // Register callbacks to listen for user input.
-    void StartTracking(content::RenderViewHost* host);
+    void StartTracking(content::RenderViewHost* host,
+                       base::TimeDelta initial_delay);
 
-    // Pause listening for user input, restarting listening after
-    // g_seconds_between_user_input_check seconds.
-    void PauseTracking(content::RenderViewHost* host);
+    // Pause listening for user input, restarting listening after a delay.
+    void PauseTracking();
 
-    // Restart listening for user input.
-    void ResumeTracking();
+    void SwitchRenderViewHost(content::RenderViewHost* new_host);
 
     // Stop listening for user input.
-    void StopTracking(content::RenderViewHost* host);
+    void StopTracking();
 
-    // Set the timer object for testing purposes.
-    void SetTimerForTesting(scoped_ptr<base::Timer> timer);
+    // Returns whether the InputTracker has been started for a RenderViewHost.
+    bool is_tracking() const { return is_tracking_; }
+
+    // Returns whether input tracking callbacks have been added to
+    // RenderViewHost.
+    bool callbacks_added() const { return callbacks_added_; }
 
-    bool callbacks_added() { return callbacks_added_; }
+    // Set the timer object for testing purposes.
+    void SetPauseTimerForTesting(scoped_ptr<base::Timer> timer);
 
    private:
+    // Starts the timer for adding callbacks to the RenderViewHost.
+    void StartTimer(base::TimeDelta delay);
+
+    // Adds/removes tracking callbacks to the RenderViewHost.
+    void AddCallbacks();
+    void RemoveCallbacks();
+
     SiteEngagementHelper* helper_;
     scoped_ptr<base::Timer> pause_timer_;
+    content::RenderViewHost* host_;
     content::RenderWidgetHost::KeyPressEventCallback key_press_event_callback_;
     content::RenderWidgetHost::MouseEventCallback mouse_event_callback_;
+    bool is_tracking_;
     bool callbacks_added_;
   };
 
@@ -85,16 +100,12 @@ class SiteEngagementHelper
   // current contents location.
   void RecordUserInput(SiteEngagementMetrics::EngagementType type);
 
-  bool ShouldRecordEngagement();
-
   // content::WebContentsObserver overrides.
   void DidNavigateMainFrame(
       const content::LoadCommittedDetails& details,
       const content::FrameNavigateParams& params) override;
-
   void RenderViewHostChanged(content::RenderViewHost* old_host,
                              content::RenderViewHost* new_host) override;
-
   void WasShown() override;
   void WasHidden() override;