Save favicon during reading list distillation

components/dom_distiller/ios/distiller_page_factory_ios.mm

Index: components/dom_distiller/ios/distiller_page_factory_ios.mm
diff --git a/components/dom_distiller/ios/distiller_page_factory_ios.mm b/components/dom_distiller/ios/distiller_page_factory_ios.mm
index ad355f9cd18cb87eb9ae69dcf77277295a459681..527984236697ecd400b4c688c7623123257b7ccd 100644
--- a/components/dom_distiller/ios/distiller_page_factory_ios.mm
+++ b/components/dom_distiller/ios/distiller_page_factory_ios.mm
@@ -6,24 +6,28 @@
 
 #include "base/memory/ptr_util.h"
 #include "components/dom_distiller/ios/distiller_page_ios.h"
+#include "components/dom_distiller/ios/favicon_web_state_dispatcher.h"
 #include "ios/web/public/browser_state.h"
 
 namespace dom_distiller {
 
 DistillerPageFactoryIOS::DistillerPageFactoryIOS(
-    web::BrowserState* browser_state)
-    : browser_state_(browser_state) {
-}
+    std::unique_ptr<FaviconWebStateDispatcher> web_state_dispatcher)
+    : web_state_dispatcher_(std::move(web_state_dispatcher)) {}
+
+DistillerPageFactoryIOS::~DistillerPageFactoryIOS() {}
 
 std::unique_ptr<DistillerPage> DistillerPageFactoryIOS::CreateDistillerPage(
     const gfx::Size& view_size) const {
-  return base::WrapUnique<DistillerPage>(new DistillerPageIOS(browser_state_));
+  return base::WrapUnique<DistillerPage>(

[sdefresne, 2016/12/20 15:40:59]

nit: please use base::MakeUnique<> here (or add a comment saying that it crash
with Xcode's clang so that I don't have to ask you everytime I review this
file). If you add a comment, please create a bug and add a TODO too ;-)

[gambard, 2016/12/20 17:11:42]

Done.

+      new DistillerPageIOS(web_state_dispatcher_.get()));
 }
 
 std::unique_ptr<DistillerPage>
 DistillerPageFactoryIOS::CreateDistillerPageWithHandle(
     std::unique_ptr<SourcePageHandle> handle) const {
-  return base::WrapUnique<DistillerPage>(new DistillerPageIOS(browser_state_));
+  return base::WrapUnique<DistillerPage>(

[sdefresne, 2016/12/20 15:40:59]

nit: please use base::MakeUnique<> here (or add a comment saying that it crash
with Xcode's clang so that I don't have to ask you everytime I review this
file). If you add a comment, please create a bug and add a TODO too ;-)

[gambard, 2016/12/20 17:11:42]

Done.

+      new DistillerPageIOS(web_state_dispatcher_.get()));
 }
 
 }  // namespace dom_distiller