[Reland 1] Save pdf file instead of web page through context menu for embedded pdf file

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 5a7909e0c5c698a98914021b56cf0bca8845a32e..7fc60cb7b9f7cbe7eb565698b4af41248fbb5d75 100644
--- a/chrome/browser/renderer_context_menu/render_view_context_menu.cc
+++ b/chrome/browser/renderer_context_menu/render_view_context_menu.cc
@@ -1171,7 +1171,8 @@ void RenderViewContextMenu::AppendMediaItems() {
 
 void RenderViewContextMenu::AppendPluginItems() {
   if (params_.page_url == params_.src_url ||
-      guest_view::GuestViewBase::IsGuest(source_web_contents_)) {
+      (guest_view::GuestViewBase::IsGuest(source_web_contents_) &&
+       !embedder_web_contents_->IsSavable())) {

[lazyboy, 2017/05/11 18:15:17]

Can you add comments to explain this a bit? Something like:
// Full page pdf -> embedder is not savable, so we potentially show page items.
Otherwise we default to plugin items.
!embedder_web_contents_->IsSavable()

Also you can verify whether ^^^ is correct :)

[Wei Li, 2017/05/17 05:37:09]

Done.

     // Full page plugin, so show page menu items.
     if (params_.link_url.is_empty() && params_.selection_text.empty())
       AppendPageItems();