Update hard-coded search engines favicon URLs.

components/search_engines/prepopulated_engines.json

Index: components/search_engines/prepopulated_engines.json
diff --git a/components/search_engines/prepopulated_engines.json b/components/search_engines/prepopulated_engines.json
index 97dad3e51dc41d344149c0760375a618a93d6980..1492424f59aff0edec39a3cfea7ba7dc86a95dda 100644
--- a/components/search_engines/prepopulated_engines.json
+++ b/components/search_engines/prepopulated_engines.json
@@ -88,7 +88,7 @@
     "bing": {
       "name": "Bing",
       "keyword": "bing.com",
-      "favicon_url": "https://www.bing.com/s/a/bing_p.ico",
+      "favicon_url": "http://www.bing.com/sa/simg/bing_p_rr_teal_min.ico",

[Peter Kasting, 2016/12/01 04:56:22]

Why did you change the scheme here?  The link rel tag does not force HTTP, and
the search URL here (which is what users who use this engine will land on) uses
HTTPS.

[dpapad, 2016/12/08 22:58:39]

This string needs to match exactly (including the scheme) with what is returned
to the UI at [1] and [2]. Same holds for the other comment below. You have
already seen my more in-depth findings [3] explaining how something is broken
design-wise in the end-to-end favicon flow. I was not planning to just land this
CL, but I just wanted to showcase what is broken (for the pre-defined search
engines).

[1] New settings:
https://cs.chromium.org/chromium/src/chrome/browser/ui/webui/settings/search_...
[2] Old settings:
https://cs.chromium.org/chromium/src/chrome/browser/ui/webui/options/search_e...
[3]
https://docs.google.com/document/d/1YN4OkEOi5gskFFiTuW8N93kFOdD6MJuEBq4wF__iF...

[Peter Kasting, 2016/12/08 23:01:02]

Right, I'm just saying, the actual URL in the cache (which we need to match so
there's favicon data to show) will be based on whatever the user actually ends
up loading, and I think they will load the HTTPS URL here and the HTTP one
below.

       "search_url": "https://www.bing.com/search?q={searchTerms}&PC=U316&FORM=CHROMN",
       "suggest_url": "https://www.bing.com/osjson.aspx?query={searchTerms}&language={language}&PC=U316",
       "image_url": "https://www.bing.com/images/detail/search?iss=sbi&FORM=CHROMI#enterInsights",
@@ -111,7 +111,7 @@
     "google": {
       "name": "Google",
       "keyword": "google.com",
-      "favicon_url": "http://www.google.com/favicon.ico",
+      "favicon_url": "https://www.google.com/favicon.ico",

[Peter Kasting, 2016/12/01 04:56:22]

This doesn't seem to match the link rel tag on the current SRP, which links to
https://www.google.com/images/branding/product/ico/googleg_lodp.ico .

       "search_url": "{google:baseURL}search?q={searchTerms}&{google:RLZ}{google:originalQueryForSuggestion}{google:assistedQueryStats}{google:searchFieldtrialParameter}{google:iOSSearchLanguage}{google:searchClient}{google:sourceId}{google:instantExtendedEnabledParameter}{google:contextualSearchVersion}ie={inputEncoding}",
       "suggest_url": "{google:baseSuggestURL}search?{google:searchFieldtrialParameter}client={google:suggestClient}&gs_ri={google:suggestRid}&xssi=t&q={searchTerms}&{google:inputType}{google:cursorPosition}{google:currentPageUrl}{google:pageClassification}{google:searchVersion}{google:sessionToken}{google:prefetchQuery}sugkey={google:suggestAPIKeyParameter}",
       "instant_url": "{google:baseURL}webhp?sourceid=chrome-instant&{google:RLZ}{google:forceInstantResults}{google:instantExtendedEnabledParameter}ie={inputEncoding}",