Remove usage of deprecated functions AppendToPath and ToWStringHack in importer_unittest.cc

Remove usage of deprecated functions AppendToPath and ToWStringHack in importer_unittest.cc

BUG=None
TEST=trybots

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

[tfarina, 2011-01-07 22:44:28]

Please, take a look.

[evanm, 2011-01-07 22:46:01]

LGTM either way

http://codereview.chromium.org/6147003/diff/1/chrome/browser/importer/importe...
File chrome/browser/importer/importer_unittest.cc (right):

http://codereview.chromium.org/6147003/diff/1/chrome/browser/importer/importe...
chrome/browser/importer/importer_unittest.cc:362: std::wstring path =
file_path.value();
Can do it on one line:
std::wstring path = test_path_.AppendASCII("Favorites").value();

http://codereview.chromium.org/6147003/diff/1/chrome/browser/importer/importe...
chrome/browser/importer/importer_unittest.cc:382: file_util::WriteFile(path +
L"\\InvalidUrlFile.url", "x", 1);
I wonder if we'll need to change more of this file once we change ::WriteFile to
only accept FilePath too.

[tfarina, 2011-01-07 22:50:00]

http://codereview.chromium.org/6147003/diff/1/chrome/browser/importer/importe...
File chrome/browser/importer/importer_unittest.cc (right):

http://codereview.chromium.org/6147003/diff/1/chrome/browser/importer/importe...
chrome/browser/importer/importer_unittest.cc:362: std::wstring path =
file_path.value();
On 2011/01/07 22:46:01, evanm wrote:
> Can do it on one line:
> std::wstring path = test_path_.AppendASCII("Favorites").value();

Done.

http://codereview.chromium.org/6147003/diff/1/chrome/browser/importer/importe...
chrome/browser/importer/importer_unittest.cc:382: file_util::WriteFile(path +
L"\\InvalidUrlFile.url", "x", 1);
On 2011/01/07 22:46:01, evanm wrote:
> I wonder if we'll need to change more of this file once we change ::WriteFile
to
> only accept FilePath too.

I'm scared about this, and I'd prefer to let more experienced programmers to do
the changes needed.

[evanm, 2011-01-07 22:55:43]

LGTM++