GuestViews should be able to specify task manager entries

chrome/browser/task_manager/guest_information.cc

Index: chrome/browser/task_manager/guest_information.cc
diff --git a/chrome/browser/task_manager/guest_information.cc b/chrome/browser/task_manager/guest_information.cc
index 131503557b08857d161284c1554b4bf58136c5e4..06ac06545311c5de4354b003c703815a5bb3f97c 100644
--- a/chrome/browser/task_manager/guest_information.cc
+++ b/chrome/browser/task_manager/guest_information.cc
@@ -20,6 +20,7 @@
 #include "content/public/browser/render_widget_host_iterator.h"
 #include "content/public/browser/site_instance.h"
 #include "content/public/browser/web_contents.h"
+#include "extensions/browser/guest_view/guest_view_base.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/gfx/image/image.h"
 #include "ui/gfx/image/image_skia.h"
@@ -62,12 +63,16 @@ Resource::Type GuestResource::GetType() const {
 
 base::string16 GuestResource::GetTitle() const {
   WebContents* web_contents = GetWebContents();
-  const int message_id = IDS_TASK_MANAGER_WEBVIEW_TAG_PREFIX;
-  if (web_contents) {
-    base::string16 title = util::GetTitleFromWebContents(web_contents);
-    return l10n_util::GetStringFUTF16(message_id, title);
+  if (!web_contents) {
+    const int message_id = IDS_TASK_MANAGER_WEBVIEW_TAG_PREFIX;
+    return l10n_util::GetStringFUTF16(message_id, base::string16());
   }
-  return l10n_util::GetStringFUTF16(message_id, base::string16());
+  extensions::GuestViewBase* guest =
+      extensions::GuestViewBase::FromWebContents(web_contents);
+  int message_id = guest->GetTaskPrefix() ? guest->GetTaskPrefix() :
+      IDS_TASK_MANAGER_WEBVIEW_TAG_PREFIX;
+  return l10n_util::GetStringFUTF16(message_id,
+                                    guest->GetTaskTitle());

[Charlie Reis, 2014/08/25 20:25:41]

Why do we need GetTaskTitle?  It looks like that's identical to
util::GetTitleFromWebContents(web_contents), which would let us avoid the code
duplication.

[Fady Samuel, 2014/08/25 23:05:52]

That code lives in chrome, this code lives in extensions.

[Charlie Reis, 2014/08/25 23:47:22]

But the only caller of this is in chrome, right?  Why do we need this in
extensions at all?

 }
 
 gfx::ImageSkia GuestResource::GetIcon() const {
@@ -93,7 +98,7 @@ GuestInformation::~GuestInformation() {}
 
 bool GuestInformation::CheckOwnership(WebContents* web_contents) {
   // Guest WebContentses are created and owned internally by the content layer.
-  return web_contents->IsSubframe();
+  return extensions::GuestViewBase::IsGuest(web_contents);
 }
 
 void GuestInformation::GetAll(const NewWebContentsCallback& callback) {
@@ -103,7 +108,7 @@ void GuestInformation::GetAll(const NewWebContentsCallback& callback) {
     if (widget->IsRenderView()) {
       content::RenderViewHost* rvh = content::RenderViewHost::From(widget);
       WebContents* web_contents = WebContents::FromRenderViewHost(rvh);
-      if (web_contents && web_contents->IsSubframe())
+      if (web_contents && extensions::GuestViewBase::IsGuest(web_contents))
         callback.Run(web_contents);
     }
   }