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

chrome/browser/task_management/providers/web_contents/devtools_task.h

Index: chrome/browser/task_management/providers/web_contents/devtools_task.h
diff --git a/chrome/browser/task_management/providers/web_contents/devtools_task.h b/chrome/browser/task_management/providers/web_contents/devtools_task.h
index 08cd5825c70a01396706a7617b235fe9781a64d5..e98967bdcf6eb3aad4f0ebe3097e4c115b84b450 100644
--- a/chrome/browser/task_management/providers/web_contents/devtools_task.h
+++ b/chrome/browser/task_management/providers/web_contents/devtools_task.h
@@ -6,20 +6,16 @@
 #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);
 };