NTP: VariationParamsManager resetting works, so make it a member.

NTP: VariationParamsManager resetting works, so make it a member.

This CL makes use of the resetting function VariationParamsManager.
This should work now and therefore it is sufficient to hold the manager
as a simple member.

BUG=672010
Committed: https://crrev.com/08db2becc667bf907330d414f372f4c297b8a54f
Cr-Commit-Position: refs/heads/master@{#439839}

[fhorschig, 2016-12-20 13:31:27]

Hi Jan,
this small CL is related to http://crbug.com/672010 and shows that clearing the
params works now. Do you want to take a look?

[jkrcal, 2016-12-20 14:44:43]

jkrcal@chromium.org changed reviewers:
+ jkrcal@chromium.org

[jkrcal, 2016-12-20 14:44:44]

lgtm, thanks!

(By the way, you've forgotten to address me in the previous message ;)

[fhorschig, 2016-12-20 14:48:45]

On 2016/12/20 14:44:44, jkrcal wrote:
> lgtm, thanks!
> 
> (By the way, you've forgotten to address me in the previous message ;)

Thank you for noticing it anyway!

[fhorschig, 2016-12-20 17:01:02]

The CQ bit was checked by fhorschig@chromium.org

[commit-bot: I haz the power, 2016-12-20 17:01:18]

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-12-20 17:46:45]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1482253262293890,
"parent_rev": "ac7d89043184919dece338bdf181d8ba0f93ead3", "commit_rev":
"c5607c8ff723dd47416af7e7a07f883c314ced5f"}

[commit-bot: I haz the power, 2016-12-20 17:47:16]

Description was changed from

==========
NTP: VariationParamsManager resetting works, so make it a member.

This CL makes use of the resetting function VariationParamsManager.
This should work now and therefore it is sufficient to hold the manager
as a simple member.

BUG=672010
==========

to

==========
NTP: VariationParamsManager resetting works, so make it a member.

This CL makes use of the resetting function VariationParamsManager.
This should work now and therefore it is sufficient to hold the manager
as a simple member.

BUG=672010

Review-Url: https://codereview.chromium.org/2592663003
==========

[commit-bot: I haz the power, 2016-12-20 17:47:17]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-20 17:49:59]

Description was changed from

==========
NTP: VariationParamsManager resetting works, so make it a member.

This CL makes use of the resetting function VariationParamsManager.
This should work now and therefore it is sufficient to hold the manager
as a simple member.

BUG=672010

Review-Url: https://codereview.chromium.org/2592663003
==========

to

==========
NTP: VariationParamsManager resetting works, so make it a member.

This CL makes use of the resetting function VariationParamsManager.
This should work now and therefore it is sufficient to hold the manager
as a simple member.

BUG=672010

Committed: https://crrev.com/08db2becc667bf907330d414f372f4c297b8a54f
Cr-Commit-Position: refs/heads/master@{#439839}
==========

[commit-bot: I haz the power, 2016-12-20 17:50:00]

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