Reload the Most Visited section when it changes

Reload the Most Visited section when it changes

When the Most Visited model changes, the UI should be updated only on user
interaction.

BUG=707754, 724427, 724499
Review-Url: https://codereview.chromium.org/2923973002
Cr-Commit-Position: refs/heads/master@{#477610}
Committed: https://chromium.googlesource.com/chromium/src/+/28050dfc93a9ff9de70fcfba4e49aefdfc08d5fc

[gambard, 2017-06-07 08:22:27]

gambard@chromium.org changed reviewers:
+ lpromero@chromium.org

[gambard, 2017-06-07 08:22:27]

PTAL.

[lpromero, 2017-06-07 08:57:30]

lgtm

https://codereview.chromium.org/2923973002/diff/20001/ios/chrome/browser/cont...
File ios/chrome/browser/content_suggestions/content_suggestions_mediator.mm
(right):

https://codereview.chromium.org/2923973002/diff/20001/ios/chrome/browser/cont...
ios/chrome/browser/content_suggestions/content_suggestions_mediator.mm:386: //
prevent updating the all the tiles without any action from the user.
"the all the"?

[gambard, 2017-06-07 10:54:08]

Thanks!

https://codereview.chromium.org/2923973002/diff/20001/ios/chrome/browser/cont...
File ios/chrome/browser/content_suggestions/content_suggestions_mediator.mm
(right):

https://codereview.chromium.org/2923973002/diff/20001/ios/chrome/browser/cont...
ios/chrome/browser/content_suggestions/content_suggestions_mediator.mm:386: //
prevent updating the all the tiles without any action from the user.
On 2017/06/07 08:57:30, lpromero wrote:
> "the all the"?

Done.

[gambard, 2017-06-07 10:54:11]

The CQ bit was checked by gambard@chromium.org

[gambard, 2017-06-07 10:54:12]

The patchset sent to the CQ was uploaded after l-g-t-m from
lpromero@chromium.org
Link to the patchset: https://codereview.chromium.org/2923973002/#ps40001
(title: "Address comment")

[commit-bot: I haz the power, 2017-06-07 10:54:29]

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-07 11:05:40]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1496832851712540,
"parent_rev": "aa7c4bc5bf0972405be27c56e35837da956f67c0", "commit_rev":
"28050dfc93a9ff9de70fcfba4e49aefdfc08d5fc"}

[commit-bot: I haz the power, 2017-06-07 11:05:53]

Description was changed from

==========
Reload the Most Visited section when it changes

When the Most Visited model changes, the UI should be updated only on user
interaction.

BUG=707754, 724427
==========

to

==========
Reload the Most Visited section when it changes

When the Most Visited model changes, the UI should be updated only on user
interaction.

BUG=707754, 724427

Review-Url: https://codereview.chromium.org/2923973002
Cr-Commit-Position: refs/heads/master@{#477610}
Committed:
https://chromium.googlesource.com/chromium/src/+/28050dfc93a9ff9de70fcfba4e49...
==========

[commit-bot: I haz the power, 2017-06-07 11:05:54]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/28050dfc93a9ff9de70fcfba4e49...

[gambard, 2017-06-07 12:52:18]

Description was changed from

==========
Reload the Most Visited section when it changes

When the Most Visited model changes, the UI should be updated only on user
interaction.

BUG=707754, 724427

Review-Url: https://codereview.chromium.org/2923973002
Cr-Commit-Position: refs/heads/master@{#477610}
Committed:
https://chromium.googlesource.com/chromium/src/+/28050dfc93a9ff9de70fcfba4e49...
==========

to

==========
Reload the Most Visited section when it changes

When the Most Visited model changes, the UI should be updated only on user
interaction.

BUG=707754, 724427, 724499

Review-Url: https://codereview.chromium.org/2923973002
Cr-Commit-Position: refs/heads/master@{#477610}
Committed:
https://chromium.googlesource.com/chromium/src/+/28050dfc93a9ff9de70fcfba4e49...
==========