Factor out ShouldIgnoreNavigation and add tests for the function

chrome/browser/chromeos/arc/intent_helper/arc_external_protocol_dialog_unittest.cc

Index: chrome/browser/chromeos/arc/intent_helper/arc_external_protocol_dialog_unittest.cc
diff --git a/chrome/browser/chromeos/arc/intent_helper/arc_external_protocol_dialog_unittest.cc b/chrome/browser/chromeos/arc/intent_helper/arc_external_protocol_dialog_unittest.cc
index 06824e3e7209152fcadfa07646abfa1458017a02..8001672f3ecd330531f6a63a9d40a9c22e20232c 100644
--- a/chrome/browser/chromeos/arc/intent_helper/arc_external_protocol_dialog_unittest.cc
+++ b/chrome/browser/chromeos/arc/intent_helper/arc_external_protocol_dialog_unittest.cc
@@ -431,4 +431,18 @@ TEST(ArcExternalProtocolDialogTest, TestGetActionWithGeoUrlAsFallback) {
   EXPECT_EQ(kChromePackageName, url_and_package.second);
 }
 
+// Test that ShouldIgnoreNavigation accepts CLIENT_REDIRECT qualifier but does
+// not other ones.
+TEST(ArcExternalProtocolDialogTest, TestShouldIgnoreNavigation) {
+  EXPECT_FALSE(ShouldIgnoreNavigationForTesting(
+      ui::PageTransitionFromInt(ui::PAGE_TRANSITION_LINK)));
+  EXPECT_FALSE(ShouldIgnoreNavigationForTesting(ui::PageTransitionFromInt(
+      ui::PAGE_TRANSITION_LINK | ui::PAGE_TRANSITION_CLIENT_REDIRECT)));
+  EXPECT_TRUE(ShouldIgnoreNavigationForTesting(ui::PageTransitionFromInt(
+      ui::PAGE_TRANSITION_LINK | ui::PAGE_TRANSITION_CLIENT_REDIRECT |
+      ui::PAGE_TRANSITION_HOME_PAGE)));
+  EXPECT_TRUE(ShouldIgnoreNavigationForTesting(ui::PageTransitionFromInt(
+      ui::PAGE_TRANSITION_LINK | ui::PAGE_TRANSITION_HOME_PAGE)));
+}
+
 }  // namespace arc