diff options
| author | tc@google.com <tc@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-12 21:38:59 +0000 |
|---|---|---|
| committer | tc@google.com <tc@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-12 21:38:59 +0000 |
| commit | b39881224053fa1000f77b2543af51afda17f0cf (patch) | |
| tree | a6ff6854445955436af5e9335b067e83c00c97f7 | |
| parent | b5c37c06811e605d37c5ba5b29210ded0fc76e1d (diff) | |
| download | chromium_src-b39881224053fa1000f77b2543af51afda17f0cf.zip chromium_src-b39881224053fa1000f77b2543af51afda17f0cf.tar.gz chromium_src-b39881224053fa1000f77b2543af51afda17f0cf.tar.bz2 | |
Remove two context menu entries that were never implemented!
Leave some stub code so someone can implement in the future.
BUG=11827
Review URL: http://codereview.chromium.org/113289
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@15904 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/browser/tab_contents/render_view_context_menu.cc | 23 |
1 files changed, 6 insertions, 17 deletions
diff --git a/chrome/browser/tab_contents/render_view_context_menu.cc b/chrome/browser/tab_contents/render_view_context_menu.cc index 0f01ea7..c99cafa 100644 --- a/chrome/browser/tab_contents/render_view_context_menu.cc +++ b/chrome/browser/tab_contents/render_view_context_menu.cc @@ -4,9 +4,6 @@ #include "chrome/browser/tab_contents/render_view_context_menu.h" -#if defined(OS_WIN) -#include "app/win_util.h" -#endif #include "app/l10n_util.h" #include "base/clipboard.h" #include "base/command_line.h" @@ -114,8 +111,10 @@ void RenderViewContextMenu::AppendFrameItems() { AppendMenuItem(IDS_CONTENT_CONTEXT_OPENFRAMENEWWINDOW); AppendMenuItem(IDS_CONTENT_CONTEXT_OPENFRAMEOFFTHERECORD); AppendSeparator(); - AppendMenuItem(IDS_CONTENT_CONTEXT_SAVEFRAMEAS); - AppendMenuItem(IDS_CONTENT_CONTEXT_PRINTFRAME); + // These two menu items have yet to be implemented. + // http://code.google.com/p/chromium/issues/detail?id=11827 + //AppendMenuItem(IDS_CONTENT_CONTEXT_SAVEFRAMEAS); + //AppendMenuItem(IDS_CONTENT_CONTEXT_PRINTFRAME); AppendMenuItem(IDS_CONTENT_CONTEXT_VIEWFRAMESOURCE); AppendMenuItem(IDS_CONTENT_CONTEXT_VIEWFRAMEINFO); } @@ -459,23 +458,13 @@ void RenderViewContextMenu::ExecuteItemCommand(int id) { break; case IDS_CONTENT_CONTEXT_SAVEFRAMEAS: -#if defined(OS_WIN) - win_util::MessageBox(NULL, L"Context Menu Action", L"Save Frame As", - MB_OK); -#else - // TODO(port): message box equivalent + // http://code.google.com/p/chromium/issues/detail?id=11827 NOTIMPLEMENTED() << "IDS_CONTENT_CONTEXT_SAVEFRAMEAS"; -#endif break; case IDS_CONTENT_CONTEXT_PRINTFRAME: -#if defined(OS_WIN) - win_util::MessageBox(NULL, L"Context Menu Action", L"Print Frame", - MB_OK); -#else - // TODO(port): message box equivalent + // http://code.google.com/p/chromium/issues/detail?id=11827 NOTIMPLEMENTED() << "IDS_CONTENT_CONTEXT_PRINTFRAME"; -#endif break; case IDS_CONTENT_CONTEXT_VIEWFRAMESOURCE: |