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

Unified Diff: chrome/browser/android/offline_pages/offline_page_utils.h

Issue 2284933002: Remove OfflineURL from offline page (Closed)
Patch Set: Fix trybot Created 4 years, 4 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/android/offline_pages/offline_page_utils.h
diff --git a/chrome/browser/android/offline_pages/offline_page_utils.h b/chrome/browser/android/offline_pages/offline_page_utils.h
index 5e74803cb26fae1be1b8cfe8b4312c1d3c0864b7..2ba09faffb6839599f35039c37606503d51b4905 100644
--- a/chrome/browser/android/offline_pages/offline_page_utils.h
+++ b/chrome/browser/android/offline_pages/offline_page_utils.h
@@ -21,9 +21,6 @@ struct OfflinePageItem;
class OfflinePageUtils {
public:
- // Returns true if |url| might point to an offline page.
- static bool MightBeOfflineURL(const GURL& url);
-
// Returns via callback an offline page saved for |online_url|, if any. The
// page is chosen based on creation date; a more recently created offline
// page will be preferred over an older one. The offline page captured from
@@ -35,26 +32,6 @@ class OfflinePageUtils {
int tab_id,
const base::Callback<void(const OfflinePageItem*)>& callback);
- // Gets an online URL of an offline page with |offline_url| if one exists.
- // Deprecated. Use |GetOnlineURLForOfflineURL|.
- static GURL MaybeGetOnlineURLForOfflineURL(
- content::BrowserContext* browser_context,
- const GURL& offline_url);
-
- static void GetOnlineURLForOfflineURL(
- content::BrowserContext* browser_context,
- const GURL& offline_url,
- const base::Callback<void(const GURL&)>& callback);
-
- // Checks whether |offline_url| points to an offline page.
- // Deprecated. Use something else.
- static bool IsOfflinePage(content::BrowserContext* browser_context,
- const GURL& offline_url);
-
- // Marks that the offline page related to the |offline_url| has been accessed.
- static void MarkPageAccessed(content::BrowserContext* browser_context,
- const GURL& offline_url);
-
// Gets the offline page corresponding to the given web contents. The
// returned pointer is owned by the web_contents and may be deleted by user
// navigation, so it is unsafe to store a copy of the returned pointer.

Powered by Google App Engine
This is Rietveld 408576698