summaryrefslogtreecommitdiffstats
path: root/chrome/browser/chrome_content_browser_client_browsertest.cc
diff options
context:
space:
mode:
authorjered@chromium.org <jered@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-10-08 04:19:22 +0000
committerjered@chromium.org <jered@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-10-08 04:19:22 +0000
commit83b0c24c16bd8fabbfc6d4004a301d757db00bc1 (patch)
tree233f749f08b8447174e326ed664180a6e3fa00e5 /chrome/browser/chrome_content_browser_client_browsertest.cc
parent03e6a4eea1d75428b70c81cbec01b1a4d52f50ec (diff)
downloadchromium_src-83b0c24c16bd8fabbfc6d4004a301d757db00bc1.zip
chromium_src-83b0c24c16bd8fabbfc6d4004a301d757db00bc1.tar.gz
chromium_src-83b0c24c16bd8fabbfc6d4004a301d757db00bc1.tar.bz2
appcache ntp: Fallback to local, don't prerender.
GoogleBaseURL starts out as http and is only upgraded to https after search domain check, so we'll try to load http://<newtabpage> and fail. The failed load either 404s or shows the local new tab page, depending on when the search domain check finishes. Instead, if the new tab url isn't secure, prefer the the baked-in new tab page. Also fall back to the local ntp if the user has no search provider, if the user's search provider has no custom ntp, or if the user disables search suggestions. Also, don't prerender the old instant ntp anymore. TEST=manual,unit BUG=304184,304901,304397 Review URL: https://codereview.chromium.org/26024002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@227460 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/chrome_content_browser_client_browsertest.cc')
-rw-r--r--chrome/browser/chrome_content_browser_client_browsertest.cc4
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome/browser/chrome_content_browser_client_browsertest.cc b/chrome/browser/chrome_content_browser_client_browsertest.cc
index 3bb56a5..6dbd60c 100644
--- a/chrome/browser/chrome_content_browser_client_browsertest.cc
+++ b/chrome/browser/chrome_content_browser_client_browsertest.cc
@@ -109,7 +109,7 @@ class InstantNTPURLRewriteTest : public ChromeContentBrowserClientBrowserTest {
IN_PROC_BROWSER_TEST_F(InstantNTPURLRewriteTest,
UberURLHandler_InstantExtendedNewTabPage) {
const GURL url_original("chrome://newtab");
- const GURL url_rewritten("http://example.com/newtab");
+ const GURL url_rewritten("https://example.com/newtab");
InstallTemplateURLWithNewTabPage(url_rewritten);
ASSERT_TRUE(base::FieldTrialList::CreateFieldTrial("InstantExtended",
"Group1 use_cacheable_ntp:1"));
@@ -128,7 +128,7 @@ IN_PROC_BROWSER_TEST_F(ChromeContentBrowserClientBrowserTest,
// Don't do the kSingleProcess shenanigans here (see the dual test) because
// otherwise RenderViewImpl crashes in a paranoid fit on startup.
const GURL url_original("chrome://newtab");
- const GURL url_rewritten("http://example.com/newtab");
+ const GURL url_rewritten("https://example.com/newtab");
InstallTemplateURLWithNewTabPage(url_rewritten);
ASSERT_TRUE(base::FieldTrialList::CreateFieldTrial("InstantExtended",
"Group1 use_cacheable_ntp:1"));