Use TaskScheduler instead of blocking pool in offline_page_model_factory.cc.

chrome/browser/android/offline_pages/offline_page_model_factory.cc

Index: chrome/browser/android/offline_pages/offline_page_model_factory.cc
diff --git a/chrome/browser/android/offline_pages/offline_page_model_factory.cc b/chrome/browser/android/offline_pages/offline_page_model_factory.cc
index 089b5583289439decfe71af308d637c35b86927a..766080215b03a5c9167d51444658eb598dd17124 100644
--- a/chrome/browser/android/offline_pages/offline_page_model_factory.cc
+++ b/chrome/browser/android/offline_pages/offline_page_model_factory.cc
@@ -9,14 +9,13 @@
 #include "base/files/file_path.h"
 #include "base/memory/singleton.h"
 #include "base/sequenced_task_runner.h"
-#include "base/threading/sequenced_worker_pool.h"
+#include "base/task_scheduler/post_task.h"
 #include "chrome/browser/profiles/incognito_helpers.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/chrome_constants.h"
 #include "components/keyed_service/content/browser_context_dependency_manager.h"
 #include "components/offline_pages/core/offline_page_metadata_store_sql.h"
 #include "components/offline_pages/core/offline_page_model_impl.h"
-#include "content/public/browser/browser_thread.h"
 
 namespace offline_pages {
 
@@ -42,8 +41,9 @@ KeyedService* OfflinePageModelFactory::BuildServiceInstanceFor(
     content::BrowserContext* context) const {
   Profile* profile = Profile::FromBrowserContext(context);
   scoped_refptr<base::SequencedTaskRunner> background_task_runner =
-      content::BrowserThread::GetBlockingPool()->GetSequencedTaskRunner(
-          content::BrowserThread::GetBlockingPool()->GetSequenceToken());
+      base::CreateSequencedTaskRunnerWithTraits(
+          base::TaskTraits().MayBlock().WithPriority(
+              base::TaskPriority::BACKGROUND));

[dewittj, 2017/04/24 18:44:11]

The tasks on this task runner will sometimes be used to populate UI from a
database, so we were using the blocking pool.  But the comments in TaskPriority
are confusing and suggest that maybe this should be USER_VISIBLE.

[fdoray, 2017/04/26 19:33:06]

Switched to no explicit priority. Priority will be inherited from the calling
context. This is appropriate for code that may be used in different contexts.

 
   base::FilePath store_path =
       profile->GetPath().Append(chrome::kOfflinePageMetadataDirname);