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

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

Issue 776903002: Cleanup: Remove some unneeded string allocations. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix win Created 6 years 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_aurax11.cc
diff --git a/ui/base/clipboard/clipboard_aurax11.cc b/ui/base/clipboard/clipboard_aurax11.cc
index d7e2ad0bd66c6658b19544657f731efe0d93c634..617e38021bb950c1002f1163cb321b795dded432 100644
--- a/ui/base/clipboard/clipboard_aurax11.cc
+++ b/ui/base/clipboard/clipboard_aurax11.cc
@@ -890,7 +890,8 @@ void ClipboardAuraX11::WriteBookmark(const char* title_data,
size_t url_len) {
// Write as a mozilla url (UTF16: URL, newline, title).
base::string16 url = base::UTF8ToUTF16(std::string(url_data, url_len) + "\n");
- base::string16 title = base::UTF8ToUTF16(std::string(title_data, title_len));
+ base::string16 title =
+ base::UTF8ToUTF16(base::StringPiece(title_data, title_len));
std::vector<unsigned char> data;
ui::AddString16ToVector(url, &data);
« components/user_manager/empty_user_info.cc ('K') | « net/base/mime_util_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698