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

Issue 7453062: Added another class 'EnterpriseTestSetTwo'to test another set of policy values. (Closed)

Created:
9 years, 4 months ago by aocampo
Modified:
9 years, 4 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

- Added another class 'EnterpriseTestSetTwo'to test another set of policy values. - Modified enterprise_helper_linux script to handle changing file permissions for multiple json files. BUG=NONE Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=95718

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Total comments: 6

Patch Set 4 : '' #

Patch Set 5 : '' #

Patch Set 6 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+265 lines, -2 lines) Patch
M enterprise.py View 1 2 3 4 5 1 chunk +261 lines, -0 lines 0 comments Download
M enterprise_helper_linux.py View 1 2 3 1 chunk +4 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
aocampo
9 years, 4 months ago (2011-08-02 00:10:27 UTC) #1
Nirnimesh
LGTM. Sorry for the delay. Please look out on the internal QA bots after committing ...
9 years, 4 months ago (2011-08-05 19:05:23 UTC) #2
aocampo
http://codereview.chromium.org/7453062/diff/6001/enterprise.py File enterprise.py (right): http://codereview.chromium.org/7453062/diff/6001/enterprise.py#newcode310 enterprise.py:310: class EnterpriseTestSetTwo(pyauto.PyUITest): On 2011/08/05 19:05:23, Nirnimesh wrote: > 'SetTwo' ...
9 years, 4 months ago (2011-08-05 23:04:31 UTC) #3
aocampo
9 years, 4 months ago (2011-08-05 23:05:21 UTC) #4
Nirnimesh
LGTM still
9 years, 4 months ago (2011-08-05 23:15:38 UTC) #5
commit-bot: I haz the power
Presubmit check for 7453062-14001 failed and returned exit status 1. Running presubmit commit checks ...
9 years, 4 months ago (2011-08-06 00:16:29 UTC) #6
commit-bot: I haz the power
9 years, 4 months ago (2011-08-06 03:15:31 UTC) #7
Change committed as 95718

Powered by Google App Engine
This is Rietveld 408576698