Replace DidGetUserGesture with DidGetUserInteraction in the download request limiter.

chrome/browser/download/download_request_limiter.cc

 // Copyright (c) 2012 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/download/download_request_limiter.h"
 
 #include "base/bind.h"
 #include "base/stl_util.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/content_settings/host_content_settings_map_factory.h"
@@ skipping 79 matching lines @@
       // automatic downloads content settings. Alternatively, they can copy the
       // URL into a new tab, which will make a new DownloadRequestLimiter.
       // See also the initial_page_host_ logic in Observe() for
       // NOTIFICATION_NAV_ENTRY_PENDING.
       break;
     default:
       NOTREACHED();
   }
 }
 
-void DownloadRequestLimiter::TabDownloadState::DidGetUserGesture() {
-  if (is_showing_prompt()) {
-    // Don't change the state if the user clicks on the page somewhere.
+void DownloadRequestLimiter::TabDownloadState::DidGetUserInteraction(
+    const blink::WebInputEvent::Type type) {
+  if (is_showing_prompt() || type == blink::WebInputEvent::MouseWheel) {
+    // Don't change the state if the user clicks on the page somewhere or has
+    // scrolled.

[asanka, 2016/02/22 22:16:50]

It seems the code is trying to avoid changing the state while showing a prompt,
rather than when the user clicks somewhere on the page. Update the comment?

[dominickn, 2016/02/22 23:39:26]

Done.

     return;
   }
 
 #if BUILDFLAG(ANDROID_JAVA_UI)
   bool promptable = InfoBarService::FromWebContents(web_contents()) != nullptr;
 #else
   bool promptable =
       PermissionBubbleManager::FromWebContents(web_contents()) != nullptr;
 #endif
 
@@ skipping 374 matching lines @@
       NOTREACHED();
   }
 }
 
 void DownloadRequestLimiter::Remove(TabDownloadState* state,
                                     content::WebContents* contents) {
   DCHECK(ContainsKey(state_map_, contents));
   state_map_.erase(contents);
   delete state;
 }