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

Issue 2937953002: Chromad: Drop delay for policy refresh on startup (Closed)

Created:
3 years, 6 months ago by Thiemo Nagel
Modified:
3 years, 6 months ago
CC:
chromium-reviews, oshima+watch_chromium.org, davemoore+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Chromad: Drop delay for policy refresh on startup Policy fetch used to be slow and thus was delayed to prevent blocking login. Now that policy fetch has become faster, this CL removes the delay. BUG=732801 Review-Url: https://codereview.chromium.org/2937953002 Cr-Commit-Position: refs/heads/master@{#479371} Committed: https://chromium.googlesource.com/chromium/src/+/adacf601d936418bd1604e62c32f43258e2d7946

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -5 lines) Patch
M chrome/browser/chromeos/policy/active_directory_policy_manager.cc View 2 chunks +1 line, -5 lines 0 comments Download

Messages

Total messages: 15 (9 generated)
Thiemo Nagel
Hey Roman, could you please take a look? Thank you, Thiemo
3 years, 6 months ago (2017-06-14 13:06:01 UTC) #6
Roman Sorokin (ftl)
lgtm
3 years, 6 months ago (2017-06-14 13:31:59 UTC) #7
Thiemo Nagel
3 years, 6 months ago (2017-06-14 13:35:31 UTC) #8
Thiemo Nagel
On 2017/06/14 13:31:59, Roman Sorokin (ftl) wrote: > lgtm Thanks!
3 years, 6 months ago (2017-06-14 13:35:50 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2937953002/1
3 years, 6 months ago (2017-06-14 13:35:58 UTC) #11
commit-bot: I haz the power
3 years, 6 months ago (2017-06-14 13:41:12 UTC) #15
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/adacf601d936418bd1604e62c32f...

Powered by Google App Engine
This is Rietveld 408576698