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

Side by Side Diff: ui/base/dragdrop/os_exchange_data_provider_aurax11_unittest.cc

Issue 135633002: Don't populate URL data in WebDropData when dragging files. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix Blink style enums Created 6 years, 11 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "ui/base/dragdrop/os_exchange_data_provider_aurax11.h" 5 #include "ui/base/dragdrop/os_exchange_data_provider_aurax11.h"
6 6
7 // Clean up X11 header polution 7 // Clean up X11 header polution
8 #undef None 8 #undef None
9 #undef Bool 9 #undef Bool
10 10
(...skipping 28 matching lines...) Expand all
39 base::MessageLoopForUI message_loop; 39 base::MessageLoopForUI message_loop;
40 ui::OSExchangeDataProviderAuraX11 provider; 40 ui::OSExchangeDataProviderAuraX11 provider;
41 }; 41 };
42 42
43 TEST_F(OSExchangeDataProviderAuraX11Test, MozillaURL) { 43 TEST_F(OSExchangeDataProviderAuraX11Test, MozillaURL) {
44 // Check that we can get titled entries. 44 // Check that we can get titled entries.
45 provider.SetURL(GURL(kGoogleURL), base::ASCIIToUTF16(kGoogleTitle)); 45 provider.SetURL(GURL(kGoogleURL), base::ASCIIToUTF16(kGoogleTitle));
46 { 46 {
47 GURL out_gurl; 47 GURL out_gurl;
48 base::string16 out_str; 48 base::string16 out_str;
49 EXPECT_TRUE(provider.GetURLAndTitle(&out_gurl, &out_str)); 49 EXPECT_TRUE(provider.GetURLAndTitle(
50 OSExchangeData::DO_NOT_CONVERT_FILENAMES, &out_gurl, &out_str));
50 EXPECT_EQ(base::ASCIIToUTF16(kGoogleTitle), out_str); 51 EXPECT_EQ(base::ASCIIToUTF16(kGoogleTitle), out_str);
51 EXPECT_EQ(kGoogleURL, out_gurl.spec()); 52 EXPECT_EQ(kGoogleURL, out_gurl.spec());
52 } 53 }
53 54
54 // Check that we can get non-titled entries. 55 // Check that we can get non-titled entries.
55 provider.SetURL(GURL(kGoogleURL), base::string16()); 56 provider.SetURL(GURL(kGoogleURL), base::string16());
56 { 57 {
57 GURL out_gurl; 58 GURL out_gurl;
58 base::string16 out_str; 59 base::string16 out_str;
59 EXPECT_TRUE(provider.GetURLAndTitle(&out_gurl, &out_str)); 60 EXPECT_TRUE(provider.GetURLAndTitle(
61 OSExchangeData::DO_NOT_CONVERT_FILENAMES, &out_gurl, &out_str));
60 EXPECT_EQ(base::string16(), out_str); 62 EXPECT_EQ(base::string16(), out_str);
61 EXPECT_EQ(kGoogleURL, out_gurl.spec()); 63 EXPECT_EQ(kGoogleURL, out_gurl.spec());
62 } 64 }
63 } 65 }
64 66
65 TEST_F(OSExchangeDataProviderAuraX11Test, FilesArentURLs) { 67 TEST_F(OSExchangeDataProviderAuraX11Test, FilesArentURLs) {
66 AddURLList(kFileURL); 68 AddURLList(kFileURL);
67 69
68 EXPECT_TRUE(provider.HasFile()); 70 EXPECT_TRUE(provider.HasFile());
69 EXPECT_FALSE(provider.HasURL()); 71 EXPECT_FALSE(provider.HasURL());
(...skipping 14 matching lines...) Expand all
84 86
85 // We should only receive the file from GetFilenames(). 87 // We should only receive the file from GetFilenames().
86 std::vector<OSExchangeData::FileInfo> filenames; 88 std::vector<OSExchangeData::FileInfo> filenames;
87 EXPECT_TRUE(provider.GetFilenames(&filenames)); 89 EXPECT_TRUE(provider.GetFilenames(&filenames));
88 ASSERT_EQ(1u, filenames.size()); 90 ASSERT_EQ(1u, filenames.size());
89 EXPECT_EQ(kFileName, filenames[0].path.value()); 91 EXPECT_EQ(kFileName, filenames[0].path.value());
90 92
91 // We should only receive the URL here. 93 // We should only receive the URL here.
92 GURL out_gurl; 94 GURL out_gurl;
93 base::string16 out_str; 95 base::string16 out_str;
94 EXPECT_TRUE(provider.GetURLAndTitle(&out_gurl, &out_str)); 96 EXPECT_TRUE(provider.GetURLAndTitle(
97 OSExchangeData::DO_NOT_CONVERT_FILENAMES, &out_gurl, &out_str));
95 EXPECT_EQ(base::string16(), out_str); 98 EXPECT_EQ(base::string16(), out_str);
96 EXPECT_EQ(kGoogleURL, out_gurl.spec()); 99 EXPECT_EQ(kGoogleURL, out_gurl.spec());
97 } 100 }
98 101
99 TEST_F(OSExchangeDataProviderAuraX11Test, OnlyStringURLIsUnfiltered) { 102 TEST_F(OSExchangeDataProviderAuraX11Test, OnlyStringURLIsUnfiltered) {
100 const base::string16 file_url = base::UTF8ToUTF16(kFileURL); 103 const base::string16 file_url = base::UTF8ToUTF16(kFileURL);
101 provider.SetString(file_url); 104 provider.SetString(file_url);
102 105
103 EXPECT_TRUE(provider.HasString()); 106 EXPECT_TRUE(provider.HasString());
104 EXPECT_FALSE(provider.HasURL()); 107 EXPECT_FALSE(provider.HasURL());
105 } 108 }
106 109
107 TEST_F(OSExchangeDataProviderAuraX11Test, StringAndURIListFilterString) { 110 TEST_F(OSExchangeDataProviderAuraX11Test, StringAndURIListFilterString) {
108 const base::string16 file_url = base::UTF8ToUTF16(kFileURL); 111 const base::string16 file_url = base::UTF8ToUTF16(kFileURL);
109 provider.SetString(file_url); 112 provider.SetString(file_url);
110 AddURLList(kFileURL); 113 AddURLList(kFileURL);
111 114
112 EXPECT_FALSE(provider.HasString()); 115 EXPECT_FALSE(provider.HasString());
113 base::string16 out_str; 116 base::string16 out_str;
114 EXPECT_FALSE(provider.GetString(&out_str)); 117 EXPECT_FALSE(provider.GetString(&out_str));
115 118
116 EXPECT_TRUE(provider.HasFile()); 119 EXPECT_TRUE(provider.HasFile());
117 } 120 }
118 121
119 } // namespace ui 122 } // namespace ui
OLDNEW
« no previous file with comments | « ui/base/dragdrop/os_exchange_data_provider_aurax11.cc ('k') | ui/base/dragdrop/os_exchange_data_provider_win.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698