Add option to open PDFs in system viewer to OS X and Linux.

chrome/browser/download/download_shelf_context_menu.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/download/download_shelf_context_menu.h"
 
 #include "base/command_line.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/download/download_crx_util.h"
 #include "chrome/browser/download/download_item_model.h"
@@ skipping 32 matching lines @@
 
 DownloadShelfContextMenu::DownloadShelfContextMenu(
     DownloadItem* download_item,
     content::PageNavigator* navigator)
     : download_item_(download_item),
       navigator_(navigator) {
   DCHECK(download_item_);
   download_item_->AddObserver(this);
 
 #if defined(OS_WIN)
-  is_pdf_reader_up_to_date_ = false;
+  is_adobe_pdf_reader_up_to_date_ = false;
   if (IsDownloadPdf() && IsAdobeReaderDefaultPDFViewer()) {
-    is_pdf_reader_up_to_date_ =
+    is_adobe_pdf_reader_up_to_date_ =
         DownloadTargetDeterminer::IsAdobeReaderUpToDate();
   }
 #endif  // defined(OS_WIN)
 }
 
 ui::SimpleMenuModel* DownloadShelfContextMenu::GetMenuModel() {
   ui::SimpleMenuModel* model = NULL;
 
   if (!download_item_)
     return NULL;
@@ skipping 49 matching lines @@
 
 bool DownloadShelfContextMenu::IsCommandIdChecked(int command_id) const {
   if (!download_item_)
     return false;
 
   switch (command_id) {
     case OPEN_WHEN_COMPLETE:
       return download_item_->GetOpenWhenComplete() ||
           download_crx_util::IsExtensionDownload(*download_item_);
     case ALWAYS_OPEN_TYPE:
-#if defined(OS_WIN)
-      if (CanOpenPdfInReader()) {
+#if defined(OS_WIN) || defined(OS_LINUX) || \
+    (defined(OS_MACOSX) && !defined(OS_IOS))
+      if (CanOpenPdfInSystemViewer()) {
         DownloadPrefs* prefs = DownloadPrefs::FromBrowserContext(
             download_item_->GetBrowserContext());
-        return prefs->ShouldOpenPdfInAdobeReader();
+        return prefs->ShouldOpenPdfInSystemReader();
       }
 #endif
       return download_item_->ShouldOpenFileBasedOnExtension();
     case TOGGLE_PAUSE:
       return download_item_->IsPaused();
   }
   return false;
 }
 
 bool DownloadShelfContextMenu::IsCommandIdVisible(int command_id) const {
@@ skipping 14 matching lines @@
     case SHOW_IN_FOLDER:
       download_item_->ShowDownloadInShell();
       break;
     case OPEN_WHEN_COMPLETE:
       download_item_->OpenDownload();
       break;
     case ALWAYS_OPEN_TYPE: {
       bool is_checked = IsCommandIdChecked(ALWAYS_OPEN_TYPE);
       DownloadPrefs* prefs = DownloadPrefs::FromBrowserContext(
           download_item_->GetBrowserContext());
-#if defined(OS_WIN)
-      if (CanOpenPdfInReader()) {
-        prefs->SetShouldOpenPdfInAdobeReader(!is_checked);
+#if defined(OS_WIN) || defined(OS_LINUX) || \
+    (defined(OS_MACOSX) && !defined(OS_IOS))
+      if (CanOpenPdfInSystemViewer()) {
+        prefs->SetShouldOpenPdfInSystemReader(!is_checked);
         DownloadItemModel(download_item_).SetShouldPreferOpeningInBrowser(
             is_checked);
         break;
       }
 #endif
       base::FilePath path = download_item_->GetTargetFilePath();
       if (is_checked)
         prefs->DisableAutoOpenBasedOnExtension(path);
       else
         prefs->EnableAutoOpenBasedOnExtension(path);
@@ skipping 202 matching lines @@
 
   DownloadItemModel download_model(download_item_);
   malicious_download_menu_model_->AddItemWithStringId(
       LEARN_MORE_SCANNING, IDS_DOWNLOAD_MENU_LEARN_MORE_SCANNING);
 
   return malicious_download_menu_model_.get();
 }
 
 int DownloadShelfContextMenu::GetAlwaysOpenStringId() const {
 #if defined(OS_WIN)
-  if (CanOpenPdfInReader())
+  if (CanOpenPdfInSystemViewer())
     return IDS_DOWNLOAD_MENU_ALWAYS_OPEN_PDF_IN_READER;
+#elif defined(OS_MACOSX) || defined(OS_LINUX)
+  if (CanOpenPdfInSystemViewer())
+    return IDS_DOWNLOAD_MENU_PLATFORM_OPEN_ALWAYS;
 #endif
   return IDS_DOWNLOAD_MENU_ALWAYS_OPEN_TYPE;
 }
 
-#if defined(OS_WIN)
+#if defined(OS_WIN) || defined(OS_MACOSX) || defined(OS_LINUX)
 bool DownloadShelfContextMenu::IsDownloadPdf() const {
   base::FilePath path = download_item_->GetTargetFilePath();
   return path.MatchesExtension(FILE_PATH_LITERAL(".pdf"));
 }
+#endif
 
-bool DownloadShelfContextMenu::CanOpenPdfInReader() const {
-  return (is_pdf_reader_up_to_date_ && IsDownloadPdf());
+bool DownloadShelfContextMenu::CanOpenPdfInSystemViewer() const {
+#if defined(OS_WIN)
+  return IsDownloadPdf() &&
+         (IsAdobeReaderDefaultPDFViewer() ? is_adobe_pdf_reader_up_to_date_ :
+                                            true);
+#elif defined(OS_MACOSX) || defined(OS_LINUX)
+  return IsDownloadPdf();
+#endif
 }
-#endif