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

Side by Side Diff: components/offline_pages/offline_page_model.cc

Issue 2469933002: Offline Pages: Replace Observer::OfflinePageModelChanged with OfflinePageAdded. (Closed)
Patch Set: Touch ups. Created 4 years, 1 month 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/offline_pages/offline_page_model.h" 5 #include "components/offline_pages/offline_page_model.h"
6 6
7 #include "url/gurl.h" 7 #include "url/gurl.h"
8 8
9 namespace offline_pages { 9 namespace offline_pages {
10 using Observer = OfflinePageModel::Observer;
11
12 // Invoked when the model has finished loading.
13 void Observer::OfflinePageModelLoaded(OfflinePageModel* model) {}
14
15 // Invoked when the model is being updated due to adding an offline page.
16 void Observer::OfflinePageAdded(OfflinePageModel* model,
17 SavePageResult result,
18 const OfflinePageItem* added) {}
19
20 // Invoked when an offline copy related to |offline_id| was deleted.
21 void Observer::OfflinePageDeleted(int64_t offline_id,
22 const ClientId& client_id) {}
23
24 Observer::~Observer() = default;
10 25
11 // static 26 // static
12 bool OfflinePageModel::CanSaveURL(const GURL& url) { 27 bool OfflinePageModel::CanSaveURL(const GURL& url) {
13 return url.is_valid() && url.SchemeIsHTTPOrHTTPS(); 28 return url.is_valid() && url.SchemeIsHTTPOrHTTPS();
14 } 29 }
15 30
16 OfflinePageModel::OfflinePageModel() {} 31 OfflinePageModel::OfflinePageModel() {}
17 32
18 OfflinePageModel::~OfflinePageModel() {} 33 OfflinePageModel::~OfflinePageModel() {}
19 34
20 } // namespace offline_pages 35 } // namespace offline_pages
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698