Use FilePaths in content::DropData to avoid redundant conversions.

ui/base/dragdrop/os_exchange_data_provider_win.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 "ui/base/dragdrop/os_exchange_data_provider_win.h"
 
 #include <algorithm>
 
 #include "base/basictypes.h"
 #include "base/files/file_path.h"
 #include "base/i18n/file_util_icu.h"
 #include "base/logging.h"
 #include "base/memory/scoped_handle.h"
 #include "base/pickle.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/win/scoped_hglobal.h"
 #include "grit/ui_strings.h"
 #include "net/base/net_util.h"
 #include "ui/base/clipboard/clipboard.h"
 #include "ui/base/clipboard/clipboard_util_win.h"
+#include "ui/base/dragdrop/file_info.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "url/gurl.h"
 
 namespace ui {
 
 static const OSExchangeData::CustomFormat& GetRendererTaintCustomType() {
   CR_DEFINE_STATIC_LOCAL(
       ui::OSExchangeData::CustomFormat,
       format,
       (ui::Clipboard::GetFormatType("chromium/x-renderer-taint")));
@@ skipping 314 matching lines @@
 
   storage = GetIDListStorageForFileName(path);
   if (!storage)
     return;
   info = new DataObjectImpl::StoredDataInfo(
       Clipboard::GetIDListFormatType().ToFormatEtc(), storage);
   data_->contents_.push_back(info);
 }
 
 void OSExchangeDataProviderWin::SetFilenames(
-    const std::vector<OSExchangeData::FileInfo>& filenames) {
+    const std::vector<FileInfo>& filenames) {
   for (size_t i = 0; i < filenames.size(); ++i) {
     STGMEDIUM* storage = GetStorageForFileName(filenames[i].path);
     DataObjectImpl::StoredDataInfo* info = new DataObjectImpl::StoredDataInfo(
         Clipboard::GetCFHDropFormatType().ToFormatEtc(), storage);
     data_->contents_.push_back(info);
   }
 }
 
 void OSExchangeDataProviderWin::SetPickledData(
     const OSExchangeData::CustomFormat& format,
@@ skipping 66 matching lines @@
 
 bool OSExchangeDataProviderWin::GetFilename(base::FilePath* path) const {
   std::vector<base::string16> filenames;
   bool success = ClipboardUtil::GetFilenames(source_object_, &filenames);
   if (success)
     *path = base::FilePath(filenames[0]);
   return success;
 }
 
 bool OSExchangeDataProviderWin::GetFilenames(
-    std::vector<OSExchangeData::FileInfo>* filenames) const {
+    std::vector<FileInfo>* filenames) const {
   std::vector<base::string16> filenames_local;
   bool success = ClipboardUtil::GetFilenames(source_object_, &filenames_local);
   if (success) {
     for (size_t i = 0; i < filenames_local.size(); ++i)
       filenames->push_back(
-          OSExchangeData::FileInfo(base::FilePath(filenames_local[i]),
-                                   base::FilePath()));
+          FileInfo(base::FilePath(filenames_local[i]), base::FilePath()));
   }
   return success;
 }
 
 bool OSExchangeDataProviderWin::GetPickledData(
     const OSExchangeData::CustomFormat& format,
     Pickle* data) const {
   DCHECK(data);
   bool success = false;
   STGMEDIUM medium;
@@ skipping 595 matching lines @@
 
 ///////////////////////////////////////////////////////////////////////////////
 // OSExchangeData, public:
 
 // static
 OSExchangeData::Provider* OSExchangeData::CreateProvider() {
   return new OSExchangeDataProviderWin();
 }
 
 }  // namespace ui