Modifies how Arc's throttle handles redirections

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

 // Copyright 2016 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 <algorithm>
 
 #include "chrome/browser/chromeos/arc/intent_helper/arc_navigation_throttle.h"
 #include "components/arc/intent_helper/arc_intent_helper_bridge.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
@@ skipping 45 matching lines @@
   EXPECT_FALSE(ArcNavigationThrottle::ShouldOverrideUrlLoadingForTesting(
       GURL("http://a.google.com/"), GURL()));
   EXPECT_FALSE(ArcNavigationThrottle::ShouldOverrideUrlLoadingForTesting(
       GURL(), GURL()));
 
   // A navigation not within the same domain can be overridden.
   EXPECT_TRUE(ArcNavigationThrottle::ShouldOverrideUrlLoadingForTesting(
       GURL("http://www.google.com"), GURL("http://www.not-google.com/")));
   EXPECT_TRUE(ArcNavigationThrottle::ShouldOverrideUrlLoadingForTesting(
       GURL("http://www.not-google.com"), GURL("http://www.google.com/")));
+
+  // A navigation with neither an http nor https scheme cannot be overriden.

[Yusuke Sato, 2016/10/31 20:26:51]

Thanks!

+  EXPECT_FALSE(GURL("chrome-extension://fake_document"),
+               GURL("http://www.a.com"));
+  EXPECT_FALSE(GURL("http://www.a.com"),
+               GURL("chrome-extension://fake_document"));
+  EXPECT_FALSE(GURL("chrome-extension://fake_document"),
+               GURL("https://www.a.com"));
+  EXPECT_FALSE(GURL("https://www.a.com"),
+               GURL("chrome-extension://fake_document"));
+  EXPECT_FALSE(GURL("chrome-extension://fake_a"),
+               GURL("chrome-extension://fake_b"));
 }
 
 TEST(ArcNavigationThrottleTest, TestIsAppAvailable) {
   // Test an empty array.
   EXPECT_FALSE(
       ArcNavigationThrottle::IsAppAvailableForTesting(CreateArray(0, 0)));
   // Chrome only.
   EXPECT_FALSE(
       ArcNavigationThrottle::IsAppAvailableForTesting(CreateArray(1, 0)));
   // Chrome and another app.
@@ skipping 101 matching lines @@
   for (size_t i = 0; i <= ArcNavigationThrottle::kMaxAppResults * 2; ++i) {
     // When Chrome does not exist in |handlers|, swap is unnecessary.
     mojo::Array<mojom::IntentHandlerInfoPtr> handlers = CreateArray(i, i);
     EXPECT_FALSE(ArcNavigationThrottle::IsSwapElementsNeededForTesting(
         handlers, &indices))
         << i;
   }
 }
 
 }  // namespace arc