Print Preview: Disable right click -> "Save As" for preview PDF. Users should just print to PDF i...

chrome/browser/tab_contents/render_view_context_menu.cc

Index: chrome/browser/tab_contents/render_view_context_menu.cc
===================================================================
--- chrome/browser/tab_contents/render_view_context_menu.cc	(revision 101261)
+++ chrome/browser/tab_contents/render_view_context_menu.cc	(working copy)
@@ -1190,10 +1190,12 @@
       if (!local_state->GetBoolean(prefs::kAllowFileSelectionDialogs))
         return false;
 
-      return (params_.media_flags &
-              WebContextMenuData::MediaCanSave) &&
-             params_.src_url.is_valid() &&
-             ProfileIOData::IsHandledProtocol(params_.src_url.scheme());
+      const GURL& url = params_.src_url;
+      return (params_.media_flags & WebContextMenuData::MediaCanSave) &&
+          url.is_valid() && ProfileIOData::IsHandledProtocol(url.scheme()) &&
+          // Do not save the preview PDF on the print preview page.
+          !(url.SchemeIs(chrome::kChromeUIScheme) &&

[brettw, 2011/09/16 05:19:14]

This code is getting a bit crazy. Can you make a local helper function
IsPrintPreviewURL(const GURL&) that you call here instead?

[Lei Zhang, 2011/09/16 20:00:50]

Done.

+            url.host() == chrome::kChromeUIPrintHost);
     }
 
     case IDC_CONTENT_CONTEXT_OPENAVNEWTAB: