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

Unified Diff: app/clipboard/clipboard_win.cc

Issue 3056029: Move the number conversions from string_util to a new file.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 5 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 | « no previous file | app/gfx/font_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: app/clipboard/clipboard_win.cc
===================================================================
--- app/clipboard/clipboard_win.cc (revision 54340)
+++ app/clipboard/clipboard_win.cc (working copy)
@@ -16,7 +16,8 @@
#include "base/logging.h"
#include "base/message_loop.h"
#include "base/shared_memory.h"
-#include "base/string_util.h"
+#include "base/string_util.h" // Remove when ASCIIToWide is in utf_string_...
+#include "base/string_number_conversions.h"
#include "base/utf_string_conversions.h"
#include "gfx/size.h"
@@ -188,7 +189,7 @@
std::string html_fragment = ClipboardUtil::HtmlToCFHtml(markup, url);
HGLOBAL glob = CreateGlobalData(html_fragment);
- WriteToClipboard(StringToInt(GetHtmlFormatType()), glob);
+ WriteToClipboard(ClipboardUtil::GetHtmlFormat()->cfFormat, glob);
}
void Clipboard::WriteBookmark(const char* title_data,
@@ -202,12 +203,13 @@
string16 wide_bookmark = UTF8ToWide(bookmark);
HGLOBAL glob = CreateGlobalData(wide_bookmark);
- WriteToClipboard(StringToInt(GetUrlWFormatType()), glob);
+ WriteToClipboard(ClipboardUtil::GetUrlWFormat()->cfFormat, glob);
}
void Clipboard::WriteWebSmartPaste() {
DCHECK(clipboard_owner_);
- ::SetClipboardData(StringToInt(GetWebKitSmartPasteFormatType()), NULL);
+ ::SetClipboardData(ClipboardUtil::GetWebKitSmartPasteFormat()->cfFormat,
+ NULL);
}
void Clipboard::WriteBitmap(const char* pixel_data, const char* size_data) {
@@ -314,7 +316,10 @@
bool Clipboard::IsFormatAvailable(const Clipboard::FormatType& format,
Clipboard::Buffer buffer) const {
DCHECK_EQ(buffer, BUFFER_STANDARD);
- return ::IsClipboardFormatAvailable(StringToInt(format)) != FALSE;
+ int f;
+ if (!base::StringToInt(format, &f))
+ return false;
+ return ::IsClipboardFormatAvailable(f) != FALSE;
}
bool Clipboard::IsFormatAvailableByString(
@@ -384,7 +389,7 @@
if (!clipboard.Acquire(GetClipboardWindow()))
return;
- HANDLE data = ::GetClipboardData(StringToInt(GetHtmlFormatType()));
+ HANDLE data = ::GetClipboardData(ClipboardUtil::GetHtmlFormat()->cfFormat);
if (!data)
return;
@@ -410,7 +415,7 @@
if (!clipboard.Acquire(GetClipboardWindow()))
return;
- HANDLE data = ::GetClipboardData(StringToInt(GetUrlWFormatType()));
+ HANDLE data = ::GetClipboardData(ClipboardUtil::GetUrlWFormat()->cfFormat);
if (!data)
return;
@@ -507,74 +512,75 @@
// static
Clipboard::FormatType Clipboard::GetUrlFormatType() {
- return IntToString(ClipboardUtil::GetUrlFormat()->cfFormat);
+ return base::IntToString(ClipboardUtil::GetUrlFormat()->cfFormat);
}
// static
Clipboard::FormatType Clipboard::GetUrlWFormatType() {
- return IntToString(ClipboardUtil::GetUrlWFormat()->cfFormat);
+ return base::IntToString(ClipboardUtil::GetUrlWFormat()->cfFormat);
}
// static
Clipboard::FormatType Clipboard::GetMozUrlFormatType() {
- return IntToString(ClipboardUtil::GetMozUrlFormat()->cfFormat);
+ return base::IntToString(ClipboardUtil::GetMozUrlFormat()->cfFormat);
}
// static
Clipboard::FormatType Clipboard::GetPlainTextFormatType() {
- return IntToString(ClipboardUtil::GetPlainTextFormat()->cfFormat);
+ return base::IntToString(ClipboardUtil::GetPlainTextFormat()->cfFormat);
}
// static
Clipboard::FormatType Clipboard::GetPlainTextWFormatType() {
- return IntToString(ClipboardUtil::GetPlainTextWFormat()->cfFormat);
+ return base::IntToString(ClipboardUtil::GetPlainTextWFormat()->cfFormat);
}
// static
Clipboard::FormatType Clipboard::GetFilenameFormatType() {
- return IntToString(ClipboardUtil::GetFilenameFormat()->cfFormat);
+ return base::IntToString(ClipboardUtil::GetFilenameFormat()->cfFormat);
}
// static
Clipboard::FormatType Clipboard::GetFilenameWFormatType() {
- return IntToString(ClipboardUtil::GetFilenameWFormat()->cfFormat);
+ return base::IntToString(ClipboardUtil::GetFilenameWFormat()->cfFormat);
}
// MS HTML Format
// static
Clipboard::FormatType Clipboard::GetHtmlFormatType() {
- return IntToString(ClipboardUtil::GetHtmlFormat()->cfFormat);
+ return base::IntToString(ClipboardUtil::GetHtmlFormat()->cfFormat);
}
// static
Clipboard::FormatType Clipboard::GetBitmapFormatType() {
- return IntToString(CF_BITMAP);
+ return base::IntToString(CF_BITMAP);
}
// Firefox text/html
// static
Clipboard::FormatType Clipboard::GetTextHtmlFormatType() {
- return IntToString(ClipboardUtil::GetTextHtmlFormat()->cfFormat);
+ return base::IntToString(ClipboardUtil::GetTextHtmlFormat()->cfFormat);
}
// static
Clipboard::FormatType Clipboard::GetCFHDropFormatType() {
- return IntToString(ClipboardUtil::GetCFHDropFormat()->cfFormat);
+ return base::IntToString(ClipboardUtil::GetCFHDropFormat()->cfFormat);
}
// static
Clipboard::FormatType Clipboard::GetFileDescriptorFormatType() {
- return IntToString(ClipboardUtil::GetFileDescriptorFormat()->cfFormat);
+ return base::IntToString(ClipboardUtil::GetFileDescriptorFormat()->cfFormat);
}
// static
Clipboard::FormatType Clipboard::GetFileContentFormatZeroType() {
- return IntToString(ClipboardUtil::GetFileContentFormatZero()->cfFormat);
+ return base::IntToString(ClipboardUtil::GetFileContentFormatZero()->cfFormat);
}
// static
Clipboard::FormatType Clipboard::GetWebKitSmartPasteFormatType() {
- return IntToString(ClipboardUtil::GetWebKitSmartPasteFormat()->cfFormat);
+ return base::IntToString(
+ ClipboardUtil::GetWebKitSmartPasteFormat()->cfFormat);
}
// static
« no previous file with comments | « no previous file | app/gfx/font_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698