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

Issue 6225001: AU: Don't allow stateful /etc/lsb-release override in normal boot mode... (Closed)

Created:
9 years, 11 months ago by petkov
Modified:
9 years, 7 months ago
Reviewers:
adlr
CC:
chromium-os-reviews_chromium.org, petkov, krisr, adlr
Visibility:
Public.

Description

AU: Don't allow stateful /etc/lsb-release override in normal boot mode... ... on official images except for the release track. BUG=chromium-os:6244 TEST=unit tests, tested on device Change-Id: I2d2a634c7b5ac7dd4619f52d1f79b6d88c359e46 Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=10d02dd

Patch Set 1 #

Total comments: 2

Patch Set 2 : ShouldLockDown and no rootfs track validation #

Unified diffs Side-by-side diffs Delta from patch set Stats (+69 lines, -57 lines) Patch
M omaha_request_params.h View 1 2 chunks +11 lines, -7 lines 0 comments Download
M omaha_request_params.cc View 1 5 chunks +20 lines, -15 lines 0 comments Download
M omaha_request_params_unittest.cc View 1 11 chunks +38 lines, -35 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
petkov
9 years, 11 months ago (2011-01-10 21:39:43 UTC) #1
adlr
http://codereview.chromium.org/6225001/diff/1/omaha_request_params.cc File omaha_request_params.cc (right): http://codereview.chromium.org/6225001/diff/1/omaha_request_params.cc#newcode196 omaha_request_params.cc:196: if (!IsOfficialBuild()) { perhaps: if (!IsOfficialBuild() || Dev_Mode_is_Active()) { ...
9 years, 11 months ago (2011-01-10 22:16:17 UTC) #2
petkov
PTAL. Simplified to checking a single ShouldLockDown method and removed rootfs track validation -- we ...
9 years, 11 months ago (2011-01-10 22:51:14 UTC) #3
adlr
9 years, 11 months ago (2011-01-10 22:55:24 UTC) #4
LGTM. thanks!

Powered by Google App Engine
This is Rietveld 408576698