diff options
author | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-14 19:36:31 +0000 |
---|---|---|
committer | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-14 19:36:31 +0000 |
commit | b65de8b9d5bfa020d97122c1eff2115fa8e97561 (patch) | |
tree | 3eb86aed4e3352cbca7cc4b78e76f5ee26d7388c /chrome/browser/printing | |
parent | 5e70f14644629b7f9fe9126553758aed25e17128 (diff) | |
download | chromium_src-b65de8b9d5bfa020d97122c1eff2115fa8e97561.zip chromium_src-b65de8b9d5bfa020d97122c1eff2115fa8e97561.tar.gz chromium_src-b65de8b9d5bfa020d97122c1eff2115fa8e97561.tar.bz2 |
Fix to use FilePath in more unittests.
Landing patch for Thiago Farina, original review: http://codereview.chromium.org/187005
BUG=None
TEST=run unit_tests.exe, base_unittests.exe.
Review URL: http://codereview.chromium.org/193101
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26142 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/printing')
-rw-r--r-- | chrome/browser/printing/printing_layout_uitest.cc | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/chrome/browser/printing/printing_layout_uitest.cc b/chrome/browser/printing/printing_layout_uitest.cc index f135566..6b86e83 100644 --- a/chrome/browser/printing/printing_layout_uitest.cc +++ b/chrome/browser/printing/printing_layout_uitest.cc @@ -27,10 +27,10 @@ const wchar_t kDocRoot[] = L"chrome/test/data"; class PrintingLayoutTest : public PrintingTest<UITest> { public: PrintingLayoutTest() { - emf_path_ = browser_directory_.ToWStringHack(); - file_util::AppendToPath(&emf_path_, L"metafile_dumps"); + emf_path_ = browser_directory_; + emf_path_ = emf_path_.AppendASCII("metafile_dumps"); launch_arguments_.AppendSwitchWithValue(L"debug-print", - L'"' + emf_path_ + L'"'); + L'"' + emf_path_.value() + L'"'); show_window_ = true; } @@ -156,8 +156,7 @@ class PrintingLayoutTest : public PrintingTest<UITest> { bool found_emf = false; bool found_prn = false; for (int i = 0; i < 100; ++i) { - file_util::FileEnumerator enumerator( - FilePath::FromWStringHack(emf_path()), false, + file_util::FileEnumerator enumerator(emf_path(), false, file_util::FileEnumerator::FILES); emf_file.clear(); prn_file.clear(); @@ -198,11 +197,11 @@ class PrintingLayoutTest : public PrintingTest<UITest> { return CommandLine::ForCurrentProcess()->HasSwitch(kGenerateSwitch); } - const std::wstring& emf_path() const { return emf_path_; } + const FilePath& emf_path() const { return emf_path_; } - std::wstring emf_path_; + FilePath emf_path_; - DISALLOW_EVIL_CONSTRUCTORS(PrintingLayoutTest); + DISALLOW_COPY_AND_ASSIGN(PrintingLayoutTest); }; // Tests that don't need UI access. |