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

Issue 172463003: Copy the job factory for the SystemPolicyRequestContext. (Closed)

Created:
6 years, 10 months ago by Joao da Silva
Modified:
6 years, 10 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Copy the job factory for the SystemPolicyRequestContext. The SystemPolicyRequestContext wraps the system request context and is used to make cloud policy requests that shouldn't be related to a particular user. Some tests intercept requests sent via the system request context. By copying the job factory, those interceptors will also see the requests going through the SystemPolicyRequestContext. BUG=None Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=252095

Patch Set 1 #

Patch Set 2 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -0 lines) Patch
M components/policy/core/common/cloud/system_policy_request_context.cc View 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Joao da Silva
PTAL, thanks!
6 years, 10 months ago (2014-02-19 15:31:14 UTC) #1
Andrew T Wilson (Slow)
lgtm
6 years, 10 months ago (2014-02-19 15:45:11 UTC) #2
Joao da Silva
The CQ bit was checked by joaodasilva@chromium.org
6 years, 10 months ago (2014-02-19 16:01:33 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/joaodasilva@chromium.org/172463003/30001
6 years, 10 months ago (2014-02-19 16:01:53 UTC) #4
commit-bot: I haz the power
6 years, 10 months ago (2014-02-19 21:17:09 UTC) #5
Message was sent while issue was closed.
Change committed as 252095

Powered by Google App Engine
This is Rietveld 408576698