scheduler: Implement task observers

content/renderer/scheduler/webthread_impl_for_scheduler.h

+// Copyright 2015 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.
+
+#ifndef CONTENT_RENDERER_SCHEDULER_WEBTHREAD_IMPL_FOR_SCHEDULER_H_
+#define CONTENT_RENDERER_SCHEDULER_WEBTHREAD_IMPL_FOR_SCHEDULER_H_
+
+#include "content/child/webthread_impl.h"
+
+#include "base/containers/scoped_ptr_hash_map.h"
+
+namespace content {
+namespace internal {
+class NestedTaskObserver;
+}
+
+class RendererScheduler;
+
+class CONTENT_EXPORT WebThreadImplForScheduler : public WebThreadBase {
+ public:
+  explicit WebThreadImplForScheduler(RendererScheduler* scheduler);
+  virtual ~WebThreadImplForScheduler();
+
+  // blink::WebThread implementation.
+  virtual blink::PlatformThreadId threadId() const;
+  virtual void addTaskObserver(TaskObserver* observer);
+  virtual void removeTaskObserver(TaskObserver* observer);

[rmcilroy, 2015/02/19 11:47:49]

Should these be marked override instead (they are overriding WebThreadBase, not
implementing blink::WebThread)?

[Sami, 2015/02/19 16:31:09]

True, I guess I was being overly paranoid. Fixed.

+
+  // WebThreadBase implementation.
+  base::SingleThreadTaskRunner* TaskRunner() const override;
+
+ private:
+  base::MessageLoop* MessageLoop() const override;
+
+  typedef base::ScopedPtrHashMap<blink::WebThread::TaskObserver*,
+                                 internal::NestedTaskObserver>
+      NestedTaskObserverMap;
+  NestedTaskObserverMap nested_task_observer_map_;
+
+  scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
+  RendererScheduler* scheduler_;  // Not owned.
+  blink::PlatformThreadId thread_id_;
+};
+
+}  // namespace content
+
+#endif  // CONTENT_RENDERER_SCHEDULER_WEBTHREAD_IMPL_FOR_SCHEDULER_H_