Fix a bug in user input detection for the site engagement service.

chrome/browser/engagement/site_engagement_helper.cc

 // Copyright 2015 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/engagement/site_engagement_helper.h"
 
 #include <utility>
 
 #include "base/time/time.h"
 #include "base/trace_event/trace_event.h"
@@ skipping 169 matching lines @@
 }
 
 void SiteEngagementService::Helper::RecordMediaPlaying(bool is_hidden) {
   content::WebContents* contents = web_contents();
   if (contents && service_)
     service_->HandleMediaPlaying(contents, is_hidden);
 }
 
 void SiteEngagementService::Helper::DidFinishNavigation(
     content::NavigationHandle* handle) {
-  input_tracker_.Stop();
-  media_tracker_.Stop();
-
   // Ignore all schemes except HTTP and HTTPS, as well as uncommitted, non
   // main-frame, same page, or error page navigations.
   record_engagement_ = handle->GetURL().SchemeIsHTTPOrHTTPS();
   if (!handle->HasCommitted() || !handle->IsInMainFrame() ||
       handle->IsSamePage() || handle->IsErrorPage() || !record_engagement_) {
     return;
   }
 
+  input_tracker_.Stop();
+  media_tracker_.Stop();
+
   // Ignore prerender loads. This means that prerenders will not receive
   // navigation engagement. The implications are as follows:
   //
   // - Instant search prerenders from the omnibox trigger DidFinishNavigation
   //   twice: once for the prerender, and again when the page swaps in. The
   //   second trigger has transition GENERATED and receives navigation
   //   engagement.
   // - Prerenders initiated by <link rel="prerender"> (e.g. search results) are
   //   always assigned the LINK transition, which is ignored for navigation
   //   engagement.
@@ skipping 33 matching lines @@
 void SiteEngagementService::Helper::SetSecondsTrackingDelayAfterNavigation(
     int seconds) {
   g_seconds_delay_after_navigation = seconds;
 }
 
 // static
 void SiteEngagementService::Helper::SetSecondsTrackingDelayAfterShow(
     int seconds) {
   g_seconds_delay_after_show = seconds;
 }