Issue 6310012: Allow policy refresh rate to be configured through policy.

Issue 6310012: Allow policy refresh rate to be configured through policy. (Closed)

Created:
9 years, 11 months ago by Mattias Nissler (ping if slow)

Modified:
9 years, 7 months ago

Reviewers:
danno, Jakob Kummerow

CC:
chromium-reviews, Paweł Hajdan Jr.

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

Description

Allow policy refresh rate to be configured through policy. BUG=64877 TEST=Existing unittests; manual testing. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=71790

Patch Set 1 #

Patch Set 2 : Fix unit test. #

Patch Set 3 : Fix unit test. #

Created: 9 years, 11 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+493 lines, -287 lines)			Patch
M	chrome/app/policy/policy_templates.json	View		1 chunk	+11 lines, -0 lines	0 comments	Download
M	chrome/browser/policy/config_dir_policy_provider_unittest.cc	View		1 chunk	+4 lines, -1 line	0 comments	Download
M	chrome/browser/policy/configuration_policy_pref_store.cc	View		2 chunks	+4 lines, -0 lines	0 comments	Download
M	chrome/browser/policy/configuration_policy_pref_store_unittest.cc	View		1 chunk	+3 lines, -1 line	0 comments	Download
M	chrome/browser/policy/configuration_policy_provider.cc	View		1 chunk	+3 lines, -1 line	0 comments	Download
M	chrome/browser/policy/configuration_policy_provider_mac_unittest.cc	View		1 chunk	+4 lines, -1 line	0 comments	Download
M	chrome/browser/policy/configuration_policy_provider_win_unittest.cc	View		1 chunk	+4 lines, -1 line	0 comments	Download
M	chrome/browser/policy/configuration_policy_store_interface.h	View		1 chunk	+2 lines, -1 line	0 comments	Download
M	chrome/browser/policy/device_management_policy_cache.h	View	1	1 chunk	+1 line, -1 line	0 comments	Download
M	chrome/browser/policy/device_management_policy_cache.cc	View	1	1 chunk	+4 lines, -12 lines	0 comments	Download
M	chrome/browser/policy/device_management_policy_provider.h	View		8 chunks	+43 lines, -20 lines	0 comments	Download
M	chrome/browser/policy/device_management_policy_provider.cc	View	1	7 chunks	+195 lines, -168 lines	0 comments	Download
M	chrome/browser/policy/device_management_policy_provider_unittest.cc	View		8 chunks	+108 lines, -75 lines	0 comments	Download
M	chrome/browser/policy/device_token_fetcher.cc	View		2 chunks	+6 lines, -1 line	0 comments	Download
M	chrome/browser/policy/device_token_fetcher_unittest.cc	View		1 chunk	+24 lines, -0 lines	0 comments	Download
M	chrome/browser/policy/profile_policy_context.h	View		3 chunks	+20 lines, -1 line	0 comments	Download
M	chrome/browser/policy/profile_policy_context.cc	View		3 chunks	+44 lines, -0 lines	0 comments	Download
M	chrome/browser/prefs/browser_prefs.cc	View		2 chunks	+5 lines, -3 lines	0 comments	Download
M	chrome/common/policy_constants.h	View		1 chunk	+1 line, -0 lines	0 comments	Download
M	chrome/common/policy_constants.cc	View		1 chunk	+1 line, -0 lines	0 comments	Download
M	chrome/common/pref_names.h	View		1 chunk	+1 line, -0 lines	0 comments	Download
M	chrome/common/pref_names.cc	View		1 chunk	+5 lines, -0 lines	0 comments	Download

Messages

Total messages: 2 (0 generated)

Expand Messages | Collapse Messages

Mattias Nissler (ping if slow)

Please re-review. The first try had an issue with the refresh task looping endlessly on ...

9 years, 11 months ago (2011-01-19 11:15:57 UTC) #1

still LGTM

Expand Messages | Collapse Messages