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

Unified Diff: ios/chrome/browser/favicon/favicon_client_impl.h

Issue 2609363003: Remove ChromeBrowserProvider::GetFaviconForURL(). (Closed)
Patch Set: Remove incorrect include and obsolete deps. 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ios/chrome/browser/favicon/BUILD.gn ('k') | ios/chrome/browser/favicon/favicon_client_impl.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/favicon/favicon_client_impl.h
diff --git a/ios/chrome/browser/favicon/favicon_client_impl.h b/ios/chrome/browser/favicon/favicon_client_impl.h
index 3874fb5dbb941d04edaf3c4848defa2288dbef8d..06803030aa08809a2dceaa1279b214e96f63983f 100644
--- a/ios/chrome/browser/favicon/favicon_client_impl.h
+++ b/ios/chrome/browser/favicon/favicon_client_impl.h
@@ -12,16 +12,12 @@
#include "components/favicon/core/favicon_client.h"
#include "components/favicon_base/favicon_callback.h"
-namespace ios {
-class ChromeBrowserState;
-}
-
class GURL;
// FaviconClientImpl implements the favicon::FaviconClient interface on iOS.
class FaviconClientImpl : public favicon::FaviconClient {
public:
- FaviconClientImpl(ios::ChromeBrowserState* browser_state);
+ FaviconClientImpl();
~FaviconClientImpl() override;
private:
@@ -33,8 +29,6 @@ class FaviconClientImpl : public favicon::FaviconClient {
const favicon_base::FaviconResultsCallback& callback,
base::CancelableTaskTracker* tracker) override;
- ios::ChromeBrowserState* browser_state_;
-
DISALLOW_COPY_AND_ASSIGN(FaviconClientImpl);
};
« no previous file with comments | « ios/chrome/browser/favicon/BUILD.gn ('k') | ios/chrome/browser/favicon/favicon_client_impl.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698