New Task Manager - Phase 1.3.2.B: Implement Tab Contents Task Providing (Prerender)

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

Index: chrome/browser/task_management/providers/web_contents/web_contents_tags_manager.h
diff --git a/chrome/browser/task_management/providers/web_contents/web_contents_tags_manager.h b/chrome/browser/task_management/providers/web_contents/web_contents_tags_manager.h
index 62ac3cff62b7bee0c126dd6c04c1f2bee859bca9..d64b6a738cc56e05ebf80b1f0aa5fa82274c8f88 100644
--- a/chrome/browser/task_management/providers/web_contents/web_contents_tags_manager.h
+++ b/chrome/browser/task_management/providers/web_contents/web_contents_tags_manager.h
@@ -33,6 +33,12 @@ class WebContentsTagsManager {
   void SetProvider(WebContentsTaskProvider* provider);
   void ClearProvider();
 
+  // This should be called by the destructor of certain tags whose destruction

[ncarter (slow), 2015/06/19 19:50:51]

If you move the call to ClearTag() as I suggest elsewhere, make sure update this
comment too.

[afakhry, 2015/06/19 21:49:22]

Done.

+  // doesn't not correspond to the destruction of their WebContents (like the

[ncarter (slow), 2015/06/19 19:50:51]

"doesn't not"

[afakhry, 2015/06/19 21:49:22]

Done.

+  // PrerenderTag). In this case the provider (if any) must be manually cleared,
+  // or else the corresponding task for the |tag| will continue to exist.
+  void ClearFromProvider(WebContentsTag* tag);
+
   const std::set<WebContentsTag*>& tracked_tags() const {
     return tracked_tags_;
   }