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

chrome/browser/task_management/providers/browser_process/browser_process_task_provider.cc

+// 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.
+
+#include "chrome/browser/task_management/providers/browser_process/browser_process_task_provider.h"
+
+namespace task_management {
+
+BrowserProcessTaskProvider::BrowserProcessTaskProvider()
+    : browser_process_task_() {

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

I don't think this is needed?

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

It's not needed, but it's good practice, and better documentation. It also helps
indexers like that of Eclipse to find where that object was created.

+}
+
+BrowserProcessTaskProvider::~BrowserProcessTaskProvider() {
+}
+
+Task* BrowserProcessTaskProvider::GetTaskOfUrlRequest(int origin_pid,
+                                                      int child_id,
+                                                      int route_id) {
+  if (origin_pid == 0 && child_id == -1)

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

Since I don't understand what the parameters to GetTaskOfUrlRequest() are
exactly, I also don't understand why these magic values designate the browser
process.

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

I added detailed comments to this function's declaration in the base class.

+    return &browser_process_task_;
+
+  return nullptr;
+}
+
+void BrowserProcessTaskProvider::StartUpdating() {
+  NotifyObserverTaskAdded(&browser_process_task_);
+}
+
+void BrowserProcessTaskProvider::StopUpdating() {
+  // There's nothing to do here. The browser process task live as long as the
+  // browser lives and when StopUpdating() is called the |observer_| has already
+  // been cleared.
+}
+
+}  // namespace task_management