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

Unified Diff: base/clipboard_unittest.cc

Issue 8038: Simultaneously store multiple formats on the linux clipboard. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 12 years, 2 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 | « base/clipboard_linux.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/clipboard_unittest.cc
===================================================================
--- base/clipboard_unittest.cc (revision 3794)
+++ base/clipboard_unittest.cc (working copy)
@@ -42,6 +42,21 @@
EXPECT_EQ(WideToUTF8(text), ascii_text);
}
+TEST_F(ClipboardTest, OverwriteTest) {
+ Clipboard clipboard;
+
+ std::wstring text1(L"first string"), text2(L"second string"), text_result;
+
+ clipboard.Clear();
+ clipboard.WriteText(text1);
+ clipboard.WriteText(text2);
+
+ EXPECT_TRUE(clipboard.IsFormatAvailable(
+ Clipboard::GetPlainTextWFormatType()));
+ clipboard.ReadText(&text_result);
+ EXPECT_EQ(text2, text_result);
+}
+
TEST_F(ClipboardTest, HTMLTest) {
Clipboard clipboard;
@@ -61,8 +76,6 @@
#endif
}
-// TODO(estade): Port the following tests.
-#if !defined(OS_LINUX)
TEST_F(ClipboardTest, TrickyHTMLTest) {
Clipboard clipboard;
@@ -75,15 +88,15 @@
Clipboard::GetHtmlFormatType()));
clipboard.ReadHTML(&markup_result, &url_result);
EXPECT_EQ(markup, markup_result);
-
-#if defined(OS_MACOSX)
- // TODO(playmobil): It's not clear that the OS X clipboard needs to support
+#if defined(OS_WIN)
+ // TODO(playmobil): It's not clear that non windows clipboards need to support
// this.
-#else
EXPECT_EQ(url, url_result);
#endif
}
+// TODO(estade): Port the following test (decide what target we use for urls)
+#if !defined(OS_LINUX)
TEST_F(ClipboardTest, BookmarkTest) {
Clipboard clipboard;
@@ -98,6 +111,7 @@
EXPECT_EQ(title, title_result);
EXPECT_EQ(url, url_result);
}
+#endif
TEST_F(ClipboardTest, MultiFormatTest) {
Clipboard clipboard;
@@ -118,10 +132,9 @@
Clipboard::GetPlainTextFormatType()));
clipboard.ReadHTML(&markup_result, &url_result);
EXPECT_EQ(markup, markup_result);
-#if defined(OS_MACOSX)
- // TODO(playmobil): It's not clear that the OS X clipboard needs to support
+#if defined(OS_WIN)
+ // TODO(playmobil): It's not clear that non windows clipboards need to support
// this.
-#else
EXPECT_EQ(url, url_result);
#endif
clipboard.ReadText(&text_result);
@@ -130,6 +143,8 @@
EXPECT_EQ(WideToUTF8(text), ascii_text);
}
+// TODO(estade): Port the following tests (decide what targets we use for files)
+#if !defined(OS_LINUX)
// Files for this test don't actually need to exist on the file system, just
// don't try to use a non-existent file you've retrieved from the clipboard.
TEST_F(ClipboardTest, FileTest) {
@@ -176,6 +191,7 @@
for (size_t i = 0; i < out_files.size(); ++i)
EXPECT_EQ(files[i], out_files[i]);
}
+#endif // !defined(OS_LINUX)
#if defined(OS_WIN) // Windows only tests.
TEST_F(ClipboardTest, HyperlinkTest) {
@@ -224,4 +240,3 @@
Clipboard::GetBitmapFormatType()));
}
#endif
-#endif // !defined(OS_LINUX)
« no previous file with comments | « base/clipboard_linux.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698