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

Issue 6523058: New policy protobuf protocol. (Closed)

Created:
9 years, 10 months ago by Jakob Kummerow
Modified:
9 years, 6 months ago
Reviewers:
danno
CC:
chromium-reviews, darin-cc_chromium.org, amit, pam+watch_chromium.org, cbentzel+watch_chromium.org, Paweł Hajdan Jr.
Visibility:
Public.

Description

New policy protobuf protocol. (Second attempt to land http://codereview.chromium.org/6409040/) - cloud_policy.proto autogenerated from policy_templats.json - C++ method decoding the protobuf also autogenerated from policy_templates.json - changed policy fetching mechanism to fetch new-style policy protobufs BUG=68309, chromium-os:11253, chromium-os:11255 TEST=CloudPolicyCacheTest.*; also manual test against python testserver Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=75106

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2158 lines, -1339 lines) Patch
A chrome/app/policy/cloud_policy_codegen.gyp View 1 chunk +202 lines, -0 lines 0 comments Download
M chrome/app/policy/policy_templates.gypi View 1 chunk +1 line, -91 lines 0 comments Download
M chrome/app/policy/policy_templates.json View 63 chunks +84 lines, -13 lines 0 comments Download
M chrome/app/policy/syntax_check_policy_template_json.py View 6 chunks +44 lines, -9 lines 0 comments Download
M chrome/browser/policy/asynchronous_policy_provider.cc View 1 chunk +1 line, -1 line 0 comments Download
A chrome/browser/policy/cloud_policy_cache.h View 1 chunk +141 lines, -0 lines 0 comments Download
A chrome/browser/policy/cloud_policy_cache.cc View 1 chunk +429 lines, -0 lines 0 comments Download
A chrome/browser/policy/cloud_policy_cache_unittest.cc View 1 chunk +660 lines, -0 lines 0 comments Download
M chrome/browser/policy/configuration_policy_provider.h View 3 chunks +10 lines, -1 line 0 comments Download
M chrome/browser/policy/configuration_policy_provider.cc View 2 chunks +14 lines, -1 line 0 comments Download
M chrome/browser/policy/configuration_policy_store_interface.h View 1 chunk +0 lines, -2 lines 0 comments Download
M chrome/browser/policy/device_management_backend.h View 2 chunks +10 lines, -0 lines 0 comments Download
M chrome/browser/policy/device_management_backend_impl.h View 2 chunks +10 lines, -0 lines 0 comments Download
M chrome/browser/policy/device_management_backend_impl.cc View 8 chunks +81 lines, -53 lines 0 comments Download
M chrome/browser/policy/device_management_backend_mock.h View 1 chunk +1 line, -0 lines 0 comments Download
D chrome/browser/policy/device_management_policy_cache.h View 1 chunk +0 lines, -94 lines 0 comments Download
D chrome/browser/policy/device_management_policy_cache.cc View 1 chunk +0 lines, -270 lines 0 comments Download
D chrome/browser/policy/device_management_policy_cache_unittest.cc View 1 chunk +0 lines, -321 lines 0 comments Download
M chrome/browser/policy/device_management_policy_provider.h View 3 chunks +6 lines, -3 lines 0 comments Download
M chrome/browser/policy/device_management_policy_provider.cc View 8 chunks +48 lines, -16 lines 0 comments Download
M chrome/browser/policy/device_management_policy_provider_unittest.cc View 12 chunks +63 lines, -44 lines 0 comments Download
M chrome/browser/policy/mock_device_management_backend.h View 3 chunks +25 lines, -0 lines 0 comments Download
M chrome/browser/policy/profile_policy_context.cc View 2 chunks +4 lines, -1 line 0 comments Download
D chrome/browser/policy/proto/cloud_policy.proto View 1 chunk +0 lines, -221 lines 0 comments Download
M chrome/browser/policy/proto/device_management_backend.proto View 6 chunks +85 lines, -47 lines 0 comments Download
M chrome/browser/policy/proto/device_management_local.proto View 1 chunk +8 lines, -4 lines 0 comments Download
D chrome/browser/policy/proto/device_management_proto.gyp View 1 chunk +0 lines, -81 lines 0 comments Download
M chrome/chrome.gyp View 3 chunks +2 lines, -3 lines 0 comments Download
M chrome/chrome_browser.gypi View 5 chunks +3 lines, -6 lines 0 comments Download
M chrome/chrome_common.gypi View 3 chunks +3 lines, -2 lines 0 comments Download
M chrome/chrome_dll.gypi View 1 chunk +1 line, -1 line 0 comments Download
M chrome/chrome_exe.gypi View 2 chunks +4 lines, -4 lines 0 comments Download
M chrome/chrome_tests.gypi View 21 chunks +3 lines, -20 lines 0 comments Download
M chrome/tools/build/generate_policy_source.py View 5 chunks +210 lines, -24 lines 0 comments Download
M chrome_frame/chrome_frame.gyp View 2 chunks +2 lines, -2 lines 0 comments Download
M net/net.gyp View 1 chunk +1 line, -1 line 0 comments Download
M net/tools/testserver/device_management.py View 3 chunks +2 lines, -3 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Jakob Kummerow
For your reviewing convenience: the diff to the first attempt is this: diff --git a/chrome/app/policy/cloud_policy_codegen.gyp ...
9 years, 10 months ago (2011-02-16 12:09:38 UTC) #1
danno
9 years, 10 months ago (2011-02-16 12:13:37 UTC) #2
LGTM. Let's see what the build bots say.

Powered by Google App Engine
This is Rietveld 408576698