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

Unified Diff: ios/chrome/browser/reading_list/favicon_web_state_dispatcher_impl.h

Issue 2604773002: Create distiller files for Reading List. (Closed)
Patch Set: Created 4 years 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: ios/chrome/browser/reading_list/favicon_web_state_dispatcher_impl.h
diff --git a/ios/chrome/browser/dom_distiller/favicon_web_state_dispatcher_impl.h b/ios/chrome/browser/reading_list/favicon_web_state_dispatcher_impl.h
similarity index 71%
rename from ios/chrome/browser/dom_distiller/favicon_web_state_dispatcher_impl.h
rename to ios/chrome/browser/reading_list/favicon_web_state_dispatcher_impl.h
index d2e976813ae2559e998465375e99f46cad9c6a3c..29234b8ad6536117731925de143ccb92c9347abe 100644
--- a/ios/chrome/browser/dom_distiller/favicon_web_state_dispatcher_impl.h
+++ b/ios/chrome/browser/reading_list/favicon_web_state_dispatcher_impl.h
@@ -2,20 +2,20 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef IOS_CHROME_BROWSER_DOM_DISTILLER_FAVICON_WEB_STATE_DISPATCHER_IMPL_H_
-#define IOS_CHROME_BROWSER_DOM_DISTILLER_FAVICON_WEB_STATE_DISPATCHER_IMPL_H_
+#ifndef IOS_CHROME_BROWSER_READING_LIST_FAVICON_WEB_STATE_DISPATCHER_IMPL_H_
+#define IOS_CHROME_BROWSER_READING_LIST_FAVICON_WEB_STATE_DISPATCHER_IMPL_H_
#include <memory>
#include <vector>
#include "base/memory/weak_ptr.h"
-#include "components/dom_distiller/ios/favicon_web_state_dispatcher.h"
+#include "components/reading_list/ios/favicon_web_state_dispatcher.h"
namespace web {
class BrowserState;
}
-namespace dom_distiller {
+namespace reading_list {
// Implementation of the FaviconWebStateDispatcher.
class FaviconWebStateDispatcherImpl : public FaviconWebStateDispatcher {
@@ -27,8 +27,8 @@ class FaviconWebStateDispatcherImpl : public FaviconWebStateDispatcher {
~FaviconWebStateDispatcherImpl() override;
// FaviconWebStateDispatcher implementation.
- web::WebState* RequestWebState() override;
- void ReturnWebState(web::WebState* web_state) override;
+ std::unique_ptr<web::WebState> RequestWebState() override;
+ void ReturnWebState(std::unique_ptr<web::WebState> web_state) override;
private:
web::BrowserState* browser_state_;
@@ -41,4 +41,4 @@ class FaviconWebStateDispatcherImpl : public FaviconWebStateDispatcher {
} // namespace dom_distiller
gambard 2016/12/27 09:35:58 Update namespace
Olivier 2016/12/27 10:08:36 Done.
-#endif // IOS_CHROME_BROWSER_DOM_DISTILLER_FAVICON_WEB_STATE_DISPATCHER_IMPL_H_
+#endif // IOS_CHROME_BROWSER_READING_LIST_FAVICON_WEB_STATE_DISPATCHER_IMPL_H_

Powered by Google App Engine
This is Rietveld 408576698