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

Unified Diff: ui/base/dragdrop/os_exchange_data_provider_mac.mm

Issue 542203004: MacViews: Get OSExchangeDataTest to pass (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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
Index: ui/base/dragdrop/os_exchange_data_provider_mac.mm
diff --git a/ui/base/dragdrop/os_exchange_data_provider_mac.mm b/ui/base/dragdrop/os_exchange_data_provider_mac.mm
index 096b8066209089cce5ff82ab57b3b8c6a7011bc3..ce205a8030bc6c9d5b35389c129fee17b3572e16 100644
--- a/ui/base/dragdrop/os_exchange_data_provider_mac.mm
+++ b/ui/base/dragdrop/os_exchange_data_provider_mac.mm
@@ -10,7 +10,6 @@
#include "base/pickle.h"
#include "base/strings/sys_string_conversions.h"
#import "third_party/mozilla/NSPasteboard+Utils.h"
-#import "ui/base/dragdrop/cocoa_dnd_util.h"
#include "url/gurl.h"
namespace ui {
@@ -45,12 +44,16 @@ void OSExchangeDataProviderMac::SetString(const base::string16& string) {
void OSExchangeDataProviderMac::SetURL(const GURL& url,
const base::string16& title) {
- [pasteboard_ setDataForURL:base::SysUTF8ToNSString(url.spec())
- title:base::SysUTF16ToNSString(title)];
+ NSURL* ns_url = [NSURL URLWithString:base::SysUTF8ToNSString(url.spec())];
+ [pasteboard_ writeObjects:@[ ns_url ]];
+
+ [pasteboard_ setString:base::SysUTF16ToNSString(title)
+ forType:kCorePasteboardFlavorType_urln];
}
void OSExchangeDataProviderMac::SetFilename(const base::FilePath& path) {
- NOTIMPLEMENTED();
+ [pasteboard_ setPropertyList:@[ base::SysUTF8ToNSString(path.value()) ]
+ forType:NSFilenamesPboardType];
}
void OSExchangeDataProviderMac::SetFilenames(
@@ -80,13 +83,28 @@ bool OSExchangeDataProviderMac::GetURLAndTitle(
OSExchangeData::FilenameToURLPolicy policy,
GURL* url,
base::string16* title) const {
- return PopulateURLAndTitleFromPasteboard(
- url, title, pasteboard_, policy == OSExchangeData::CONVERT_FILENAMES);
+ DCHECK(url);
+ DCHECK(title);
+
+ NSArray* items = [pasteboard_ readObjectsForClasses:@[ [NSURL class] ]
+ options:@{ }];
+ if ([items count] == 0)
+ return false;
+
+ NSString* url_string = [[items objectAtIndex:0] absoluteString];
+ *url = GURL([url_string UTF8String]);
+ *title = base::SysNSStringToUTF16(
+ [pasteboard_ stringForType:kCorePasteboardFlavorType_urln]);
+ return true;
}
bool OSExchangeDataProviderMac::GetFilename(base::FilePath* path) const {
- NOTIMPLEMENTED();
- return false;
+ NSArray* paths = [pasteboard_ propertyListForType:NSFilenamesPboardType];
+ if ([paths count] == 0)
+ return false;
+
+ *path = base::FilePath([[paths objectAtIndex:0] UTF8String]);
+ return true;
}
bool OSExchangeDataProviderMac::GetFilenames(
@@ -114,14 +132,11 @@ bool OSExchangeDataProviderMac::HasString() const {
bool OSExchangeDataProviderMac::HasURL(
OSExchangeData::FilenameToURLPolicy policy) const {
- GURL url;
- base::string16 title;
- return GetURLAndTitle(policy, &url, &title);
+ return [pasteboard_ canReadObjectForClasses:@[ [NSURL class] ] options:nil];
}
bool OSExchangeDataProviderMac::HasFile() const {
- NOTIMPLEMENTED();
- return false;
+ return [[pasteboard_ types] containsObject:NSFilenamesPboardType];
}
bool OSExchangeDataProviderMac::HasCustomFormat(
« no previous file with comments | « no previous file | ui/base/dragdrop/os_exchange_data_unittest.cc » ('j') | ui/base/dragdrop/os_exchange_data_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698