[scheduler] Remove blink::scheduler::TaskQueue from public API.

third_party/WebKit/Source/platform/scheduler/child/webthread_impl_for_worker_scheduler.cc

Index: third_party/WebKit/Source/platform/scheduler/child/webthread_impl_for_worker_scheduler.cc
diff --git a/third_party/WebKit/Source/platform/scheduler/child/webthread_impl_for_worker_scheduler.cc b/third_party/WebKit/Source/platform/scheduler/child/webthread_impl_for_worker_scheduler.cc
index 7ecd037c9306cabcf4068b644f368b4d84e5d8e0..7d3e499448ad8b1f604ad3072d69161e467b2d33 100644
--- a/third_party/WebKit/Source/platform/scheduler/child/webthread_impl_for_worker_scheduler.cc
+++ b/third_party/WebKit/Source/platform/scheduler/child/webthread_impl_for_worker_scheduler.cc
@@ -10,7 +10,7 @@
 #include "base/single_thread_task_runner.h"
 #include "base/synchronization/waitable_event.h"
 #include "base/time/default_tick_clock.h"
-#include "public/platform/scheduler/base/task_queue.h"
+#include "platform/scheduler/base/task_queue.h"
 #include "platform/scheduler/child/scheduler_tqm_delegate_impl.h"
 #include "platform/scheduler/child/web_scheduler_impl.h"
 #include "platform/scheduler/child/web_task_runner_impl.h"
@@ -64,12 +64,12 @@ void WebThreadImplForWorkerScheduler::InitOnThread(
   // main thread?
   worker_scheduler_ = CreateWorkerScheduler();
   worker_scheduler_->Init();
-  task_runner_ = worker_scheduler_->DefaultTaskRunner();
+  task_runner_ = worker_scheduler_->DefaultTaskQueue();

[Sami, 2017/04/12 16:45:21]

Could we either rename all of these or none of these? This leaves us at a weird
halfway point.

[altimin, 2017/04/12 17:02:25]

I believe that it's mostly consistent (there will be a couple more patches
polishing everything).

*TaskQueue methods return TaskQueue and *TaskRunner methods return task runner.
In this case it's SingleThreadIdleTaskRunner (which is exposed to content/).

The thing happening here is that IdleTaskRunner() is a very different thing (I
wonder if we should change it) and this change only highlights existing pattern.

[Sami, 2017/04/12 17:42:02]

I was mainly referring to the fact that the variable names don't match the
getters (queue vs. runner). Ideally we'd just talk about one kind of thing since
the distinction isn't really relevant at this level.

[altimin, 2017/04/13 12:41:50]

Member variable names will be updated in a separate CL.

   idle_task_runner_ = worker_scheduler_->IdleTaskRunner();
   web_scheduler_.reset(new WebSchedulerImpl(
       worker_scheduler_.get(), worker_scheduler_->IdleTaskRunner(),
-      worker_scheduler_->DefaultTaskRunner(),
-      worker_scheduler_->DefaultTaskRunner()));
+      worker_scheduler_->DefaultTaskQueue(),
+      worker_scheduler_->DefaultTaskQueue()));
   base::MessageLoop::current()->AddDestructionObserver(this);
   web_task_runner_ = WebTaskRunnerImpl::Create(task_runner_);
   completion->Signal();