Add an error page for HTTP 404 error pages without bodies.

chrome/browser/errorpage_browsertest.cc

 // Copyright (c) 2012 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 <memory>
 #include <utility>
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
@@ skipping 86 matching lines @@
   return result;
 }
 
 // Expands the more box on the currently displayed error page.
 void ToggleHelpBox(Browser* browser) {
   EXPECT_TRUE(content::ExecuteScript(
       browser->tab_strip_model()->GetActiveWebContents(),
       "document.getElementById('details-button').click();"));
 }
 
-// Returns true if |browser| is displaying the text representation of
-// |error_code| on the current page.
-bool WARN_UNUSED_RESULT IsDisplayingNetError(Browser* browser,
-                                             net::Error error_code) {
-  return IsDisplayingText(browser, net::ErrorToShortString(error_code));
-}
-
 // Returns true if the diagnostics link suggestion is displayed.
 bool WARN_UNUSED_RESULT IsDisplayingDiagnosticsLink(Browser* browser) {
   std::string command = base::StringPrintf(
       "var diagnose_link = document.getElementById('diagnose-link');"
       "domAutomationController.send(diagnose_link != null);");
   bool result = false;
   EXPECT_TRUE(content::ExecuteScriptAndExtractBool(
       browser->tab_strip_model()->GetActiveWebContents(), command, &result));
   return result;
 }
 
 // Checks that the local error page is being displayed, without remotely
-// retrieved navigation corrections, and with the specified error code.
-void ExpectDisplayingLocalErrorPage(Browser* browser, net::Error error_code) {
-  EXPECT_TRUE(IsDisplayingNetError(browser, error_code));
+// retrieved navigation corrections, and with the specified error string.
+void ExpectDisplayingLocalErrorPage(Browser* browser,
+                                    const std::string& error_string) {
+  EXPECT_TRUE(IsDisplayingText(browser, error_string));
 
   // Locally generated error pages should not have navigation corrections.
   EXPECT_FALSE(IsDisplayingText(browser, "http://mock.http/title2.html"));
 
   // Locally generated error pages should not have a link with search terms.
   EXPECT_FALSE(IsDisplayingText(browser, "search query"));
 }
 
+// Checks that the local error page is being displayed, without remotely
+// retrieved navigation corrections, and with the specified error code.
+void ExpectDisplayingLocalErrorPage(Browser* browser, net::Error error_code) {
+  ExpectDisplayingLocalErrorPage(browser, net::ErrorToShortString(error_code));
+}
+
 // Checks that an error page with information retrieved from the navigation
 // correction service is being displayed, with the specified specified error
-// code.
+// string.
 void ExpectDisplayingNavigationCorrections(Browser* browser,
-                                           net::Error error_code) {
-  EXPECT_TRUE(IsDisplayingNetError(browser, error_code));
+                                           const std::string& error_string) {
+  EXPECT_TRUE(IsDisplayingText(browser, error_string));
 
   // Check that the mock navigation corrections are displayed.
   EXPECT_TRUE(IsDisplayingText(browser, "http://mock.http/title2.html"));
 
   // Check that the search terms are displayed as a link.
   EXPECT_TRUE(IsDisplayingText(browser, "search query"));
 
   // The diagnostics button isn't displayed when corrections were
   // retrieved from a remote server.
   EXPECT_FALSE(IsDisplayingDiagnosticsLink(browser));
 }
 
+// Checks that an error page with information retrieved from the navigation
+// correction service is being displayed, with the specified specified error
+// code.
+void ExpectDisplayingNavigationCorrections(Browser* browser,
+                                           net::Error error_code) {
+  ExpectDisplayingNavigationCorrections(browser,
+                                        net::ErrorToShortString(error_code));
+}
+
 std::string GetShowSavedButtonLabel() {
   return l10n_util::GetStringUTF8(IDS_ERRORPAGES_BUTTON_SHOW_SAVED_COPY);
 }
 
 void AddInterceptorForURL(const GURL& url,
                           std::unique_ptr<net::URLRequestInterceptor> handler) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   net::URLRequestFilter::GetInstance()->AddUrlInterceptor(url,
                                                           std::move(handler));
 }
