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

Issue 6677093: add Profile Permissions test (Closed)

Created:
9 years, 9 months ago by jimhebert
Modified:
9 years, 7 months ago
Reviewers:
fes, seano
CC:
chromium-os-reviews_chromium.org, sosa+cc_chromium.org, seano+cc_chromium.org, ericli
Visibility:
Public.

Description

add Profile Permissions test Change-Id: I6e7d0ff72b44406b2c4648417d33e2173d82465e BUG=chromium-os:2777 TEST=run_remote_tests ... security_ProfilePermissions Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=d16a1d5

Patch Set 1 #

Patch Set 2 : fix control file descriptions #

Total comments: 1

Patch Set 3 : Fix copyright dates #

Unified diffs Side-by-side diffs Delta from patch set Stats (+67 lines, -0 lines) Patch
A client/site_tests/security_ProfilePermissions/control View 1 2 1 chunk +22 lines, -0 lines 0 comments Download
A client/site_tests/security_ProfilePermissions/security_ProfilePermissions.py View 1 2 1 chunk +44 lines, -0 lines 0 comments Download
M server/site_tests/suites/control.security View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
jimhebert
9 years, 9 months ago (2011-03-16 22:14:26 UTC) #1
seano
9 years, 9 months ago (2011-03-16 22:40:25 UTC) #2
LGTM. Nice test, if it's already found a bug :)

http://codereview.chromium.org/6677093/diff/3001/client/site_tests/security_P...
File client/site_tests/security_ProfilePermissions/control (right):

http://codereview.chromium.org/6677093/diff/3001/client/site_tests/security_P...
client/site_tests/security_ProfilePermissions/control:1: # Copyright (c) 2010
The Chromium OS Authors. All rights reserved.
2011

Powered by Google App Engine
This is Rietveld 408576698