Introduce TaskRunner::RunsTasksInCurrentSequence()

content/browser/dom_storage/dom_storage_task_runner.cc

Index: content/browser/dom_storage/dom_storage_task_runner.cc
diff --git a/content/browser/dom_storage/dom_storage_task_runner.cc b/content/browser/dom_storage/dom_storage_task_runner.cc
index 738be983203a2146d0b873105597e8a10a15e38a..412f2562c8387b7accc23053d466c696fc77a6f4 100644
--- a/content/browser/dom_storage/dom_storage_task_runner.cc
+++ b/content/browser/dom_storage/dom_storage_task_runner.cc
@@ -24,7 +24,7 @@ DOMStorageWorkerPoolTaskRunner::DOMStorageWorkerPoolTaskRunner(
 
 DOMStorageWorkerPoolTaskRunner::~DOMStorageWorkerPoolTaskRunner() = default;
 
-bool DOMStorageWorkerPoolTaskRunner::RunsTasksOnCurrentThread() const {
+bool DOMStorageWorkerPoolTaskRunner::RunsTasksInCurrentSequence() const {
   // It is valid for an implementation to always return true.
   return true;

[gab, 2017/04/18 15:00:33]

We no longer want it to be okay to always return true (except in unittests that
individually decide not to care).

I think this should forward the check to
|primary_sequence_->RunsTasksInCurrentSequence()| per PostDelayedTask() doing
that.

If that doesn't work it should at least return the combination of
|primary_sequence_->RunsTasksInCurrentSequence() ||
commit_sequence_->RunsTasksInCurrentSequence()|

 }
@@ -68,8 +68,8 @@ MockDOMStorageTaskRunner::MockDOMStorageTaskRunner(
 
 MockDOMStorageTaskRunner::~MockDOMStorageTaskRunner() = default;
 
-bool MockDOMStorageTaskRunner::RunsTasksOnCurrentThread() const {
-  return task_runner_->RunsTasksOnCurrentThread();
+bool MockDOMStorageTaskRunner::RunsTasksInCurrentSequence() const {
+  return task_runner_->RunsTasksInCurrentSequence();
 }
 
 bool MockDOMStorageTaskRunner::PostDelayedTask(