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

chrome/browser/tab_contents/render_view_context_menu.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_TAB_CONTENTS_RENDER_VIEW_CONTEXT_MENU_H_
 #define CHROME_BROWSER_TAB_CONTENTS_RENDER_VIEW_CONTEXT_MENU_H_
 #pragma once
 
 #include <map>
 #include <string>
@@ skipping 177 matching lines @@
   static bool ExtensionContextAndPatternMatch(
       const ContextMenuParams& params,
       ExtensionMenuItem::ContextList contexts,
       const URLPatternSet& target_url_patterns);
   static ExtensionMenuItem::List GetRelevantExtensionItems(
       const ExtensionMenuItem::List& items,
       const ContextMenuParams& params,
       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.

   void AppendPlatformAppItems();
   bool AppendCustomItems();
   void AppendDeveloperItems();
   void AppendLinkItems();
   void AppendImageItems();
   void AppendAudioItems();
   void AppendVideoItems();
   void AppendMediaItems();
   void AppendPluginItems();
   void AppendPageItems();
@@ skipping 72 matching lines @@
   // An observer that disables menu items when print preview is active.
   scoped_ptr<PrintPreviewContextMenuObserver> print_preview_menu_observer_;
 
   // Our observers.
   mutable ObserverList<RenderViewContextMenuObserver> observers_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderViewContextMenu);
 };
 
 #endif  // CHROME_BROWSER_TAB_CONTENTS_RENDER_VIEW_CONTEXT_MENU_H_