@@ skipping 755 matching lines @@
 }
 
 // Checks that navigation corrections are not loaded when we receive an actual
 // 404 page.
 IN_PROC_BROWSER_TEST_F(ErrorPageTest, Page404) {
   NavigateToURLAndWaitForTitle(
       net::URLRequestMockHTTPJob::GetMockUrl("page404.html"), "SUCCESS", 1);
   EXPECT_EQ(0, link_doctor_interceptor()->num_requests());
 }
 
+// Checks that navigation corrections are loaded when we get a 404 page with no
+// body.
+IN_PROC_BROWSER_TEST_F(ErrorPageTest, Empty404) {
+  // The first navigation should fail and load a blank page, while it fetches
+  // the Link Doctor response.  The second navigation is the Link Doctor.
+  ui_test_utils::NavigateToURLBlockUntilNavigationsComplete(
+      browser(),
+      net::URLRequestMockHTTPJob::GetMockUrl("errorpage/empty404.html"), 2);
+  // This depends on the non-internationalized error ID string in
+  // localized_error.cc.
+  ExpectDisplayingNavigationCorrections(browser(), "HTTP ERROR 404");
+  EXPECT_EQ(1, link_doctor_interceptor()->num_requests());
+}
+
+// Checks that navigation corrections are loaded when we get a 404 page with no
+// body.

[Randy Smith (Not in Mondays), 2016/08/01 18:59:50]

Is this comment correct?  It seems to be copied from the above and not match the
test title/HTML.

[mmenke, 2016/08/01 20:18:28]

Done.

+IN_PROC_BROWSER_TEST_F(ErrorPageTest, Empty500) {
+  NavigateToFileURL("errorpage/empty500.html");

[Randy Smith (Not in Mondays), 2016/08/01 18:59:50]

I'm not familiar with the underlying logic of the helper functions, but does
using NavigateToFileURL() risk not noticing if there's a two phase navigation as
in the last test (i.e. if we load something then fail over to link doctor and
load the link doctor response as a second navigation)?  Or is the timing such
that that will always be caught (without races) by the EXPECT_EQ below?

[mmenke, 2016/08/01 20:18:28]

If there's a two phase navigation, we load a blank page, and then the link
doctor page.  So unless that changes significantly, we'll either have a blank
page or a link doctor page, both of which will fail.

I'm pretty sure that it will quit the run loop after the first navigation (i.e.,
it doesn't spin the run loop after the navigation), but I'm not positive of
that.

+  // This depends on the non-internationalized error ID string in
+  // localized_error.cc.
+  ExpectDisplayingLocalErrorPage(browser(), "HTTP ERROR 500");
+  EXPECT_EQ(0, link_doctor_interceptor()->num_requests());
+}
+
 // Checks that when an error occurs, the stale cache status of the page
 // is correctly transferred, and that stale cached copied can be loaded
 // from the javascript.
 IN_PROC_BROWSER_TEST_F(ErrorPageTest, StaleCacheStatus) {
   ASSERT_TRUE(embedded_test_server()->Start());
   // Load cache with entry with "nocache" set, to create stale
   // cache.
   GURL test_url(embedded_test_server()->GetURL("/nocache.html"));
   NavigateToURLAndWaitForTitle(test_url, "Nocache Test Page", 1);
 
@@ skipping 515 matching lines @@
 IN_PROC_BROWSER_TEST_F(ErrorPageForIDNTest, IDN) {
   // ERR_UNSAFE_PORT will not trigger navigation corrections.
   ui_test_utils::NavigateToURL(
       browser(),
       URLRequestFailedJob::GetMockHttpUrlForHostname(net::ERR_UNSAFE_PORT,
                                                      kHostname));
   EXPECT_TRUE(IsDisplayingText(browser(), kHostnameJSUnicode));
 }
 
 }  // namespace