Fix the handling of user gestures for external protocol handler dialogs.

chrome/browser/extensions/active_tab_permission_granter.cc

Index: chrome/browser/extensions/active_tab_permission_granter.cc
diff --git a/chrome/browser/extensions/active_tab_permission_granter.cc b/chrome/browser/extensions/active_tab_permission_granter.cc
index ecedd3458bd5e6bca93800f98879b9ee0fa16fa4..756007e140c2c731243e4ad4254621aa36e26f92 100644
--- a/chrome/browser/extensions/active_tab_permission_granter.cc
+++ b/chrome/browser/extensions/active_tab_permission_granter.cc
@@ -37,6 +37,9 @@ ActiveTabPermissionGranter::ActiveTabPermissionGranter(
 ActiveTabPermissionGranter::~ActiveTabPermissionGranter() {}
 
 void ActiveTabPermissionGranter::GrantIfRequested(const Extension* extension) {
+  // Active tab grant request implies there was a user gesture.
+  web_contents()->UserGestureDone();

[not at google - send to devlin, 2014/03/28 18:54:23]

and this trickles down into the protocol handler somehow?

[meacer, 2014/03/28 19:53:11]

Yes, WebContentsImpl::OnUserGesture calls ResourceDispatcherHost::OnUserGesture
which updates the last user gesture time which is then used to decide if there
is a recent user gesture to pass to the protocol handler.

+
   if (granted_extensions_.Contains(extension->id()))
     return;