[arc-intents] Adjust link-click detection code.

chrome/browser/chromeos/arc/arc_navigation_throttle.cc

Index: chrome/browser/chromeos/arc/arc_navigation_throttle.cc
diff --git a/chrome/browser/chromeos/arc/arc_navigation_throttle.cc b/chrome/browser/chromeos/arc/arc_navigation_throttle.cc
index 93131e18347c362bde6d5e6e1a02086bee645461..7b5581d4509a82293debfb4a5517590db6352462 100644
--- a/chrome/browser/chromeos/arc/arc_navigation_throttle.cc
+++ b/chrome/browser/chromeos/arc/arc_navigation_throttle.cc
@@ -16,6 +16,7 @@
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/navigation_handle.h"
 #include "net/base/registry_controlled_domains/registry_controlled_domain.h"
+#include "ui/base/page_transition_types.h"
 
 namespace arc {
 
@@ -63,7 +64,8 @@ ArcNavigationThrottle::~ArcNavigationThrottle() = default;
 content::NavigationThrottle::ThrottleCheckResult
 ArcNavigationThrottle::WillStartRequest() {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
-  if (!navigation_handle()->HasUserGesture())
+  if (navigation_handle()->GetPageTransition() != ui::PAGE_TRANSITION_LINK)

[Luis Héctor Chávez, 2016/07/21 18:08:36]

Brace elision is only valid if there is just one simple statement. This does
need braces due to the added comment.

Also, doesn't this need to be something like

(navigation_handle()->GetPageTransition() & ui::PAGE_TRANSITION_CORE_MASK) !=
ui::PAGE_TRANSITION_LINK

?

[Ben Kwa, 2016/07/21 23:07:01]

Done.  Thanks for the catch.

+    // If this navigation event wasn't spawned by the user clicking on a link.
     return content::NavigationThrottle::PROCEED;
 
   if (!ShouldOverrideUrlLoading(navigation_handle()))