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

Unified Diff: ui/base/clipboard/clipboard_unittest.cc

Issue 558913003: Remove clipboard argument from ScopedClipboardWriter constructor. (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/clipboard/clipboard_unittest.cc
diff --git a/ui/base/clipboard/clipboard_unittest.cc b/ui/base/clipboard/clipboard_unittest.cc
index 72c07cf2c8ac8ba999ec40da529c246d91c52b80..47981210e9a9d9c4e6ba70f30403d9ef4d44a6a3 100644
--- a/ui/base/clipboard/clipboard_unittest.cc
+++ b/ui/base/clipboard/clipboard_unittest.cc
@@ -81,8 +81,7 @@ bool MarkupMatches(const base::string16& expected_markup,
TEST_F(ClipboardTest, ClearTest) {
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WriteText(ASCIIToUTF16("clear me"));
}
@@ -99,8 +98,7 @@ TEST_F(ClipboardTest, TextTest) {
std::string ascii_text;
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WriteText(text);
}
@@ -121,8 +119,7 @@ TEST_F(ClipboardTest, HTMLTest) {
std::string url("http://www.example.com/"), url_result;
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WriteText(plain);
clipboard_writer.WriteHTML(markup, url);
}
@@ -147,8 +144,7 @@ TEST_F(ClipboardTest, RTFTest) {
"}";
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WriteRTF(rtf);
}
@@ -168,14 +164,12 @@ TEST_F(ClipboardTest, MultipleBufferTest) {
std::string url("http://www.example.com/"), url_result;
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WriteText(text);
}
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_SELECTION);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_SELECTION);
clipboard_writer.WriteHTML(markup, url);
}
@@ -211,8 +205,7 @@ TEST_F(ClipboardTest, TrickyHTMLTest) {
base::string16 plain(ASCIIToUTF16("Bye!")), plain_result;
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WriteText(plain);
clipboard_writer.WriteHTML(markup, url);
}
@@ -237,8 +230,7 @@ TEST_F(ClipboardTest, UniodeHTMLTest) {
std::string url, url_result;
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WriteHTML(markup, url);
}
@@ -263,8 +255,7 @@ TEST_F(ClipboardTest, BookmarkTest) {
std::string url("http://www.example.com/"), url_result;
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WriteBookmark(title, url);
}
@@ -283,8 +274,7 @@ TEST_F(ClipboardTest, MultiFormatTest) {
std::string ascii_text;
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WriteHTML(markup, url);
clipboard_writer.WriteText(text);
}
@@ -314,8 +304,7 @@ TEST_F(ClipboardTest, URLTest) {
base::string16 url(ASCIIToUTF16("http://www.google.com/"));
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WriteURL(url);
}
@@ -542,8 +531,7 @@ TEST_F(ClipboardTest, DataTest) {
write_pickle.WriteString(payload);
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WritePickledData(write_pickle, kFormat);
}
@@ -574,8 +562,7 @@ TEST_F(ClipboardTest, MultipleDataTest) {
write_pickle2.WriteString(payload2);
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WritePickledData(write_pickle1, kFormat1);
// overwrite the previous pickle for fun
clipboard_writer.WritePickledData(write_pickle2, kFormat2);
@@ -596,8 +583,7 @@ TEST_F(ClipboardTest, MultipleDataTest) {
EXPECT_EQ(payload2, unpickled_string2);
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WritePickledData(write_pickle2, kFormat2);
// overwrite the previous pickle for fun
clipboard_writer.WritePickledData(write_pickle1, kFormat1);
@@ -629,8 +615,7 @@ TEST_F(ClipboardTest, HyperlinkTest) {
std::string url_result;
base::string16 html_result;
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WriteHyperlink(ASCIIToUTF16(kTitle), kUrl);
}
@@ -646,8 +631,7 @@ TEST_F(ClipboardTest, HyperlinkTest) {
#if defined(OS_WIN) // Windows only tests.
TEST_F(ClipboardTest, WebSmartPasteTest) {
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WriteWebSmartPaste();
}
@@ -701,7 +685,7 @@ TEST_F(ClipboardTest, HtmlTest) {
// Test writing all formats we have simultaneously.
TEST_F(ClipboardTest, WriteEverything) {
{
- ScopedClipboardWriter writer(&clipboard(), CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter writer(CLIPBOARD_TYPE_COPY_PASTE);
writer.WriteText(UTF8ToUTF16("foo"));
writer.WriteURL(UTF8ToUTF16("foo"));
writer.WriteHTML(UTF8ToUTF16("foo"), "bar");
@@ -727,7 +711,7 @@ TEST_F(ClipboardTest, GetSequenceNumber) {
clipboard().GetSequenceNumber(CLIPBOARD_TYPE_COPY_PASTE);
{
- ScopedClipboardWriter writer(&clipboard(), CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter writer(CLIPBOARD_TYPE_COPY_PASTE);
writer.WriteText(UTF8ToUTF16("World"));
}
@@ -749,8 +733,7 @@ TEST_F(ClipboardTest, GetSequenceNumber) {
TEST_F(ClipboardTest, InternalClipboardInvalidation) {
// Write a Webkit smart paste tag to our clipboard.
{
- ScopedClipboardWriter clipboard_writer(&clipboard(),
- CLIPBOARD_TYPE_COPY_PASTE);
+ ScopedClipboardWriter clipboard_writer(CLIPBOARD_TYPE_COPY_PASTE);
clipboard_writer.WriteWebSmartPaste();
}
EXPECT_TRUE(clipboard().IsFormatAvailable(

Powered by Google App Engine
This is Rietveld 408576698