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

chrome/browser/tab_contents/render_view_context_menu.h

Index: chrome/browser/tab_contents/render_view_context_menu.h
diff --git a/chrome/browser/tab_contents/render_view_context_menu.h b/chrome/browser/tab_contents/render_view_context_menu.h
index e28bb76bb50f82069f4b5a7c1a2b174ab68a3ecb..5ef298d321afb5c713401f5a2fd0efd5ae1bf04b 100644
--- a/chrome/browser/tab_contents/render_view_context_menu.h
+++ b/chrome/browser/tab_contents/render_view_context_menu.h
@@ -195,6 +195,9 @@ class RenderViewContextMenu : public ui::SimpleMenuModel::Delegate,
       Profile* profile,
       bool can_cross_incognito);
 
+  // Gets the extension (if any) associated with the TabContents that we're
+  // in.

[not at google - send to devlin, 2012/01/23 05:43:51]

1 line

[benwells, 2012/01/23 06:42:23]

Done.

+  const Extension* GetExtension() const;
   // Gets the platform app (if any) associated with the TabContents that we're
   // in.
   const Extension* GetPlatformApp() const;

[not at google - send to devlin, 2012/01/23 05:43:51]

I reckon we don't need this method at all anymore. It's a bit of a conflation of
two checks (is there an extension, and if so, is it a platform app). From what I
can tell from looking at render_view_context_menu.cc, it's easily inlineable and
I think it would be clearer that way.

[benwells, 2012/01/23 06:42:23]

Done.