NTP5: Re-implementing favicons for Most Visited Pages.

NTP5: Re-implementing favicons for Most Visited Pages.

This is the reimplementation of https://codereview.chromium.org/11416091/, without the merge problems that was introduced in one of the latest Patch Lists.


BUG=

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=169841

[pedro (no code reviews), 2012-11-27 22:56:19]

Evan, could you please review the ntp_search related code?

David, could you please review the chrome/browser/ui/search/search_ui.cc file?

[dhollowa, 2012-11-27 23:07:52]

c/b/ui/search/... lgtm.

[Evan Stade, 2012-11-27 23:29:10]

lgtm, except the description should not say NTP4

[pedro (no code reviews), 2012-11-27 23:47:47]

On 2012/11/27 23:29:10, Evan Stade wrote:
> lgtm, except the description should not say NTP4

Oops! Fixed.

[commit-bot: I haz the power, 2012-11-28 00:15:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pedrosimonetti@chromium.org/11316206/1

[commit-bot: I haz the power, 2012-11-28 02:03:51]

Retried try job too often for step(s) aura_unit_tests

[commit-bot: I haz the power, 2012-11-28 02:30:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pedrosimonetti@chromium.org/11316206/1

[commit-bot: I haz the power, 2012-11-28 04:17:25]

Change committed as 169841