Updating sandbox flags after Blink-side changes.

Updating sandbox flags after Blink-side changes.

Blink added two sandbox flags; this patch brings //content into line.

BUG=483624, 487157
Committed: https://crrev.com/17d29fe84dc5e4645bd38e92d86ee3dd2bc47875
Cr-Commit-Position: refs/heads/master@{#331345}

[Mike West, 2015-05-22 14:02:17]

mkwst@chromium.org changed reviewers:
+ dcheng@chromium.org, jochen@chromium.org

[Mike West, 2015-05-22 14:02:18]

Jochen, mind stamping this? Not throwing it to the bots yet, as the modals flag
hasn't landed in Blink yet.

dcheng: Thanks for the heads up!

[jochen (gone - plz use gerrit), 2015-05-22 14:16:05]

lgtm

[dcheng, 2015-05-22 14:17:52]

dcheng@chromium.org changed reviewers:
+ creis@chromium.org

[dcheng, 2015-05-22 14:17:53]

So I've actually been thinking: we should just use WebSandboxFlags directly in
content.

There's no particular reason for us to dupe the definition into content, and
jam@ has confirmed that things like enums and PODs are OK to add to
content/browser/DEPS and content/common/DEPS.

WDYT?

[dcheng, 2015-05-22 14:18:39]

(I'm happy to do the changes for that if we're OK with that path)

[Mike West, 2015-05-22 15:44:00]

On 2015/05/22 at 14:18:39, dcheng wrote:
> (I'm happy to do the changes for that if we're OK with that path)

Sounds totally reasonable to me. And it also sounds wonderful that you're
volunteering for the work! :)

[Mike West, 2015-05-22 15:44:23]

On 2015/05/22 at 15:44:00, Mike West wrote:
> On 2015/05/22 at 14:18:39, dcheng wrote:
> > (I'm happy to do the changes for that if we're OK with that path)
> 
> Sounds totally reasonable to me. And it also sounds wonderful that you're
volunteering for the work! :)

Shall I kill this CL, or land it, and let you update from there?

[dcheng, 2015-05-22 15:48:51]

Either way is OK with me.

[Charlie Reis, 2015-05-22 17:30:25]

creis@chromium.org changed reviewers:
+ alexmos@chromium.org

[Charlie Reis, 2015-05-22 17:30:25]

[+alexmos for sandbox flag replication]

+1 to using the Blink type directly now that we know it's ok.

[Mike West, 2015-05-26 10:08:44]

The CQ bit was checked by mkwst@chromium.org

[commit-bot: I haz the power, 2015-05-26 10:09:10]

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

[Mike West, 2015-05-26 10:09:15]

On 2015/05/22 at 17:30:25, creis wrote:
> [+alexmos for sandbox flag replication]
> 
> +1 to using the Blink type directly now that we know it's ok.

I'll land this just to make Blink and Chromium consistent, and let dcheng@
convert over to the Blink type in a followup patch.

[commit-bot: I haz the power, 2015-05-26 11:16:20]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-05-26 11:17:18]

Patchset 1 (id:??) landed as
https://crrev.com/17d29fe84dc5e4645bd38e92d86ee3dd2bc47875
Cr-Commit-Position: refs/heads/master@{#331345}