Use JumpList icon caches to remove redundant favicon loading

Use JumpList icon caches to remove redundant favicon loading

This CL applies the icon caches to save unnecessary favicon loading from 
the FaviconService. The icon caches have already been used to trim out 
the redundant icon creation and deletion on disk as in crrev/2859193005. 
These two changes make the best of the icon caches.

BUG=40407, 717236
Review-Url: https://codereview.chromium.org/2936053002
Cr-Commit-Position: refs/heads/master@{#480122}
Committed: https://chromium.googlesource.com/chromium/src/+/0cf2ecfa1dfa8ab1626c75d1d27bf01be3e7aa10

[chengx, 2017-06-13 21:08:27]

The CQ bit was checked by chengx@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-13 21:09:19]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[chengx, 2017-06-13 21:27:43]

Description was changed from

==========
Use cache to remove redundant favicon loading

BUG=
==========

to

==========
Use cache to remove redundant favicon loading

BUG=717236
==========

[chengx, 2017-06-13 21:36:07]

Description was changed from

==========
Use cache to remove redundant favicon loading

BUG=717236
==========

to

==========
Use cache to remove redundant favicon loading

BUG=40407, 717236
==========

[chengx, 2017-06-13 21:59:50]

Description was changed from

==========
Use cache to remove redundant favicon loading

BUG=40407, 717236
==========

to

==========
Use icon caches to remove redundant favicon loading

BUG=40407, 717236
==========

[commit-bot: I haz the power, 2017-06-13 22:13:49]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-13 22:13:50]

Dry run: This issue passed the CQ dry run.

[chengx, 2017-06-13 22:39:57]

Description was changed from

==========
Use icon caches to remove redundant favicon loading

BUG=40407, 717236
==========

to

==========
Use icon caches to remove redundant favicon loading

This CL applies the icon caches to save unnecessary favicon loading from 
the FaviconService. The icon caches have already been used to trim out 
the redundant icon creation and deletion on disk as in crrev/2859193005. 
These two changes make the best of the icon caches.

BUG=40407, 717236
==========

[chengx, 2017-06-13 22:40:17]

Description was changed from

==========
Use icon caches to remove redundant favicon loading

This CL applies the icon caches to save unnecessary favicon loading from 
the FaviconService. The icon caches have already been used to trim out 
the redundant icon creation and deletion on disk as in crrev/2859193005. 
These two changes make the best of the icon caches.

BUG=40407, 717236
==========

to

==========
Use JumpList icon caches to remove redundant favicon loading

This CL applies the icon caches to save unnecessary favicon loading from 
the FaviconService. The icon caches have already been used to trim out 
the redundant icon creation and deletion on disk as in crrev/2859193005. 
These two changes make the best of the icon caches.

BUG=40407, 717236
==========

[chengx, 2017-06-13 22:43:00]

chengx@chromium.org changed reviewers:
+ grt@chromium.org

[chengx, 2017-06-13 22:43:01]

Hi Greg, PTAL at this small CL. Thanks!

[grt (UTC plus 2), 2017-06-14 10:57:42]

lgtm

[chengx, 2017-06-16 18:38:51]

The CQ bit was checked by chengx@chromium.org

[chengx, 2017-06-16 18:38:52]

The patchset sent to the CQ was uploaded after l-g-t-m from grt@chromium.org
Link to the patchset: https://codereview.chromium.org/2936053002/#ps20001
(title: "Merge branch 'master' of https://chromium.googlesource.com/chromium/src
into usecachetosaveiconfetch")

[commit-bot: I haz the power, 2017-06-16 18:39:25]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-16 19:27:50]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1497638331781030,
"parent_rev": "079d5c1f6b73fb4d13ddf81f92154b53c7a7713d", "commit_rev":
"0cf2ecfa1dfa8ab1626c75d1d27bf01be3e7aa10"}

[commit-bot: I haz the power, 2017-06-16 19:28:02]

Description was changed from

==========
Use JumpList icon caches to remove redundant favicon loading

This CL applies the icon caches to save unnecessary favicon loading from 
the FaviconService. The icon caches have already been used to trim out 
the redundant icon creation and deletion on disk as in crrev/2859193005. 
These two changes make the best of the icon caches.

BUG=40407, 717236
==========

to

==========
Use JumpList icon caches to remove redundant favicon loading

This CL applies the icon caches to save unnecessary favicon loading from 
the FaviconService. The icon caches have already been used to trim out 
the redundant icon creation and deletion on disk as in crrev/2859193005. 
These two changes make the best of the icon caches.

BUG=40407, 717236

Review-Url: https://codereview.chromium.org/2936053002
Cr-Commit-Position: refs/heads/master@{#480122}
Committed:
https://chromium.googlesource.com/chromium/src/+/0cf2ecfa1dfa8ab1626c75d1d27b...
==========

[commit-bot: I haz the power, 2017-06-16 19:28:03]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/0cf2ecfa1dfa8ab1626c75d1d27b...