ntp_snippets: Use base::EraseIf()

ntp_snippets: Use base::EraseIf()

BUG=None

Review-Url: https://codereview.chromium.org/2741133003
Cr-Commit-Position: refs/heads/master@{#456099}
Committed: https://chromium.googlesource.com/chromium/src/+/fd23b83100cfe96d872c45d61b51d0f878110f8d

[sfiera, 2017-03-10 10:22:08]

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

[commit-bot: I haz the power, 2017-03-10 10:22:12]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sfiera, 2017-03-10 10:22:19]

sfiera@chromium.org changed reviewers:
+ treib@chromium.org

[Marc Treib, 2017-03-10 10:38:58]

lgtm

I also saw that announcement email :)  Nice!

Description nit: Add "ntp_snippets: " to the first line?

https://codereview.chromium.org/2741133003/diff/1/components/ntp_snippets/boo...
File components/ntp_snippets/bookmarks/bookmark_last_visit_utils.cc (right):

https://codereview.chromium.org/2741133003/diff/1/components/ntp_snippets/boo...
components/ntp_snippets/bookmarks/bookmark_last_visit_utils.cc:14: #include
"base/strings/string_number_conversions.h"
#include "base/stl_util.h"

[commit-bot: I haz the power, 2017-03-10 11:10:13]

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

[commit-bot: I haz the power, 2017-03-10 11:10:14]

Dry run: This issue passed the CQ dry run.

[sfiera, 2017-03-10 17:14:37]

Description was changed from

==========
Use base::EraseIf()

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

to

==========
ntp_snippets: Use base::EraseIf()

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

[sfiera, 2017-03-10 17:14:45]

The CQ bit was checked by sfiera@chromium.org

[sfiera, 2017-03-10 17:14:45]

The patchset sent to the CQ was uploaded after l-g-t-m from treib@chromium.org
Link to the patchset: https://codereview.chromium.org/2741133003/#ps20001
(title: "Add #include")

[commit-bot: I haz the power, 2017-03-10 17:15:13]

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-03-10 17:55:29]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489166085147300,
"parent_rev": "662c24fc6debb33ee34a927d21dc26154b8973d6", "commit_rev":
"fd23b83100cfe96d872c45d61b51d0f878110f8d"}

[commit-bot: I haz the power, 2017-03-10 17:56:10]

Description was changed from

==========
ntp_snippets: Use base::EraseIf()

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

to

==========
ntp_snippets: Use base::EraseIf()

BUG=None

Review-Url: https://codereview.chromium.org/2741133003
Cr-Commit-Position: refs/heads/master@{#456099}
Committed:
https://chromium.googlesource.com/chromium/src/+/fd23b83100cfe96d872c45d61b51...
==========

[commit-bot: I haz the power, 2017-03-10 17:56:12]

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