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

Issue 6799022: Update mac sandbox *.sb file comments (Closed)

Created:
9 years, 8 months ago by dmac
Modified:
9 years, 7 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Update mac sandbox *.sb file comments BUG=NONE TEST=NONE Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=80810

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -9 lines) Patch
M chrome/browser/nacl_loader.sb View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/utility.sb View 2 chunks +3 lines, -3 lines 0 comments Download
M content/browser/worker.sb View 2 chunks +2 lines, -2 lines 0 comments Download
M content/renderer/renderer.sb View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
dmac
Just some updates while I was digging around. PTAL
9 years, 8 months ago (2011-04-06 19:39:13 UTC) #1
jeremy
LGTM
9 years, 8 months ago (2011-04-07 04:13:23 UTC) #2
dmac
John/Brett could I get a LTGM from an OWNER type person?
9 years, 8 months ago (2011-04-07 14:41:43 UTC) #3
Avi (use Gerrit)
content/ owner sez LGTM
9 years, 8 months ago (2011-04-07 14:42:39 UTC) #4
dmac
On 2011/04/07 14:42:39, Avi wrote: > content/ owner sez LGTM Avi, I don't see you ...
9 years, 8 months ago (2011-04-07 16:54:36 UTC) #5
jam
lgtm
9 years, 8 months ago (2011-04-07 16:54:59 UTC) #6
Avi (use Gerrit)
9 years, 8 months ago (2011-04-07 17:02:14 UTC) #7
No, you're not. Delusions of grandeur on my part. LGTM still for whatever it's
worth.

Powered by Google App Engine
This is Rietveld 408576698