DevTools: Extension context menu does appear for selected text in the devtools of the devtools.

chrome/browser/tab_contents/render_view_context_menu.cc

 // Copyright (c) 2011 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.
 
 #include <algorithm>
 #include <set>
 
 #include "chrome/browser/tab_contents/render_view_context_menu.h"
 
 #include "base/command_line.h"
@@ skipping 466 matching lines @@
 
   if (AppendCustomItems()) {
     // Don't add items for Pepper menu.
     if (!params_.custom_context.is_pepper_menu)
       AppendDeveloperItems();
     return;
   }
 
   // When no special node or text is selected and selection has no link,
   // show page items.
-  bool is_devtools = false;
   if (params_.media_type == WebContextMenuData::MediaTypeNone &&
       !has_link &&
       !params_.is_editable &&
       !has_selection) {
     if (!params_.page_url.is_empty()) {
-      is_devtools = IsDevToolsURL(params_.page_url);
+      bool is_devtools = IsDevToolsURL(params_.page_url);
       if (!is_devtools && !IsInternalResourcesURL(params_.page_url)) {
         AppendPageItems();
         // Merge in frame items if we clicked within a frame that needs them.
         if (!params_.frame_url.is_empty()) {
           is_devtools = IsDevToolsURL(params_.frame_url);
           if (!is_devtools && !IsInternalResourcesURL(params_.frame_url)) {
             menu_model_.AddSeparator();
             AppendFrameItems();
           }
         }
@@ skipping 31 matching lines @@
   }
 
   if (params_.is_editable)
     AppendEditableItems();
   else if (has_selection)
     AppendCopyItem();
 
   if (has_selection)
     AppendSearchProvider();
 
-  if (!is_devtools)
+  if (!IsDevToolsURL(params_.page_url))

[yurys, 2011/08/15 15:05:52]

It doesn't take into account IsDevToolsURL(params_.frame_url) check anymore, is
it intentional?

     AppendAllExtensionItems();
 
   AppendDeveloperItems();
 }
 
 void RenderViewContextMenu::LookUpInDictionary() {
   // Used only in the Mac port.
   NOTREACHED();
 }
 
@@ skipping 1278 matching lines @@
       g_browser_process->clipboard());
 }
 
 void RenderViewContextMenu::MediaPlayerActionAt(
     const gfx::Point& location,
     const WebMediaPlayerAction& action) {
   RenderViewHost* rvh = source_tab_contents_->render_view_host();
   rvh->Send(new ViewMsg_MediaPlayerActionAt(
       rvh->routing_id(), location, action));
 }