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

Unified Diff: components/offline_pages/offline_page_model.h

Issue 2284933002: Remove OfflineURL from offline page (Closed)
Patch Set: Remove accidentally added new file during rebase 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: components/offline_pages/offline_page_model.h
diff --git a/components/offline_pages/offline_page_model.h b/components/offline_pages/offline_page_model.h
index 48cdad74b11ca32250255274b2574101bdf67c76..c1036b7b065f5d6a06c488481ead5ae6101cd485 100644
--- a/components/offline_pages/offline_page_model.h
+++ b/components/offline_pages/offline_page_model.h
@@ -141,20 +141,6 @@ class OfflinePageModel : public base::SupportsUserData {
virtual const OfflinePageItem* MaybeGetPageByOfflineId(
int64_t offline_id) const = 0;
- // Returns the offline page that is stored under |offline_url|, if any.
- virtual void GetPageByOfflineURL(
- const GURL& offline_url,
- const SingleOfflinePageItemCallback& callback) = 0;
-
- // Returns an offline page that is stored as |offline_url|. A nullptr is
- // returned if not found.
- //
- // This function is deprecated, and may return |nullptr| even if a page
- // exists, depending on the implementation details of OfflinePageModel.
- // Use |GetPageByOfflineURL| instead.
- virtual const OfflinePageItem* MaybeGetPageByOfflineURL(
- const GURL& offline_url) const = 0;
-
// Returns the offline pages that are stored under |online_url|.
virtual void GetPagesByOnlineURL(
const GURL& online_url,

Powered by Google App Engine
This is Rietveld 408576698