Save favicon during reading list distillation

components/dom_distiller/ios/distiller_favicon_ios.h

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_DOM_DISTILLER_IOS_DISTILLER_FAVICON_IOS_H_
+#define COMPONENTS_DOM_DISTILLER_IOS_DISTILLER_FAVICON_IOS_H_
+
+#include <vector>
+
+namespace web {
+struct FaviconURL;
+}
+
+class GURL;
+
+namespace dom_distiller {
+
+class DistillerFaviconIOS {

[kkhorimoto, 2016/11/28 23:48:25]

This needs class comments.  Additionally, we only name classes with the *IOS
suffix if they're iOS implementations of interfaces that are shared across
platforms.  Since this class is specific to iOS, I don't think the suffix is
necessary here; maybe just use an abstract interface in components/ then an
implementation with the *Impl suffix would work better.

+ public:
+  // Makes sure the favicons located at |urls| are associated with |page_url|.
+  virtual void HandleFaviconURL(std::vector<web::FaviconURL> urls,

[Eugene But (OOO till 7-30), 2016/11/30 18:51:30]

Optional nit: Per C+++ Style Guide this should be HandleFaviconUrl, however we
don't follow that rule and generally very inconsistent with URL capitalization. 

+                                const GURL& page_url) = 0;
+};
+
+}  // namespace dom_distiller
+
+#endif  // COMPONENTS_DOM_DISTILLER_IOS_DISTILLER_FAVICON_IOS_H_