Reduce/remove usage of BrowserThread in content/browser/loader.

content/browser/loader/resource_requester_info.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/loader/resource_requester_info.h"
 
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "content/browser/appcache/chrome_appcache_service.h"
 #include "content/browser/blob_storage/chrome_blob_storage_context.h"
+#include "content/browser/loader/loader_globals.h"
 #include "content/browser/service_worker/service_worker_context_wrapper.h"
-#include "content/public/browser/browser_thread.h"
 #include "content/public/browser/resource_context.h"
 #include "content/public/common/browser_side_navigation_policy.h"
 #include "storage/browser/fileapi/file_system_context.h"
 
 namespace content {
 
 namespace {
 
 void GetContextsCallbackForNavigationPreload(
     scoped_refptr<ServiceWorkerContextWrapper> service_worker_context,
@@ skipping 19 matching lines @@
       appcache_service_(appcache_service),
       blob_storage_context_(blob_storage_context),
       file_system_context_(file_system_context),
       service_worker_context_(service_worker_context),
       get_contexts_callback_(get_contexts_callback) {}
 
 ResourceRequesterInfo::~ResourceRequesterInfo() {}
 
 void ResourceRequesterInfo::set_filter(
     base::WeakPtr<ResourceMessageFilter> filter) {
-  DCHECK_CURRENTLY_ON(BrowserThread::IO);
+  DCHECK(
+      LoaderGlobals::Get()->io_thread_task_runner()->BelongsToCurrentThread());

[jam, 2017/03/29 15:44:33]

i think we can take this out, there's one caller inside content and that one
already checks its thread (and it's somewhat redundant anyways)

[ananta, 2017/03/29 19:41:04]

Done.

   DCHECK_EQ(RequesterType::RENDERER, type_);
   filter_ = filter;
 }
 
 void ResourceRequesterInfo::GetContexts(
     ResourceType resource_type,
     ResourceContext** resource_context,
     net::URLRequestContext** request_context) const {
   get_contexts_callback_.Run(resource_type, resource_context, request_context);
 }
@@ skipping 58 matching lines @@
     DCHECK(get_contexts_callback);
   }
 
   return scoped_refptr<ResourceRequesterInfo>(new ResourceRequesterInfo(
       RequesterType::NAVIGATION_PRELOAD, -1, nullptr /* appcache_service */,
       nullptr /* blob_storage_context */, nullptr /* file_system_context */,
       original_request_info->service_worker_context(), get_contexts_callback));
 }
 
 }  // namespace content