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

Issue 137903004: Removed the old policy protobufs location. (Closed)

Created:
6 years, 11 months ago by Joao da Silva
Modified:
6 years, 11 months ago
CC:
chromium-reviews, extensions-reviews_chromium.org, nkostylev+watch_chromium.org, oshima+watch_chromium.org, chromium-apps-reviews_chromium.org, stevenjb+watch_chromium.org, davemoore+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Removed the old policy protobufs location. The policy protobufs were duplicated at chrome/browser/policy/proto so that the protofiles ebuild of ChromeOS could still fetch them. That ebuild has been fixed in a separate CL at: https://chromium-review.googlesource.com/#/c/182300/ BUG=327345 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=245133

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -1510 lines) Patch
D chrome/browser/chromeos/policy/proto/PRESUBMIT.py View 1 chunk +0 lines, -67 lines 0 comments Download
D chrome/browser/policy/proto/PRESUBMIT.py View 1 chunk +0 lines, -67 lines 0 comments Download
D chrome/browser/policy/proto/chromeos/chrome_device_policy.proto View 1 chunk +0 lines, -585 lines 0 comments Download
D chrome/browser/policy/proto/chromeos/install_attributes.proto View 1 chunk +0 lines, -20 lines 0 comments Download
D chrome/browser/policy/proto/cloud/chrome_extension_policy.proto View 1 chunk +0 lines, -43 lines 0 comments Download
D chrome/browser/policy/proto/cloud/device_management_backend.proto View 1 chunk +0 lines, -630 lines 0 comments Download
D chrome/browser/policy/proto/cloud/device_management_local.proto View 1 chunk +0 lines, -31 lines 0 comments Download
D components/policy/proto/PRESUBMIT.py View 1 chunk +0 lines, -67 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Joao da Silva
PTAL, thanks!
6 years, 11 months ago (2014-01-14 10:37:43 UTC) #1
Mattias Nissler (ping if slow)
LGTM
6 years, 11 months ago (2014-01-14 21:04:31 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/joaodasilva@chromium.org/137903004/1
6 years, 11 months ago (2014-01-15 12:57:57 UTC) #3
commit-bot: I haz the power
6 years, 11 months ago (2014-01-16 05:14:45 UTC) #4
Message was sent while issue was closed.
Change committed as 245133

Powered by Google App Engine
This is Rietveld 408576698