tree c92036107a0d54d3447eb1a4bab3bebf4144d7f7
parent b0a4e9ced63477edba613e8d6ebfe04d038c9b8a
author Tommy Li <tommycli@chromium.org> 1624917795 +0000
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1624917795 +0000

Revert "Load all updater's group policy settings into memory."

This reverts commit 6dfa212831e306a0a5aa6d7b2f29142de6332c13.

Reason for revert: May have caused the Windows policy bot failures starting from 56509 here:
https://ci.chromium.org/p/chromium/builders/ci/Win10%20Tests%20x64?limit=100

Original change's description:
> Load all updater's group policy settings into memory.
>
> Caching policy values in memory because:
> 1) To be consistent with other policy providers.
> 2) Only do EnterCriticalPolicySection() during initialization and
> skip it for all subsequent policy queries.
> EnterCriticalPolicySection() could be expensive in certain situations.
>
> Bug: 1221348
> Change-Id: Icbc6a330dfd69a23f60efbf609e94381c290c8f1
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2988823
> Reviewed-by: Sorin Jianu <sorin@chromium.org>
> Commit-Queue: Xiaoling Bao <xiaolingbao@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#896356}

Bug: 1221348, 1224683
Change-Id: I077f7d67d55f286df2c5277034c59d7e1c7baa6f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2993176
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Owners-Override: Tommy Li <tommycli@chromium.org>
Reviewed-by: Sorin Jianu <sorin@chromium.org>
Commit-Queue: Tommy Li <tommycli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#896667}