Enable download page action for error page

chrome/browser/net/net_error_tab_helper_unittest.cc

 // Copyright 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 "chrome/browser/net/net_error_tab_helper.h"
 
 #include "chrome/common/render_messages.h"
 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
 #include "components/error_page/common/net_error_info.h"
 #include "content/public/browser/browser_thread.h"
@@ skipping 26 matching lines @@
     EXPECT_TRUE(mock_probe_running_);
     OnDnsProbeFinished(status);
     mock_probe_running_ = false;
   }
 
   bool mock_probe_running() const { return mock_probe_running_; }
   DnsProbeStatus last_status_sent() const { return last_status_sent_; }
   int mock_sent_count() const { return mock_sent_count_; }
 
 #if defined(OS_ANDROID)
-  using NetErrorTabHelper::DownloadPageLater;
+  using NetErrorTabHelper::OnDownloadPageLater;
 
   const GURL& download_page_later_url() const {
     return download_page_later_url_;
   }
 
   int times_download_page_later_invoked() const {
     return times_download_page_later_invoked_;
   }
 #endif  // defined(OS_ANDROID)
 
@@ skipping 110 matching lines @@
       content::RenderFrameHostTester::For(main_rfh())->
           SimulateNavigationErrorPageCommit();
     }
   }
 
 #if defined(OS_ANDROID)
   void NoDownloadPageLaterForNonHttpSchemes(const char* url_string,
                                             bool succeeded) {
     GURL url(url_string);
     LoadURL(url, succeeded);
-    tab_helper()->DownloadPageLater();
+    tab_helper()->OnDownloadPageLater();
     EXPECT_EQ(0, tab_helper()->times_download_page_later_invoked());
   }
 #endif
 
   bool probe_running() { return tab_helper_->mock_probe_running(); }
   DnsProbeStatus last_status_sent() { return tab_helper_->last_status_sent(); }
   int sent_count() { return tab_helper_->mock_sent_count(); }
 
   TestNetErrorTabHelper* tab_helper() { return tab_helper_.get(); }
 
@@ skipping 180 matching lines @@
     tab_helper()->network_diagnostics_interface()
         ->RunNetworkDiagnostics(GURL(url));
     EXPECT_EQ(0, tab_helper()->times_diagnostics_dialog_invoked());
   }
 }
 
 #if defined(OS_ANDROID)
 TEST_F(NetErrorTabHelperTest, DownloadPageLater) {
   GURL url("http://somewhere:123/");
   LoadURL(url, false /*succeeded*/);
-  tab_helper()->DownloadPageLater();
+  tab_helper()->OnDownloadPageLater();
   EXPECT_EQ(url, tab_helper()->download_page_later_url());
   EXPECT_EQ(1, tab_helper()->times_download_page_later_invoked());
 }
 
 TEST_F(NetErrorTabHelperTest, NoDownloadPageLaterOnNonErrorPage) {
   GURL url("http://somewhere:123/");
   LoadURL(url, true /*succeeded*/);
-  tab_helper()->DownloadPageLater();
+  tab_helper()->OnDownloadPageLater();
   EXPECT_EQ(0, tab_helper()->times_download_page_later_invoked());
 }
 
 // Makes sure that "Download page later" isn't run on URLs with non-HTTP/HTTPS
 // schemes.
 // NOTE: the test harness code in this file and in TestRendererHost don't always
 // deal with pending RFH correctly. This works because most tests only load
 // once. So workaround it by puting each test case in a separate test.
 TEST_F(NetErrorTabHelperTest, NoDownloadPageLaterForNonHttpSchemes1) {
   NoDownloadPageLaterForNonHttpSchemes("file:///blah/blah", false);
 }
 
 TEST_F(NetErrorTabHelperTest, NoDownloadPageLaterForNonHttpSchemes2) {
   NoDownloadPageLaterForNonHttpSchemes("chrome://blah/", false);
 }
 
 TEST_F(NetErrorTabHelperTest, NoDownloadPageLaterForNonHttpSchemes3) {
   // about:blank always succeeds, and the test harness won't handle URLs that
   // don't go to the network failing.
   NoDownloadPageLaterForNonHttpSchemes("about:blank", true);
 }
 
 #endif  // defined(OS_ANDROID)