Fall back to local page if online NTP fails to load.

chrome/browser/search/search_unittest.cc

 // Copyright (c) 2013 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 "base/command_line.h"
 #include "base/metrics/field_trial.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/browser/search/search.h"
 #include "chrome/browser/search_engines/template_url_service.h"
 #include "chrome/browser/search_engines/template_url_service_factory.h"
@@ skipping 281 matching lines @@
   };
 
   for (size_t i = 0; i < arraysize(kTestCases); ++i) {
     const SearchTestCase& test = kTestCases[i];
     EXPECT_EQ(test.expected_result,
               ShouldAssignURLToInstantRenderer(GURL(test.url), profile()))
         << test.url << " " << test.comment;
   }
 }
 
+TEST_F(SearchTest, ShouldAssignURLToInstantRendererExtendedEnabledOverridden) {
+  EnableInstantExtendedAPIForTesting();
+
+  // Override the instant url with the command line flag.
+  CommandLine::ForCurrentProcess()->AppendSwitchASCII(switches::kInstantURL,
+      "http://bar.com/searchingfast");
+
+  // Existing Instant URLs should all work, plus new ones matching the override.
+  const SearchTestCase kTestCases[] = {
+    {chrome::kChromeSearchLocalNtpUrl, true,  ""},
+    {chrome::kChromeSearchLocalGoogleNtpUrl, true,  ""},
+    {"https://foo.com/instant?strk",      true,  ""},
+    {"https://foo.com/instant#strk",      true,  ""},
+    {"https://foo.com/instant?strk=0",    true,  ""},
+    {"https://foo.com/url?strk",          true,  ""},
+    {"https://foo.com/alt?strk",          true,  ""},
+    {"http://bar.com/searchingfast",      true,  ""},
+    {"http://bar.com/alt?strk",           true,  ""},
+    {"https://bar.com/searchingfast",     true,  ""},
+    {"http://foo.com/instant",            false, "Non-HTTPS"},
+    {"http://foo.com/instant?strk",       false, "Non-HTTPS"},
+    {"http://foo.com/instant?strk=1",     false, "Non-HTTPS"},
+    {"https://foo.com/instant",           false, "No search terms replacement"},
+    {"https://foo.com/?strk",             false, "Non-exact path"},
+    {"https://www.bar.com/searchingfast", false,  "Host mismatch"},
+  };
+
+  for (size_t i = 0; i < arraysize(kTestCases); ++i) {
+    const SearchTestCase& test = kTestCases[i];
+    EXPECT_EQ(test.expected_result,
+              ShouldAssignURLToInstantRenderer(GURL(test.url), profile()))
+        << test.url << " " << test.comment;
+  }
+}

[sreeram, 2013/05/03 23:17:31]

Given my comment earlier about not doing "return true" for the --instant-url, I
don't see the need for this new test at all.

[David Black, 2013/05/04 00:49:23]

Done.

+
 TEST_F(SearchTest, CoerceCommandLineURLToTemplateURL) {
   TemplateURL* template_url =
       TemplateURLServiceFactory::GetForProfile(profile())->
           GetDefaultSearchProvider();
   EXPECT_EQ(
       GURL("https://foo.com/dev?bar=bar#bar=bar"),
       CoerceCommandLineURLToTemplateURL(
           GURL("http://myserver.com:9000/dev?bar=bar#bar=bar"),
           template_url->instant_url_ref(), kDisableStartMargin));
 }
@@ skipping 254 matching lines @@
   profile()->GetPrefs()->SetBoolean(prefs::kSearchSuggestEnabled, true);
 
   // Now that suggest is back on and the instant url is good, the checkbox
   // should be enabled and checked again.
   EXPECT_TRUE(IsInstantCheckboxEnabled(profile()));
   EXPECT_TRUE(IsInstantCheckboxChecked(profile()));
 }
 
 
 }  // namespace chrome