Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(378)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/browser/web_contents/web_drag_dest_mac.mm

Issue 211383007: Use FilePaths in content::DropData to avoid redundant conversions. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: clang-format Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/browser/web_contents/web_drag_dest_gtk.cc ('k') | content/common/drag_traits.h » ('j') | ui/base/dragdrop/file_info.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/web_contents/web_drag_dest_mac.mm
diff --git a/content/browser/web_contents/web_drag_dest_mac.mm b/content/browser/web_contents/web_drag_dest_mac.mm
index 20a26f34091415e85b7f52f47534f18885417f5d..b0cb8e2d376c94ebe906b2b683ca9254e6401bd2 100644
--- a/content/browser/web_contents/web_drag_dest_mac.mm
+++ b/content/browser/web_contents/web_drag_dest_mac.mm
@@ -291,9 +291,9 @@ int GetModifierFlags() {
BOOL exists = [[NSFileManager defaultManager]
fileExistsAtPath:filename];
if (exists) {
- data->filenames.push_back(
- DropData::FileInfo(
- base::SysNSStringToUTF16(filename), base::string16()));
+ data->filenames.push_back(ui::FileInfo(
+ base::FilePath::FromUTF8Unsafe(base::SysNSStringToUTF8(filename)),
+ base::FilePath()));
}
}
}
« no previous file with comments | « content/browser/web_contents/web_drag_dest_gtk.cc ('k') | content/common/drag_traits.h » ('j') | ui/base/dragdrop/file_info.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698