New Task Manager - Phase 1.1: Implement Browser Process Task Providing

chrome/browser/task_management/providers/task_provider_observer.h

Index: chrome/browser/task_management/providers/task_provider_observer.h
diff --git a/chrome/browser/task_management/providers/task_provider_observer.h b/chrome/browser/task_management/providers/task_provider_observer.h
new file mode 100644
index 0000000000000000000000000000000000000000..9d4b95d06b96ecd10b475767fa77e2a38cc36f6a
--- /dev/null
+++ b/chrome/browser/task_management/providers/task_provider_observer.h
@@ -0,0 +1,27 @@
+// 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 CHROME_BROWSER_TASK_MANAGEMENT_PROVIDERS_TASK_PROVIDER_OBSERVER_H_
+#define CHROME_BROWSER_TASK_MANAGEMENT_PROVIDERS_TASK_PROVIDER_OBSERVER_H_
+
+#include "chrome/browser/task_management/providers/task.h"
+
+namespace task_management {
+
+// Defines an interface for observing tasks addition / removal.
+class TaskProviderObserver {
+ public:
+  TaskProviderObserver() {}
+  virtual ~TaskProviderObserver() {}
+
+  virtual void TaskAdded(Task* task) = 0;
+  virtual void TaskRemoved(Task* task) = 0;

[Lei Zhang, 2015/04/01 22:47:51]

Is |task| still valid when given to the observer? You may want to document pure
virtual methods, even if they appear obvious.

[afakhry, 2015/04/03 01:51:01]

You're right. I added more documentation.

+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(TaskProviderObserver);
+};
+
+}  // namespace task_management
+
+#endif  // CHROME_BROWSER_TASK_MANAGEMENT_PROVIDERS_TASK_PROVIDER_OBSERVER_H_