Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1409)

Unified Diff: chrome/browser/net/net_error_tab_helper.h

Issue 2588253002: Enable download page action for error page (Closed)
Patch Set: Address download feedback Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/net/net_error_tab_helper.h
diff --git a/chrome/browser/net/net_error_tab_helper.h b/chrome/browser/net/net_error_tab_helper.h
index 42b2abe17b2e4b651dbd5605a87daeaba540a511..3a03feef1db3106bb4be7ef67470e763c13f567b 100644
--- a/chrome/browser/net/net_error_tab_helper.h
+++ b/chrome/browser/net/net_error_tab_helper.h
@@ -51,6 +51,12 @@ class NetErrorTabHelper
dns_probe_status_snoop_callback_ = dns_probe_status_snoop_callback;
}
+#if defined(OS_ANDROID)
+ bool is_showing_download_button_in_error_page() const {
+ return is_showing_download_button_in_error_page_;
+ }
+#endif // defined(OS_ANDROID)
+
// content::WebContentsObserver implementation.
void RenderFrameCreated(content::RenderFrameHost* render_frame_host) override;
void DidStartNavigation(
@@ -78,7 +84,8 @@ class NetErrorTabHelper
}
#if defined(OS_ANDROID)
- void DownloadPageLater();
+ void OnDownloadPageLater();
+ void OnSetIsShowingDownloadButtonInErrorPage(bool is_showing_download_button);
#endif // defined(OS_ANDROID)
private:
@@ -115,6 +122,11 @@ class NetErrorTabHelper
// is true. (This should never be true if |dns_error_active_| is false.)
bool dns_error_page_committed_;
+#if defined(OS_ANDROID)
+ // True if download button is being shown when the error page commits.
+ bool is_showing_download_button_in_error_page_;
+#endif // defined(OS_ANDROID)
+
// The status of a DNS probe that may or may not have started or finished.
// Since the renderer can change out from under the helper (in cross-process
// navigations), it re-sends the status whenever an error page commits.
« no previous file with comments | « chrome/browser/android/offline_pages/offline_page_utils.cc ('k') | chrome/browser/net/net_error_tab_helper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698