| Index: chrome/renderer/net/net_error_helper_core_unittest.cc
|
| diff --git a/chrome/renderer/net/net_error_helper_core_unittest.cc b/chrome/renderer/net/net_error_helper_core_unittest.cc
|
| index f41da625c8a2d8a6ee29229811a3862c7728a885..f30030e65713332b7b53b7fa9c65458a81ed0f75 100644
|
| --- a/chrome/renderer/net/net_error_helper_core_unittest.cc
|
| +++ b/chrome/renderer/net/net_error_helper_core_unittest.cc
|
| @@ -119,7 +119,8 @@ class NetErrorHelperCoreTest : public testing::Test,
|
| update_count_(0),
|
| error_html_update_count_(0),
|
| reload_count_(0),
|
| - enable_stale_load_bindings_count_(0) {
|
| + load_stale_count_(0),
|
| + enable_page_helper_functions_count_(0) {
|
| core_.set_auto_reload_enabled(false);
|
| core_.set_timer_for_testing(scoped_ptr<base::Timer>(timer_));
|
| }
|
| @@ -138,8 +139,16 @@ class NetErrorHelperCoreTest : public testing::Test,
|
| return reload_count_;
|
| }
|
|
|
| - int enable_stale_load_bindings_count() const {
|
| - return enable_stale_load_bindings_count_;
|
| + int load_stale_count() const {
|
| + return load_stale_count_;
|
| + }
|
| +
|
| + const GURL& load_stale_url() const {
|
| + return load_stale_url_;
|
| + }
|
| +
|
| + int enable_page_helper_functions_count() const {
|
| + return enable_page_helper_functions_count_;
|
| }
|
|
|
| const std::string& last_update_string() const { return last_update_string_; }
|
| @@ -225,8 +234,12 @@ class NetErrorHelperCoreTest : public testing::Test,
|
| const WebURLError& error,
|
| bool is_failed_post,
|
| scoped_ptr<LocalizedError::ErrorPageParams> params,
|
| + bool* reload_button_shown,
|
| + bool* load_stale_button_shown,
|
| std::string* html) const OVERRIDE {
|
| last_error_page_params_.reset(params.release());
|
| + *reload_button_shown = false;
|
| + *load_stale_button_shown = false;
|
| *html = ErrorToString(error, is_failed_post);
|
| }
|
|
|
| @@ -236,8 +249,8 @@ class NetErrorHelperCoreTest : public testing::Test,
|
| last_error_html_ = html;
|
| }
|
|
|
| - virtual void EnableStaleLoadBindings(const GURL& page_url) OVERRIDE {
|
| - enable_stale_load_bindings_count_++;
|
| + virtual void EnablePageHelperFunctions() OVERRIDE {
|
| + enable_page_helper_functions_count_++;
|
| }
|
|
|
| virtual void UpdateErrorPage(const WebURLError& error,
|
| @@ -267,6 +280,11 @@ class NetErrorHelperCoreTest : public testing::Test,
|
| reload_count_++;
|
| }
|
|
|
| + virtual void LoadPageFromCache(const GURL& error_url) OVERRIDE {
|
| + load_stale_count_++;
|
| + load_stale_url_ = error_url;
|
| + }
|
| +
|
| base::MockTimer* timer_;
|
|
|
| NetErrorHelperCore core_;
|
| @@ -289,8 +307,10 @@ class NetErrorHelperCoreTest : public testing::Test,
|
| mutable scoped_ptr<LocalizedError::ErrorPageParams> last_error_page_params_;
|
|
|
| int reload_count_;
|
| + int load_stale_count_;
|
| + GURL load_stale_url_;
|
|
|
| - int enable_stale_load_bindings_count_;
|
| + int enable_page_helper_functions_count_;
|
| };
|
|
|
| //------------------------------------------------------------------------------
|
| @@ -334,14 +354,14 @@ TEST_F(NetErrorHelperCoreTest, MainFrameNonDnsError) {
|
| EXPECT_EQ(NetErrorString(net::ERR_CONNECTION_RESET), html);
|
|
|
| // Error page loads.
|
| - EXPECT_EQ(0, enable_stale_load_bindings_count());
|
| + EXPECT_EQ(0, enable_page_helper_functions_count());
|
| core().OnStartLoad(NetErrorHelperCore::MAIN_FRAME,
|
| NetErrorHelperCore::ERROR_PAGE);
|
| core().OnCommitLoad(NetErrorHelperCore::MAIN_FRAME);
|
| core().OnFinishLoad(NetErrorHelperCore::MAIN_FRAME);
|
| EXPECT_EQ(0, update_count());
|
| EXPECT_EQ(0, error_html_update_count());
|
| - EXPECT_EQ(1, enable_stale_load_bindings_count());
|
| + EXPECT_EQ(1, enable_page_helper_functions_count());
|
| }
|
|
|
| TEST_F(NetErrorHelperCoreTest, MainFrameNonDnsErrorWithCorrections) {
|
| @@ -739,13 +759,13 @@ TEST_F(NetErrorHelperCoreTest, FinishedBeforeProbePost) {
|
| html);
|
|
|
| // Error page loads.
|
| - EXPECT_EQ(0, enable_stale_load_bindings_count());
|
| + EXPECT_EQ(0, enable_page_helper_functions_count());
|
| core().OnStartLoad(NetErrorHelperCore::MAIN_FRAME,
|
| NetErrorHelperCore::ERROR_PAGE);
|
| core().OnCommitLoad(NetErrorHelperCore::MAIN_FRAME);
|
| core().OnFinishLoad(NetErrorHelperCore::MAIN_FRAME);
|
| EXPECT_EQ(0, update_count());
|
| - EXPECT_EQ(0, enable_stale_load_bindings_count());
|
| + EXPECT_EQ(1, enable_page_helper_functions_count());
|
|
|
| core().OnNetErrorInfo(chrome_common_net::DNS_PROBE_STARTED);
|
| EXPECT_EQ(1, update_count());
|
| @@ -1984,3 +2004,20 @@ TEST_F(NetErrorHelperCoreTest, ShouldSuppressErrorPage) {
|
| EXPECT_TRUE(core().ShouldSuppressErrorPage(NetErrorHelperCore::MAIN_FRAME,
|
| GURL(kFailedUrl)));
|
| }
|
| +
|
| +TEST_F(NetErrorHelperCoreTest, ExplicitReloadSucceeds) {
|
| + DoErrorLoad(net::ERR_CONNECTION_RESET);
|
| + EXPECT_EQ(0, reload_count());
|
| + core().ExecuteButtonPress(NetErrorHelperCore::RELOAD_BUTTON);
|
| + EXPECT_EQ(1, reload_count());
|
| +}
|
| +
|
| +TEST_F(NetErrorHelperCoreTest, ExplicitLoadStaleSucceeds) {
|
| + DoErrorLoad(net::ERR_CONNECTION_RESET);
|
| + EXPECT_EQ(0, load_stale_count());
|
| + core().ExecuteButtonPress(NetErrorHelperCore::LOAD_STALE_BUTTON);
|
| + EXPECT_EQ(1, load_stale_count());
|
| + EXPECT_EQ(GURL(kFailedUrl), load_stale_url());
|
| +}
|
| +
|
| +
|
|
|