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

Issue 19482020: Set configuration_policy=1 on Android. (Closed)

Created:
7 years, 5 months ago by Joao da Silva
Modified:
7 years, 5 months ago
Reviewers:
Yaron
CC:
chromium-reviews, Andrew T Wilson (Slow), Maria, aberent, Andrew Hayden (chromium.org)
Visibility:
Public.

Description

Set configuration_policy=1 on Android. BUG=247435 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=213284

Patch Set 1 #

Patch Set 2 : fixed unittest #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -2 lines) Patch
M build/common.gypi View 1 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/policy/policy_statistics_collector_unittest.cc View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (0 generated)
Joao da Silva
Features should have been enabled on trunk about 2 weeks before branch point, so here ...
7 years, 5 months ago (2013-07-22 15:23:43 UTC) #1
Joao da Silva
Yaron, can you have a quick look? The failing unittest has been fixed.
7 years, 5 months ago (2013-07-23 19:50:52 UTC) #2
Yaron
lgtm I know you've done measurements and this shouldn't impact it but +some folks who ...
7 years, 5 months ago (2013-07-23 21:15:37 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/joaodasilva@chromium.org/19482020/11001
7 years, 5 months ago (2013-07-23 21:26:29 UTC) #4
commit-bot: I haz the power
7 years, 5 months ago (2013-07-24 00:04:06 UTC) #5
Message was sent while issue was closed.
Change committed as 213284

Powered by Google App Engine
This is Rietveld 408576698