Remove redundant FaviconDriverImpl::large_icon_handler_ on Android

Remove redundant FaviconDriverImpl::large_icon_handler_ on Android

When the Icon NTP is enabled on Android, FaviconDriverImpl creates two identical
FaviconHandlers: |FaviconDriver::touch_icon_handler_| and
|FaviconDriver::large_icon_handler_|.

This CL removes |FaviconDriver::large_icon_handler_| because it is redundant.

BUG=None

Committed: https://crrev.com/00f98d85ceb689e71f05e88d46df71acbd2b7393
Cr-Commit-Position: refs/heads/master@{#356306}

[pkotwicz, 2015-10-18 21:57:20]

pkotwicz@chromium.org changed reviewers:
+ huangs@chromium.org

[pkotwicz, 2015-10-18 21:57:21]

huangs@ can you please take a look?

My understanding is that there is serious discussion of launching icon NTP on
Android. Having both |FaviconDriver::touch_icon_handler_| and
|FaviconDriver::large_icon_handler_| is a waste on Android.

I have done some research and apple-touch-icon and apple-touch-icon-precomposed
are very rare on non-Mobile sites so |large_icon_handler_| is not very helpful
on the desktop. (And if it were helpful, using |touch_icon_handler_| when the
icon NTP is enabled would be a better choice)

[pkotwicz, 2015-10-26 15:26:20]

huangs@ Ping!

[huangs, 2015-10-26 19:16:51]

huangs@chromium.org changed reviewers:
+ rogerm@chromium.org

[huangs, 2015-10-26 19:16:51]

LGTM.

[huangs, 2015-10-26 19:17:27]

Please update the tile and description though.

[pkotwicz, 2015-10-26 19:17:38]

Description was changed from

==========
Remove redundant FaviconHandler::LARGE FaviconHandler on Android

When the Icon NTP is enabled on Android, FaviconDriverImpl creates two identical
FaviconHandlers: |FaviconDriver::touch_icon_handler_| and
|FaviconDriver::large_icon_handler_|.

This CL removes |FaviconDriver::large_icon_handler_| because it is redundant.

BUG=None
==========

to

==========
Remove redundant FaviconDriverImpl::large_icon_handler_ on Android

When the Icon NTP is enabled on Android, FaviconDriverImpl creates two identical
FaviconHandlers: |FaviconDriver::touch_icon_handler_| and
|FaviconDriver::large_icon_handler_|.

This CL removes |FaviconDriver::large_icon_handler_| because it is redundant.

BUG=None
==========

[pkotwicz, 2015-10-26 19:17:38]

pkotwicz@chromium.org changed reviewers:
- rogerm@chromium.org

[pkotwicz, 2015-10-27 15:15:40]

The CQ bit was checked by pkotwicz@chromium.org

[commit-bot: I haz the power, 2015-10-27 15:15:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1415543002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1415543002/20001

[commit-bot: I haz the power, 2015-10-27 15:56:09]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-27 15:57:03]

Patchset 2 (id:??) landed as
https://crrev.com/00f98d85ceb689e71f05e88d46df71acbd2b7393
Cr-Commit-Position: refs/heads/master@{#356306}