[NTP Snippets] Add a pref to disable the NTPSnippetsService

[NTP Snippets] Add a pref to disable the NTPSnippetsService

The new pref isn't used yet. It will be used to implement a policy that disables generated content suggestions.

TBRing trivial and analogous iOS change
TBR=noyau

BUG=595713
Committed: https://crrev.com/2f95b0ea3d0ab826eb1e355b0fbd4618ecbba2aa
Cr-Commit-Position: refs/heads/master@{#406279}

[Marc Treib, 2016-07-19 13:51:21]

Description was changed from

==========
[NTP Snippets] Add a pref to disable the NTPSnippetsService

The new pref isn't used yet. I will be used to implement a policy that disables
generated content suggestions.

BUG=595713
==========

to

==========
[NTP Snippets] Add a pref to disable the NTPSnippetsService

The new pref isn't used yet. It will be used to implement a policy that disables
generated content suggestions.

BUG=595713
==========

[Marc Treib, 2016-07-19 14:50:56]

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

[Marc Treib, 2016-07-19 14:50:57]

PTAL!

https://codereview.chromium.org/2158373002/diff/20001/components/ntp_snippets...
File components/ntp_snippets/ntp_snippets_service.h (left):

https://codereview.chromium.org/2158373002/diff/20001/components/ntp_snippets...
components/ntp_snippets/ntp_snippets_service.h:225: void
LoadingSnippetsFinished();
unrelated: no implementation

https://codereview.chromium.org/2158373002/diff/20001/components/ntp_snippets...
components/ntp_snippets/ntp_snippets_service.h:241: void
UpdateStateForStatus(DisabledReason disabled_reason);
unrelated: confusing name

[Bernhard Bauer, 2016-07-19 14:56:14]

lgtm

[Marc Treib, 2016-07-19 14:57:02]

The CQ bit was checked by treib@chromium.org

[Marc Treib, 2016-07-19 14:57:02]

The CQ bit was unchecked by treib@chromium.org

[Marc Treib, 2016-07-19 14:57:42]

Description was changed from

==========
[NTP Snippets] Add a pref to disable the NTPSnippetsService

The new pref isn't used yet. It will be used to implement a policy that disables
generated content suggestions.

BUG=595713
==========

to

==========
[NTP Snippets] Add a pref to disable the NTPSnippetsService

The new pref isn't used yet. It will be used to implement a policy that disables
generated content suggestions.

TBRing trivial and analogous iOS change
TBR=noyau

BUG=595713
==========

[Marc Treib, 2016-07-19 14:57:51]

The CQ bit was checked by treib@chromium.org

[commit-bot: I haz the power, 2016-07-19 14:58: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-07-19 15:31:14]

Description was changed from

==========
[NTP Snippets] Add a pref to disable the NTPSnippetsService

The new pref isn't used yet. It will be used to implement a policy that disables
generated content suggestions.

TBRing trivial and analogous iOS change
TBR=noyau

BUG=595713
==========

to

==========
[NTP Snippets] Add a pref to disable the NTPSnippetsService

The new pref isn't used yet. It will be used to implement a policy that disables
generated content suggestions.

TBRing trivial and analogous iOS change
TBR=noyau

BUG=595713
==========

[commit-bot: I haz the power, 2016-07-19 15:31:15]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-19 15:32:55]

Description was changed from

==========
[NTP Snippets] Add a pref to disable the NTPSnippetsService

The new pref isn't used yet. It will be used to implement a policy that disables
generated content suggestions.

TBRing trivial and analogous iOS change
TBR=noyau

BUG=595713
==========

to

==========
[NTP Snippets] Add a pref to disable the NTPSnippetsService

The new pref isn't used yet. It will be used to implement a policy that disables
generated content suggestions.

TBRing trivial and analogous iOS change
TBR=noyau

BUG=595713

Committed: https://crrev.com/2f95b0ea3d0ab826eb1e355b0fbd4618ecbba2aa
Cr-Commit-Position: refs/heads/master@{#406279}
==========

[commit-bot: I haz the power, 2016-07-19 15:32:56]

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