Fix bugs in the new task manager Impl so that ported old browsertests pass

chrome/browser/task_management/providers/web_contents/devtools_task.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 CHROME_BROWSER_TASK_MANAGEMENT_PROVIDERS_WEB_CONTENTS_DEVTOOLS_TASK_H_
 #define CHROME_BROWSER_TASK_MANAGEMENT_PROVIDERS_WEB_CONTENTS_DEVTOOLS_TASK_H_
 
 #include "base/macros.h"
-#include "chrome/browser/task_management/providers/web_contents/renderer_task.h"
+#include "chrome/browser/task_management/providers/web_contents/tab_contents_task.h"
 
 namespace task_management {
 
 // Defines a task manager representation of the developer tools WebContents.
-class DevToolsTask : public RendererTask {
+class DevToolsTask : public TabContentsTask {

[afakhry, 2016/05/18 21:53:29]

In the old TM, there wasn't a separate DevToolsResource. It was handled the same
way as TabContentsResources.

I left a separate DevToolsTask here just in case we want to do anything specific
with it in the future.

[ncarter (slow), 2016/05/18 22:21:01]

Makes sense.

  public:
   explicit DevToolsTask(content::WebContents* web_contents);
   ~DevToolsTask() override;
 
-  // task_management::RendererTask:
-  void UpdateTitle() override;
-  void UpdateFavicon() override;
-
  private:
   DISALLOW_COPY_AND_ASSIGN(DevToolsTask);
 };
 
 }  // namespace task_management
 
 #endif  // CHROME_BROWSER_TASK_MANAGEMENT_PROVIDERS_WEB_CONTENTS_DEVTOOLS_TASK_H_