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

Side by Side Diff: components/favicon/ios/web_favicon_driver.h

Issue 2677993002: Use IOSImageDataFetcherWrapper for favicon (Closed)
Patch Set: Created 3 years, 10 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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_FAVICON_IOS_WEB_FAVICON_DRIVER_H_ 5 #ifndef COMPONENTS_FAVICON_IOS_WEB_FAVICON_DRIVER_H_
6 #define COMPONENTS_FAVICON_IOS_WEB_FAVICON_DRIVER_H_ 6 #define COMPONENTS_FAVICON_IOS_WEB_FAVICON_DRIVER_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "components/favicon/core/favicon_driver_impl.h" 9 #include "components/favicon/core/favicon_driver_impl.h"
10 #include "ios/web/public/web_state/web_state_observer.h" 10 #include "ios/web/public/web_state/web_state_observer.h"
11 #include "ios/web/public/web_state/web_state_user_data.h" 11 #include "ios/web/public/web_state/web_state_user_data.h"
12 12
13 namespace image_fetcher {
14 class IOSImageDataFetcherWrapper;
15 }
16
13 namespace web { 17 namespace web {
14 class WebState; 18 class WebState;
15 } 19 }
16 20
17 namespace favicon { 21 namespace favicon {
18 22
19 // WebFaviconDriver is an implementation of FaviconDriver that listen to 23 // WebFaviconDriver is an implementation of FaviconDriver that listen to
20 // WebState events to start download of favicons and to get informed when the 24 // WebState events to start download of favicons and to get informed when the
21 // favicon download has completed. 25 // favicon download has completed.
22 class WebFaviconDriver : public web::WebStateObserver, 26 class WebFaviconDriver : public web::WebStateObserver,
(...skipping 28 matching lines...) Expand all
51 bookmarks::BookmarkModel* bookmark_model); 55 bookmarks::BookmarkModel* bookmark_model);
52 ~WebFaviconDriver() override; 56 ~WebFaviconDriver() override;
53 57
54 // web::WebStateObserver implementation. 58 // web::WebStateObserver implementation.
55 void FaviconUrlUpdated( 59 void FaviconUrlUpdated(
56 const std::vector<web::FaviconURL>& candidates) override; 60 const std::vector<web::FaviconURL>& candidates) override;
57 61
58 // The URL passed to FetchFavicon(). 62 // The URL passed to FetchFavicon().
59 GURL fetch_favicon_url_; 63 GURL fetch_favicon_url_;
60 64
65 // Image Fetcher used to images.
Marc Treib 2017/02/06 12:35:49 This sentence no verb :)
gambard 2017/02/06 15:58:28 Done.
66 std::unique_ptr<image_fetcher::IOSImageDataFetcherWrapper> image_fetcher_;
Marc Treib 2017/02/06 12:35:49 nit: #include <memory>
gambard 2017/02/06 15:58:28 Done.
67
61 DISALLOW_COPY_AND_ASSIGN(WebFaviconDriver); 68 DISALLOW_COPY_AND_ASSIGN(WebFaviconDriver);
62 }; 69 };
63 70
64 } // namespace favicon 71 } // namespace favicon
65 72
66 #endif // COMPONENTS_FAVICON_IOS_WEB_FAVICON_DRIVER_H_ 73 #endif // COMPONENTS_FAVICON_IOS_WEB_FAVICON_DRIVER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698