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

chrome/browser/engagement/site_engagement_helper.cc

Index: chrome/browser/engagement/site_engagement_helper.cc
diff --git a/chrome/browser/engagement/site_engagement_helper.cc b/chrome/browser/engagement/site_engagement_helper.cc
index ac3d3210ad43ae6523ed455f5198a1af48b6f491..d3bab2987a4b4c05ba9001c3a783a97f224b5315 100644
--- a/chrome/browser/engagement/site_engagement_helper.cc
+++ b/chrome/browser/engagement/site_engagement_helper.cc
@@ -187,9 +187,6 @@ void SiteEngagementService::Helper::RecordMediaPlaying(bool 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();
@@ -198,6 +195,9 @@ void SiteEngagementService::Helper::DidFinishNavigation(
     return;
   }
 
+  input_tracker_.Stop();
+  media_tracker_.Stop();
+
   // Ignore prerender loads. This means that prerenders will not receive
   // navigation engagement. The implications are as follows:
   //