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

Unified Diff: components/offline_pages/offline_page_model_impl.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_impl.h
diff --git a/components/offline_pages/offline_page_model_impl.h b/components/offline_pages/offline_page_model_impl.h
index 519f2b2b5d4b495a48fabc5d5295eef4c143d850..07d2a4709e8d2ebf5aca0705406a9987061a58d3 100644
--- a/components/offline_pages/offline_page_model_impl.h
+++ b/components/offline_pages/offline_page_model_impl.h
@@ -94,11 +94,6 @@ class OfflinePageModelImpl : public OfflinePageModel, public KeyedService {
const SingleOfflinePageItemCallback& callback) override;
const OfflinePageItem* MaybeGetPageByOfflineId(
int64_t offline_id) const override;
- void GetPageByOfflineURL(
- const GURL& offline_url,
- const SingleOfflinePageItemCallback& callback) override;
- const OfflinePageItem* MaybeGetPageByOfflineURL(
- const GURL& offline_url) const override;
void GetPagesByOnlineURL(
const GURL& online_url,
const MultipleOfflinePageItemCallback& callback) override;
@@ -150,11 +145,8 @@ class OfflinePageModelImpl : public OfflinePageModel, public KeyedService {
int64_t offline_id,
const SingleOfflinePageItemCallback& callback) const;
void GetPagesByOnlineURLWhenLoadDone(
- const GURL& offline_url,
+ const GURL& online_url,
const MultipleOfflinePageItemCallback& callback) const;
- void GetPageByOfflineURLWhenLoadDone(
- const GURL& offline_url,
- const SingleOfflinePageItemCallback& callback) const;
void MarkPageAccessedWhenLoadDone(int64_t offline_id);
void CheckMetadataConsistency();

Powered by Google App Engine
This is Rietveld 408576698