Save favicon during reading list distillation

ios/web/public/favicon_url.cc

 // Copyright 2014 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.
 
 #include "ios/web/public/favicon_url.h"
 
+#include "base/values.h"
+
 namespace web {
 
 FaviconURL::FaviconURL()
   : icon_type(INVALID_ICON) {
 }
 
 FaviconURL::FaviconURL(const GURL& url,
                        IconType type,
                        const std::vector<gfx::Size>& sizes)
     : icon_url(url), icon_type(type), icon_sizes(sizes) {}
 
 FaviconURL::FaviconURL(const FaviconURL& other) = default;
 
 FaviconURL::~FaviconURL() {
 }
 
+bool ExtractFaviconURL(base::ListValue* favicons,
+                       std::vector<web::FaviconURL>& result_urls) {
+  for (size_t fav_idx = 0; fav_idx != favicons->GetSize(); ++fav_idx) {
+    base::DictionaryValue* favicon = nullptr;
+    if (!favicons->GetDictionary(fav_idx, &favicon))
+      return false;
+    std::string href;
+    std::string rel;
+    if (!favicon->GetString("href", &href)) {
+      DLOG(WARNING) << "JS message parameter not found: href";
+      return false;
+    }
+    if (!favicon->GetString("rel", &rel)) {
+      DLOG(WARNING) << "JS message parameter not found: rel";
+      return false;
+    }
+    web::FaviconURL::IconType icon_type = web::FaviconURL::FAVICON;
+    if (rel == "apple-touch-icon")
+      icon_type = web::FaviconURL::TOUCH_ICON;
+    else if (rel == "apple-touch-icon-precomposed")
+      icon_type = web::FaviconURL::TOUCH_PRECOMPOSED_ICON;
+    result_urls.push_back(
+        web::FaviconURL(GURL(href), icon_type, std::vector<gfx::Size>()));
+  }
+  return true;

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

Should we return false if |result_urls| is empty at the end here?  I notice that
you check for that when calling this from CRWWebController.

+}
+
 } // namespace web