[OSX] Remove wide->NSString conversion

[OSX] Remove wide->NSString conversion

Since we a) have a PRESUBMIT check for wstring in new code and b) this code isn't used any more.... might as well kill it.

TBR=rsesek@chromium.org
R=mark@chromium.org

BUG=none
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=150002

[Mark Mentovai, 2012-07-24 16:35:28]

http://codereview.chromium.org/10818016/diff/1/base/sys_string_conversions.h
File base/sys_string_conversions.h (right):

http://codereview.chromium.org/10818016/diff/1/base/sys_string_conversions.h#...
base/sys_string_conversions.h:65: BASE_EXPORT CFStringRef
SysWideToCFStringRef(const std::wstring& wide);
Is this one used?

[groby-ooo-7-16, 2012-07-31 02:00:14]

http://codereview.chromium.org/10818016/diff/1/base/sys_string_conversions.h
File base/sys_string_conversions.h (right):

http://codereview.chromium.org/10818016/diff/1/base/sys_string_conversions.h#...
base/sys_string_conversions.h:65: BASE_EXPORT CFStringRef
SysWideToCFStringRef(const std::wstring& wide);
No, and neither are the other widestring funcs.
On 2012/07/24 16:35:28, Mark Mentovai wrote:
> Is this one used?

[Mark Mentovai, 2012-07-31 23:18:24]

http://codereview.chromium.org/10818016/diff/5001/base/sys_string_conversions...
File base/sys_string_conversions_mac.mm (right):

http://codereview.chromium.org/10818016/diff/5001/base/sys_string_conversions...
base/sys_string_conversions_mac.mm:155: CFStringRef SysWideToCFStringRef(const
std::wstring& wide) {
You removed this from the header.

[groby-ooo-7-16, 2012-08-03 21:55:09]

Third time's the charm. I hope.

[Mark Mentovai, 2012-08-03 22:00:08]

LGTM

[commit-bot: I haz the power, 2012-08-03 22:01:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/groby@chromium.org/10818016/11001

[commit-bot: I haz the power, 2012-08-03 22:15:40]

Try job failure for 10818016-11001 (retry) on android for steps "compile, build"
(clobber build).
It's a second try, previously, steps "compile, build" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android&nu...

[commit-bot: I haz the power, 2012-08-03 23:23:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/groby@chromium.org/10818016/11001

[commit-bot: I haz the power, 2012-08-04 02:28:38]

Change committed as 150002