Enable devtools via context menu in platform apps and popup extensions.

chrome/browser/ui/views/extensions/extension_popup.cc

Index: chrome/browser/ui/views/extensions/extension_popup.cc
diff --git a/chrome/browser/ui/views/extensions/extension_popup.cc b/chrome/browser/ui/views/extensions/extension_popup.cc
index a3e984d34855bda92f716596f096f52f084bb1d3..e794f5403cdd9a2f2821398dc1621caf86fa1d28 100644
--- a/chrome/browser/ui/views/extensions/extension_popup.cc
+++ b/chrome/browser/ui/views/extensions/extension_popup.cc
@@ -57,6 +57,18 @@ ExtensionPopup::ExtensionPopup(
   // Listen for the containing view calling window.close();
   registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_HOST_VIEW_SHOULD_CLOSE,
                  content::Source<Profile>(host->profile()));
+
+  // Listen for the dev tools closing, so we can close this window if it is
+  // being inspected and the inspector is closed.
+  registrar_.Add(this, content::NOTIFICATION_DEVTOOLS_WINDOW_CLOSING,
+      content::Source<content::BrowserContext>(host->profile()));
+
+  if (!inspect_with_devtools_) {
+    // Listen for the dev tools opening on this popup, so we can stop it going
+    // away when the dev tools get focus.
+    registrar_.Add(this, content::NOTIFICATION_DEVTOOLS_WINDOW_OPENING,
+                   content::Source<Profile>(host->profile()));
+  }
 }
 
 ExtensionPopup::~ExtensionPopup() {
@@ -81,9 +93,20 @@ void ExtensionPopup::Observe(int type,
       // Make sure its the devtools window that inspecting our popup.

[msw, 2012/02/07 00:34:57]

nit: Please change "its" to "it's" or "it is", thanks.

[benwells, 2012/02/08 03:49:16]

Done.

       // Widget::Close posts a task, which should give the devtools window a
       // chance to finish detaching from the inspected RenderViewHost.
-      if (content::Details<RenderViewHost>(
-              host()->render_view_host()) == details)
+      if (content::Details<RenderViewHost>(host()->render_view_host()) ==
+          details) {
         GetWidget()->Close();
+      }
+      break;
+    case content::NOTIFICATION_DEVTOOLS_WINDOW_OPENING:
+      // First check that the devtools are being opened on this popup.
+      if (content::Details<RenderViewHost>(host()->render_view_host()) ==
+          details) {
+        // Set inspect_with_devtools_ so the popup will be kept open while
+        // the devtools are open.
+        inspect_with_devtools_ = true;
+        set_close_on_deactivate(false);
+      }
       break;
     default:
       NOTREACHED() << L"Received unexpected notification";
@@ -153,9 +176,6 @@ void ExtensionPopup::ShowBubble() {
   views::WidgetFocusManager::GetInstance()->AddFocusChangeListener(this);
 
   if (inspect_with_devtools_) {

[msw, 2012/02/07 00:34:57]

nit: you can remove these braces, if you'd like.

-    // Listen for the the devtools window closing.
-    registrar_.Add(this, content::NOTIFICATION_DEVTOOLS_WINDOW_CLOSING,
-        content::Source<content::BrowserContext>(host()->profile()));
     DevToolsWindow::ToggleDevToolsWindow(host()->render_view_host(),
         DEVTOOLS_TOGGLE_ACTION_SHOW_CONSOLE);
   }