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

Side by Side Diff: components/reading_list/ios/favicon_web_state_dispatcher.h

Issue 2604773002: Create distiller files for Reading List. (Closed)
Patch Set: feedback Created 3 years, 11 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 unified diff | Download patch
« no previous file with comments | « components/reading_list/ios/BUILD.gn ('k') | ios/chrome/browser/dom_distiller/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 #ifndef COMPONENTS_DOM_DISTILLER_IOS_FAVICON_WEB_STATE_DISPATCHER_H_ 5 #ifndef COMPONENTS_READING_LIST_IOS_FAVICON_WEB_STATE_DISPATCHER_H_
6 #define COMPONENTS_DOM_DISTILLER_IOS_FAVICON_WEB_STATE_DISPATCHER_H_ 6 #define COMPONENTS_READING_LIST_IOS_FAVICON_WEB_STATE_DISPATCHER_H_
7 7
8 namespace web { 8 namespace web {
9 class WebState; 9 class WebState;
10 } 10 }
11 11
12 namespace dom_distiller { 12 namespace reading_list {
13 13
14 // Dispatcher for WebState having a Favicon Driver, with BookmarkModel and 14 // Dispatcher for WebState having a Favicon Driver, with BookmarkModel and
15 // HistoryService attached, as observer. The Webstates are kept alive between 15 // HistoryService attached, as observer. After a WebState is returned, the
16 // their creation and their return. After a WebState is returned, the dispatcher 16 // dispatcher keeps it alive long enough for it to download the favicons.
17 // keeps it alive long enough for it to download the favicons.
18 class FaviconWebStateDispatcher { 17 class FaviconWebStateDispatcher {
19 public: 18 public:
20 FaviconWebStateDispatcher() {} 19 FaviconWebStateDispatcher() {}
21 virtual ~FaviconWebStateDispatcher() {} 20 virtual ~FaviconWebStateDispatcher() {}
22 // Returns a WebState with a Favicon Driver attached. 21 // Returns a WebState with a Favicon Driver attached.
23 virtual web::WebState* RequestWebState() = 0; 22 virtual std::unique_ptr<web::WebState> RequestWebState() = 0;
24 // Called to return a WebState. The WebState should not be used after being 23 // Called to return a WebState. The WebState should not be used after being
25 // returned. 24 // returned.
26 virtual void ReturnWebState(web::WebState* web_state) = 0; 25 virtual void ReturnWebState(std::unique_ptr<web::WebState> web_state) = 0;
27 26
28 private: 27 private:
29 DISALLOW_COPY_AND_ASSIGN(FaviconWebStateDispatcher); 28 DISALLOW_COPY_AND_ASSIGN(FaviconWebStateDispatcher);
30 }; 29 };
31 30
32 } // namespace dom_distiller 31 } // namespace reading_list
33 32
34 #endif // COMPONENTS_DOM_DISTILLER_IOS_FAVICON_WEB_STATE_DISPATCHER_H_ 33 #endif // COMPONENTS_READING_LIST_IOS_FAVICON_WEB_STATE_DISPATCHER_H_
OLDNEW
« no previous file with comments | « components/reading_list/ios/BUILD.gn ('k') | ios/chrome/browser/dom_distiller/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698