Chromium Code Reviews| Index: chrome/browser/renderer_context_menu/render_view_context_menu.cc |
| diff --git a/chrome/browser/renderer_context_menu/render_view_context_menu.cc b/chrome/browser/renderer_context_menu/render_view_context_menu.cc |
| index b0aa4fec544f8762a7ad9d77c1eae872f6f2fbb8..aa40aca9670afa8b001478b7497ad788a5776ad5 100644 |
| --- a/chrome/browser/renderer_context_menu/render_view_context_menu.cc |
| +++ b/chrome/browser/renderer_context_menu/render_view_context_menu.cc |
| @@ -825,6 +825,11 @@ void RenderViewContextMenu::AppendCopyItem() { |
| } |
| void RenderViewContextMenu::AppendPrintItem() { |
| + // AppendPluginItems() could already added |IDC_PRINT| so we are skipping |
| + // here if |IDC_PRINT| is already present in context menu. |
|
raymes
2014/10/07 17:09:14
nit:
// AppendPluginItems() could have already ad
|
| + if (menu_model_.GetIndexOfCommandId(IDC_PRINT) != -1) |
| + return; |
| + |
| if (GetPrefs(browser_context_)->GetBoolean(prefs::kPrintingEnabled) && |
| (params_.media_type == WebContextMenuData::MediaTypeNone || |
| params_.media_flags & WebContextMenuData::MediaCanPrint)) { |