diff options
| author | sidchat@google.com <sidchat@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-02-10 20:28:43 +0000 |
|---|---|---|
| committer | sidchat@google.com <sidchat@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-02-10 20:28:43 +0000 |
| commit | e654204795998e5e253b7022e2338ab01ebf7a97 (patch) | |
| tree | a5551e40051e0092e73ff41cefb817e14cbadfe2 /chrome/browser | |
| parent | f6c431bfa5437d699f94e20db399e931f35cc3f6 (diff) | |
| download | chromium_src-e654204795998e5e253b7022e2338ab01ebf7a97.zip chromium_src-e654204795998e5e253b7022e2338ab01ebf7a97.tar.gz chromium_src-e654204795998e5e253b7022e2338ab01ebf7a97.tar.bz2 | |
Disable context menu item "Open image in new tab" from context menu for thumbnail images for Most Visited page.
Issue=2608
Review URL: http://codereview.chromium.org/24003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@9505 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser')
| -rw-r--r-- | chrome/browser/tab_contents/render_view_context_menu_controller.cc | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/chrome/browser/tab_contents/render_view_context_menu_controller.cc b/chrome/browser/tab_contents/render_view_context_menu_controller.cc index 7a43ff3..0613191 100644 --- a/chrome/browser/tab_contents/render_view_context_menu_controller.cc +++ b/chrome/browser/tab_contents/render_view_context_menu_controller.cc @@ -134,6 +134,14 @@ bool RenderViewContextMenuController::IsCommandEnabled(int id) const { URLRequest::IsHandledURL(params_.image_url); case IDS_CONTENT_CONTEXT_OPENIMAGENEWTAB: + // The images shown in the most visited thumbnails do not currently open + // in a new tab as they should. Disabling this context menu option for + // now, as a quick hack, before we resolve this issue (Issue = 2608). + // TODO (sidchat): Enable this option once this issue is resolved. + if (params_.image_url.scheme() == "chrome-ui") + return false; + return true; + case IDS_CONTENT_CONTEXT_COPYIMAGELOCATION: return params_.image_url.is_valid(); |