Don't share Push API message payloads unless explicitly enabled.

Don't share Push API message payloads unless explicitly enabled.

There are open security issues about exposing payloads for the Push API,
so we'd like to guard those behind a command line flag for now.

This is part of a three-sided patch:
  [1] https://codereview.chromium.org/872013002
  [2] This patch.
  [3] https://codereview.chromium.org/869303002

BUG=449184
Committed: https://crrev.com/454b4e532e491f2582029994ae263623ae58317d
Cr-Commit-Position: refs/heads/master@{#313265}

[Peter Beverloo, 2015-01-23 18:21:04]

peter@chromium.org changed reviewers:
+ mvanouwerkerk@chromium.org

[Peter Beverloo, 2015-01-23 18:21:05]

+mvanouwerkerk for the Push API

I'll poke a content/ owner after you're happy.

[Michael van Ouwerkerk, 2015-01-23 18:35:14]

lgtm

[Peter Beverloo, 2015-01-26 14:32:52]

peter@chromium.org changed reviewers:
+ avi@chromium.org

[Peter Beverloo, 2015-01-26 14:32:53]

+avi for new command line flag

[Avi (use Gerrit), 2015-01-26 15:27:01]

lgtm

The flag is fine. You are planning on eventually removing it?

[Peter Beverloo, 2015-01-26 15:29:53]

On 2015/01/26 15:27:01, Avi (OOO 18-25 Jan) wrote:
> lgtm
> 
> The flag is fine. You are planning on eventually removing it?

Definitely. It's unclear what the final encryption story for message payloads
will look like, but we don't want to block shipping a v1 on that.

Thanks!

[Peter Beverloo, 2015-01-27 13:05:30]

The CQ bit was checked by peter@chromium.org

[commit-bot: I haz the power, 2015-01-27 13:06:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/874613002/20001

[commit-bot: I haz the power, 2015-01-27 13:13:25]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-27 13:14:32]

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