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

Unified Diff: chrome/browser/policy/cloud/external_policy_data_updater.cc

Issue 23868021: Prepare ExternalPolicyDataUpdater and ResourceCache for blocking pool (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Re-uploading due to rietveld flake. Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/policy/cloud/external_policy_data_updater.cc
diff --git a/chrome/browser/policy/cloud/external_policy_data_updater.cc b/chrome/browser/policy/cloud/external_policy_data_updater.cc
index 0f57c71775ba0c3a73fe98b71904f2e36299d23c..da2e8f18eb84c67331a7efe5250077c2eface19b 100644
--- a/chrome/browser/policy/cloud/external_policy_data_updater.cc
+++ b/chrome/browser/policy/cloud/external_policy_data_updater.cc
@@ -304,7 +304,7 @@ ExternalPolicyDataUpdater::ExternalPolicyDataUpdater(
}
ExternalPolicyDataUpdater::~ExternalPolicyDataUpdater() {
- DCHECK(CalledOnValidThread());
+ DCHECK(task_runner_->RunsTasksOnCurrentThread());
shutting_down_ = true;
STLDeleteValues(&job_map_);
}
@@ -313,7 +313,7 @@ void ExternalPolicyDataUpdater::FetchExternalData(
const std::string key,
const Request& request,
const FetchSuccessCallback& callback) {
- DCHECK(CalledOnValidThread());
+ DCHECK(task_runner_->RunsTasksOnCurrentThread());
// Check whether a job exists for this |key| already.
FetchJob* job = job_map_[key];
@@ -338,7 +338,7 @@ void ExternalPolicyDataUpdater::FetchExternalData(
void ExternalPolicyDataUpdater::CancelExternalDataFetch(
const std::string& key) {
- DCHECK(CalledOnValidThread());
+ DCHECK(task_runner_->RunsTasksOnCurrentThread());
// If a |job| exists for this |key|, delete it. If the |job| is on the queue,
// its WeakPtr will be invalidated and skipped by StartNextJobs(). If |job| is
« no previous file with comments | « chrome/browser/policy/cloud/external_policy_data_updater.h ('k') | chrome/browser/policy/cloud/resource_cache.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698