Be explicit about target type in platform_util::OpenItem()

chrome/browser/download/chrome_download_manager_delegate.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/chrome_download_manager_delegate.h"
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/bind.h"
@@ skipping 83 matching lines @@
 enum PlatformDownloadPathType {
   // Return the platform specific target path.
   PLATFORM_TARGET_PATH,
 
   // Return the platform specific current path. If the download is in-progress
   // and the download location is a local filesystem path, then
   // GetPlatformDownloadPath will return the path to the intermediate file.
   PLATFORM_CURRENT_PATH
 };
 
-// Returns a path in the form that that is expected by platform_util::OpenItem /
-// platform_util::ShowItemInFolder / DownloadTargetDeterminer.
+// Returns a path in the form that that is expected by platform_util::OpenFile /
+// platform_util::OpenFolder / platform_util::ShowItemInFolder /
+// DownloadTargetDeterminer.
 //
 // DownloadItems corresponding to Drive downloads use a temporary file as the
 // target path. The paths returned by DownloadItem::GetFullPath() /
 // GetTargetFilePath() refer to this temporary file. This function looks up the
 // corresponding path in Drive for these downloads.
 //
 // How the platform path is determined is based on PlatformDownloadPathType.
 base::FilePath GetPlatformDownloadPath(Profile* profile,
                                        const DownloadItem* download,
                                        PlatformDownloadPathType path_type) {
@@ skipping 305 matching lines @@
       can_save_as_complete,
       download_prefs_.get(),
       callback);
 }
 
 void ChromeDownloadManagerDelegate::OpenDownloadUsingPlatformHandler(
     DownloadItem* download) {
   base::FilePath platform_path(
       GetPlatformDownloadPath(profile_, download, PLATFORM_TARGET_PATH));
   DCHECK(!platform_path.empty());
-  platform_util::OpenItem(profile_, platform_path);
+  platform_util::OpenFile(profile_, platform_path,
+                          platform_util::OpenOperationCallback());
 }
 
 void ChromeDownloadManagerDelegate::OpenDownload(DownloadItem* download) {
   DCHECK_EQ(DownloadItem::COMPLETE, download->GetState());
   DCHECK(!download->GetTargetFilePath().empty());
   if (!download->CanOpenDownload())
     return;
 
   if (!DownloadItemModel(download).ShouldPreferOpeningInBrowser()) {
     RecordDownloadOpenMethod(DOWNLOAD_OPEN_METHOD_DEFAULT_PLATFORM);
@@ skipping 298 matching lines @@
       path.MatchesExtension(FILE_PATH_LITERAL(".xht")) ||
       path.MatchesExtension(FILE_PATH_LITERAL(".xhtm")) ||
       path.MatchesExtension(FILE_PATH_LITERAL(".xhtml")) ||
       path.MatchesExtension(FILE_PATH_LITERAL(".xsl")) ||
       path.MatchesExtension(FILE_PATH_LITERAL(".xslt"))) {
     return true;
   }
 #endif
   return false;
 }