Issue 10500013: Delete old policy code after AsyncPolicyLoader refactor.

Issue 10500013: Delete old policy code after AsyncPolicyLoader refactor. (Closed)

Created:
8 years, 6 months ago by Joao da Silva

Modified:
8 years, 6 months ago

Reviewers:
Mattias Nissler (ping if slow)

CC:
chromium-reviews

Base URL:
http://git.chromium.org/chromium/src.git@master

Visibility:
Public.

More Reviews

Description

Delete old policy code after AsyncPolicyLoader refactor. BUG=130918 TEST=nothing breaks Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=140784

Patch Set 1 #

Patch Set 2 : Rebased #

Patch Set 3 : rebased #

Created: 8 years, 6 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+0 lines, -1130 lines)			Patch
D	chrome/browser/policy/asynchronous_policy_loader.h	View		1 chunk	+0 lines, -117 lines	0 comments	Download
D	chrome/browser/policy/asynchronous_policy_loader.cc	View		1 chunk	+0 lines, -122 lines	0 comments	Download
D	chrome/browser/policy/asynchronous_policy_loader_unittest.cc	View		1 chunk	+0 lines, -151 lines	0 comments	Download
D	chrome/browser/policy/asynchronous_policy_provider.h	View		1 chunk	+0 lines, -73 lines	0 comments	Download
D	chrome/browser/policy/asynchronous_policy_provider.cc	View		1 chunk	+0 lines, -67 lines	0 comments	Download
D	chrome/browser/policy/asynchronous_policy_provider_unittest.cc	View		1 chunk	+0 lines, -67 lines	0 comments	Download
D	chrome/browser/policy/asynchronous_policy_test_base.h	View	1	1 chunk	+0 lines, -68 lines	0 comments	Download
D	chrome/browser/policy/asynchronous_policy_test_base.cc	View		1 chunk	+0 lines, -26 lines	0 comments	Download
D	chrome/browser/policy/file_based_policy_loader.h	View		1 chunk	+0 lines, -89 lines	0 comments	Download
D	chrome/browser/policy/file_based_policy_loader.cc	View		1 chunk	+0 lines, -156 lines	0 comments	Download
D	chrome/browser/policy/file_based_policy_provider.h	View		1 chunk	+0 lines, -54 lines	0 comments	Download
D	chrome/browser/policy/file_based_policy_provider.cc	View		1 chunk	+0 lines, -24 lines	0 comments	Download
D	chrome/browser/policy/file_based_policy_provider_unittest.cc	View		1 chunk	+0 lines, -103 lines	0 comments	Download
M	chrome/chrome_browser.gypi	View	1	2 chunks	+0 lines, -8 lines	0 comments	Download
M	chrome/chrome_tests.gypi	View	1	2 chunks	+0 lines, -5 lines	0 comments	Download

Messages

Total messages: 4 (0 generated)

Expand Messages | Collapse Messages

Joao da Silva

CL 5/5: remove the old stuff This should be an auto-lgtm if the others are ...

8 years, 6 months ago (2012-06-03 15:09:20 UTC) #1

Mattias Nissler (ping if slow)

LGTM pending the prerequisite changes

8 years, 6 months ago (2012-06-04 09:38:13 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/10500013/12001

8 years, 6 months ago (2012-06-06 16:10:09 UTC) #3

Change committed as 140784

Expand Messages | Collapse Messages