In pdf context menu 'Print..' is coming 2 times.

chrome/browser/renderer_context_menu/render_view_context_menu.cc

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..71cdfd7834698f7ef7f8e5e5b3a10e454b8ffa8b 100644
--- a/chrome/browser/renderer_context_menu/render_view_context_menu.cc
+++ b/chrome/browser/renderer_context_menu/render_view_context_menu.cc
@@ -827,7 +827,8 @@ void RenderViewContextMenu::AppendCopyItem() {
 void RenderViewContextMenu::AppendPrintItem() {
   if (GetPrefs(browser_context_)->GetBoolean(prefs::kPrintingEnabled) &&
       (params_.media_type == WebContextMenuData::MediaTypeNone ||
-       params_.media_flags & WebContextMenuData::MediaCanPrint)) {
+       params_.media_flags & WebContextMenuData::MediaCanPrint) &&
+      menu_model_.GetIndexOfCommandId(IDC_PRINT) == -1) {

[lazyboy, 2014/10/07 15:32:24]

It's unfortunate that we have 3 places that decides whether to add print items
:(
Hopefully we can clean this up later.
Can you add a note here saying AppendPluginItems() could already add IDC_PRINT
so we are skipping here.

[Deepak, 2014/10/07 15:57:11]

Done.

     menu_model_.AddItemWithStringId(IDC_PRINT, IDS_CONTENT_CONTEXT_PRINT);
   }
 }