Get rid of wstring variants of StringToFoo.

Get rid of wstring variants of StringToFoo.


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

[dsh, 2009-02-27 23:23:56]

Since the types are the same on Windows, there still are surely a lot of places
where wstring is being passed as a string16, but in fact, it may be better to do
this stuff now since it reduces the patch size.  It will at least enforce the
use of string16 on Linux.

[brettw, 2009-02-27 23:45:15]

LGTM

http://codereview.chromium.org/28281/diff/36/51
File chrome/browser/bookmarks/bookmark_codec.cc (right):

http://codereview.chromium.org/28281/diff/36/51#newcode180
Line 180: WideToUTF16Hack(last_modified_date)));
Don't wrap 4 spaces from random parts of the line. I would move
Time::FromInternalValue to the previous line and wrap the other args to 4 spaces
(same below).

http://codereview.chromium.org/28281/diff/36/52
File chrome/browser/bookmarks/bookmark_html_writer.cc (right):

http://codereview.chromium.org/28281/diff/36/52#newcode197
Line 197: base::Time::FromInternalValue(StringToInt64(WideToUTF16Hack(
See prev comment.

http://codereview.chromium.org/28281/diff/36/46
File chrome/browser/metrics/metrics_service.cc (right):

http://codereview.chromium.org/28281/diff/36/46#newcode623
Line 623: StringToInt64(WideToUTF16Hack(pref->GetString(
See previous comment.

http://codereview.chromium.org/28281/diff/36/57
File chrome/common/pref_service.cc (right):

http://codereview.chromium.org/28281/diff/36/57#newcode76
Line 76: resource_string)));
I'd wrap this and the next change to 4 spaces from the beginning of the line.