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

Unified Diff: content/public/browser/navigation_controller.h

Issue 2561983002: NavigationController: Reload methods migration (Closed)
Patch Set: merge master Created 4 years 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: content/public/browser/navigation_controller.h
diff --git a/content/public/browser/navigation_controller.h b/content/public/browser/navigation_controller.h
index 9c7d8c18d53c409eae905be114a33eeb04e0e61e..7ec3ca82f17a394952d5eb8222fb78e7f09d6567 100644
--- a/content/public/browser/navigation_controller.h
+++ b/content/public/browser/navigation_controller.h
@@ -350,29 +350,13 @@ class NavigationController {
// the offset is out of bounds.
virtual void GoToOffset(int offset) = 0;
- // Reloads the current entry under the specified ReloadType.
- // TODO(toyoshim): Change all callers to use this new Reload(), and remove
- // old Reload* methods below. One motivation of this change is Reload(bool)
- // interface is just confusing because in some contexts, the bool could be to
- // specify bypassing cache. http://crbug.com/670232
- virtual void Reload(bool check_for_repost, ReloadType reload_type) = 0;
-
- // Reloads the current entry. If |check_for_repost| is true and the current
- // entry has POST data the user is prompted to see if they really want to
- // reload the page. In nearly all cases pass in true. If a transient entry
- // is showing, initiates a new navigation to its URL.
- virtual void Reload(bool check_for_repost) = 0;
-
- // Like Reload(), but don't use caches (aka "shift-reload").
- virtual void ReloadBypassingCache(bool check_for_repost) = 0;
-
- // Reloads the current entry using the original URL used to create it. This
- // is used for cases where the user wants to refresh a page using a different
- // user agent after following a redirect.
- virtual void ReloadOriginalRequestURL(bool check_for_repost) = 0;
-
- // Like Reload(), but disables Lo-Fi.
- virtual void ReloadDisableLoFi(bool check_for_repost) = 0;
+ // Reloads the current entry under the specified ReloadType. If
+ // |check_for_repost| is true and the current entry has POST data the user is
+ // prompted to see if they really want to reload the page. In nearly all
+ // cases pass in true in production code, but would do false for testing, or
+ // in cases where no user interface is available for prompting. If a
+ // transient entry is showing, initiates a new navigation to its URL.
+ virtual void Reload(ReloadType reload_type, bool check_for_repost) = 0;
// Removing of entries -------------------------------------------------------
« no previous file with comments | « content/browser/web_contents/web_contents_view_aura_browsertest.cc ('k') | content/public/browser/reload_type.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698