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

Unified Diff: components/offline_pages/stub_offline_page_model.h

Issue 2429943002: Remove all synchronous methods from OfflinePageBridge. (Closed)
Patch Set: Fix a test and address nits. 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/stub_offline_page_model.h
diff --git a/components/offline_pages/stub_offline_page_model.h b/components/offline_pages/stub_offline_page_model.h
index 09925f8af08cf9faa261f655968c6c525bd904dc..55f0c0177d602bbc7ee7d149e28688ce5ea9e802 100644
--- a/components/offline_pages/stub_offline_page_model.h
+++ b/components/offline_pages/stub_offline_page_model.h
@@ -32,6 +32,11 @@ class StubOfflinePageModel : public OfflinePageModel, public KeyedService {
void MarkPageAccessed(int64_t offline_id) override;
void DeletePagesByOfflineId(const std::vector<int64_t>& offline_ids,
const DeletePageCallback& callback) override;
+ void DeletePagesByClientIds(const std::vector<ClientId>& client_ids,
+ const DeletePageCallback& callback) override;
+ void GetPagesByClientIds(
+ const std::vector<ClientId>& client_ids,
+ const MultipleOfflinePageItemCallback& callback) override;
void DeleteCachedPagesByURLPredicate(
const UrlPredicate& predicate,
const DeletePageCallback& callback) override;
@@ -44,18 +49,12 @@ class StubOfflinePageModel : public OfflinePageModel, public KeyedService {
void GetOfflineIdsForClientId(
const ClientId& client_id,
const MultipleOfflineIdCallback& callback) override;
- const std::vector<int64_t> MaybeGetOfflineIdsForClientId(
- const ClientId& client_id) const override;
void GetPageByOfflineId(
int64_t offline_id,
const SingleOfflinePageItemCallback& callback) override;
- const OfflinePageItem* MaybeGetPageByOfflineId(
- int64_t offline_id) const override;
void GetPagesByOnlineURL(
const GURL& online_url,
const MultipleOfflinePageItemCallback& callback) override;
- const OfflinePageItem* MaybeGetBestPageForOnlineURL(
- const GURL& online_url) const override;
void ExpirePages(const std::vector<int64_t>& offline_ids,
const base::Time& expiration_time,
const base::Callback<void(bool)>& callback) override;
« no previous file with comments | « components/offline_pages/offline_page_model_impl_unittest.cc ('k') | components/offline_pages/stub_offline_page_model.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698