Chromium Code Reviews

Issue 562073002: Forward kPpapiInProcess flag to zygote process (Closed)

Created:
6 years, 3 months ago by landell
Modified:
6 years, 3 months ago
Reviewers:
jln (very slow on Chromium), jam, Markus (顧孟勤)
CC:
chromium-reviews, darin-cc_chromium.org, Mostyn Bramley-Moore
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Forward kPpapiInProcess flag to zygote process Make sure this flag can be used for out of sandbox plugin setup. BUG= Committed: https://crrev.com/5dfcebfb91df269e8e1b09cb0b19f843921e763b Cr-Commit-Position: refs/heads/master@{#294616}

Patch Set 1 #

Patch Set 2 : Put zygote flags in alphabetical order #

Unified diffs Side-by-side diffs Stats (+6 lines, -7 lines)
M content/browser/zygote_host/zygote_host_impl_linux.cc View 1 chunk +6 lines, -7 lines 0 comments

Messages

Total messages: 8 (2 generated)
landell
This patch makes sense to Opera since we are doing file based plugin configuration which ...
6 years, 3 months ago (2014-09-11 07:14:13 UTC) #2
jln (very slow on Chromium)
lgtm (If you don't mind re-alphabetizing this list of switches, that would be nice).
6 years, 3 months ago (2014-09-11 18:33:43 UTC) #3
jam
lgtm
6 years, 3 months ago (2014-09-12 04:06:46 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patchset/562073002/20001
6 years, 3 months ago (2014-09-12 16:30:44 UTC) #6
commit-bot: I haz the power
Committed patchset #2 (id:20001) as a83251f9e448547cd4d2569cd1a20564aee0141c
6 years, 3 months ago (2014-09-12 18:16:33 UTC) #7
commit-bot: I haz the power
6 years, 3 months ago (2014-09-12 18:30:10 UTC) #8
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/5dfcebfb91df269e8e1b09cb0b19f843921e763b
Cr-Commit-Position: refs/heads/master@{#294616}

Powered by Google App Engine