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

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

Issue 2284933002: Remove OfflineURL from offline page (Closed)
Patch Set: Add comment per feedback Created 4 years, 2 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 fb69cbf920025baad3560fc4526498e1ad435ad8..f5fac023e4e1cefc7cb6007091f2a42ebdf19a78 100644
--- a/chrome/browser/android/offline_pages/offline_page_utils.h
+++ b/chrome/browser/android/offline_pages/offline_page_utils.h
@@ -22,9 +22,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
@@ -36,26 +33,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