Introduce dedicated enum value for icons from Web Manifests

components/favicon/core/favicon_driver_impl.cc

Index: components/favicon/core/favicon_driver_impl.cc
diff --git a/components/favicon/core/favicon_driver_impl.cc b/components/favicon/core/favicon_driver_impl.cc
index 545def8f27886a2ff8997d8acd3f54d2f0b2953f..6b92e684cec3a9df89bb2a3512e96eaea916f9ce 100644
--- a/components/favicon/core/favicon_driver_impl.cc
+++ b/components/favicon/core/favicon_driver_impl.cc
@@ -106,7 +106,7 @@ void FaviconDriverImpl::OnUpdateCandidates(
     // FAVICON, because those compete which each other (i.e. manifest icons
     // override inline favicons).

[pkotwicz, 2017/05/26 14:49:56]

You need to update this comment

[mastiz, 2017/05/26 18:30:25]

Done.

     handler->OnUpdateCandidates(page_url, candidates,
-                                handler->icon_types() & favicon_base::FAVICON
+                                handler->icon_types() & favicon_base::TOUCH_ICON

[pkotwicz, 2017/05/26 14:49:56]

Might as well make this WEB_MANIFEST_ICON

[mastiz, 2017/05/26 18:30:25]

Done.

                                     ? manifest_url
                                     : GURL::EmptyGURL());
   }