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

Unified Diff: content/renderer/webclipboard_impl.cc

Issue 17420005: Move the following clipboard files from webkit\glue to webkit\renderer. These files now live in the… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 7 years, 6 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/renderer/webclipboard_impl.h ('k') | webkit/glue/clipboard_client.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/webclipboard_impl.cc
===================================================================
--- content/renderer/webclipboard_impl.cc (revision 207064)
+++ content/renderer/webclipboard_impl.cc (working copy)
@@ -1,15 +1,15 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "webkit/glue/webclipboard_impl.h"
+#include "content/renderer/webclipboard_impl.h"
#include "base/logging.h"
#include "base/pickle.h"
#include "base/strings/string_util.h"
#include "base/strings/utf_string_conversions.h"
+#include "content/renderer/scoped_clipboard_writer_glue.h"
#include "googleurl/src/gurl.h"
-#include "net/base/escape.h"
#include "third_party/WebKit/public/platform/WebData.h"
#include "third_party/WebKit/public/platform/WebDragData.h"
#include "third_party/WebKit/public/platform/WebImage.h"
@@ -21,8 +21,8 @@
#include "ui/base/clipboard/clipboard.h"
#include "ui/base/clipboard/custom_data_helper.h"
#include "webkit/common/webdropdata.h"
-#include "webkit/glue/scoped_clipboard_writer_glue.h"
#include "webkit/glue/webkit_glue.h"
+#include "webkit/renderer/clipboard_utils.h"
using WebKit::WebClipboard;
using WebKit::WebData;
@@ -32,35 +32,8 @@
using WebKit::WebURL;
using WebKit::WebVector;
-namespace webkit_glue {
+namespace content {
-// Static
-std::string WebClipboardImpl::URLToMarkup(const WebURL& url,
- const WebString& title) {
- std::string markup("<a href=\"");
- markup.append(url.spec());
- markup.append("\">");
- // TODO(darin): HTML escape this
- markup.append(net::EscapeForHTML(UTF16ToUTF8(title)));
- markup.append("</a>");
- return markup;
-}
-
-// Static
-std::string WebClipboardImpl::URLToImageMarkup(const WebURL& url,
- const WebString& title) {
- std::string markup("<img src=\"");
- markup.append(url.spec());
- markup.append("\"");
- if (!title.isEmpty()) {
- markup.append(" alt=\"");
- markup.append(net::EscapeForHTML(UTF16ToUTF8(title)));
- markup.append("\"");
- }
- markup.append("/>");
- return markup;
-}
-
WebClipboardImpl::WebClipboardImpl(ClipboardClient* client)
: client_(client) {
}
@@ -201,7 +174,8 @@
ScopedClipboardWriterGlue scw(client_);
scw.WriteBookmark(title, url.spec());
- scw.WriteHTML(UTF8ToUTF16(URLToMarkup(url, title)), std::string());
+ scw.WriteHTML(UTF8ToUTF16(webkit_clipboard::URLToMarkup(url, title)),
+ std::string());
scw.WriteText(UTF8ToUTF16(std::string(url.spec())));
}
@@ -225,7 +199,8 @@
// We also don't want to write HTML on a Mac, since Mail.app prefers to use
// the image markup over attaching the actual image. See
// http://crbug.com/33016 for details.
- scw.WriteHTML(UTF8ToUTF16(URLToImageMarkup(url, title)), std::string());
+ scw.WriteHTML(UTF8ToUTF16(webkit_clipboard::URLToImageMarkup(url, title)),
+ std::string());
#endif
}
}
@@ -275,4 +250,5 @@
return true;
}
-} // namespace webkit_glue
+} // namespace content
+
« no previous file with comments | « content/renderer/webclipboard_impl.h ('k') | webkit/glue/clipboard_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698