Top Hosts computation should not use redirected urls and subframe urls

Top Hosts computation should not use redirected urls and subframe urls

BUG=614188
Committed: https://crrev.com/276df9be47cf9aec1dc7501afef203e1e5b2b052
Cr-Commit-Position: refs/heads/master@{#416658}

[Raj, 2016-08-31 05:56:29]

rajendrant@chromium.org changed reviewers:
+ sky@chromium.org

[Raj, 2016-08-31 05:56:30]

sky: ptal

[sky, 2016-08-31 16:29:47]

https://codereview.chromium.org/2298093002/diff/1/components/history/core/bro...
File components/history/core/browser/history_database.cc (right):

https://codereview.chromium.org/2298093002/diff/1/components/history/core/bro...
components/history/core/browser/history_database.cc:196: TopHostsList
HistoryDatabase::TopHosts(size_t num_hosts) {
I'm not familiar with this function. I believe you had sdefresne review this
addition, please have him review the change.

[Raj, 2016-08-31 16:34:49]

rajendrant@chromium.org changed reviewers:
+ sdefresne@chromium.org

[Raj, 2016-08-31 16:34:50]

sdefresne: ptal

Thanks sky

[sdefresne, 2016-09-06 16:14:51]

lgtm

[Raj, 2016-09-06 16:17:45]

The CQ bit was checked by rajendrant@chromium.org

[commit-bot: I haz the power, 2016-09-06 16:18:06]

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, 2016-09-06 17:51:36]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-06 17:53:44]

Description was changed from

==========
Top Hosts computation should not use redirected urls and subframe urls

BUG=614188
==========

to

==========
Top Hosts computation should not use redirected urls and subframe urls

BUG=614188

Committed: https://crrev.com/276df9be47cf9aec1dc7501afef203e1e5b2b052
Cr-Commit-Position: refs/heads/master@{#416658}
==========

[commit-bot: I haz the power, 2016-09-06 17:53:45]

Patchset 1 (id:??) landed as
https://crrev.com/276df9be47cf9aec1dc7501afef203e1e5b2b052
Cr-Commit-Position: refs/heads/master@{#416658}