Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(147)

Issue 3245006: Sandboxing built-in flash... (Closed)

Created:
10 years, 3 months ago by cpu_(ooo_6.6-7.5)
Modified:
9 years, 7 months ago
CC:
chromium-reviews, jam, darin-cc_chromium.org, stuartmorgan+watch_chromium.org
Visibility:
Public.

Description

Sandboxing built-in flash This is the last change needed to have an experimental sandboxed flash for windows - Adds an export so flash can lower the token - Thightents the policy a bit - Sets a separate flash data directory. BUG=50796 TES=see bug Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=57899

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -12 lines) Patch
M chrome/common/sandbox_policy.cc View 3 chunks +9 lines, -9 lines 0 comments Download
M chrome/plugin/plugin_main.cc View 2 chunks +32 lines, -3 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
cpu_(ooo_6.6-7.5)
10 years, 3 months ago (2010-08-29 01:05:30 UTC) #1
nsylvain
LGTM. I assume you made sure that no untrusted threads were running before you call ...
10 years, 3 months ago (2010-08-29 18:44:53 UTC) #2
cpu_(ooo_6.6-7.5)
10 years, 3 months ago (2010-08-30 19:46:25 UTC) #3
That is an interesting point. Let me check that.


On 2010/08/29 18:44:53, nsylvain wrote:
> LGTM. I assume you made sure that no untrusted threads were running before you
> call RevertToSelf.

Powered by Google App Engine
This is Rietveld 408576698