Update the TabContents::SavePage parameter to take FilePath instead of wstring to avoid conversions "To and From wstring".
BUG=None
TEST=run unit_tests.exe
Review URL: http://codereview.chromium.org/217008
Patch from tfarina.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27331 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/chrome/browser/tab_contents/tab_contents.cc b/chrome/browser/tab_contents/tab_contents.cc
index 24fd51a..d36d53fb 100644
--- a/chrome/browser/tab_contents/tab_contents.cc
+++ b/chrome/browser/tab_contents/tab_contents.cc
@@ -1127,15 +1127,12 @@
// Used in automated testing to bypass prompting the user for file names.
// Instead, the names and paths are hard coded rather than running them through
// file name sanitation and extension / mime checking.
-bool TabContents::SavePage(const std::wstring& main_file,
- const std::wstring& dir_path,
+bool TabContents::SavePage(const FilePath& main_file, const FilePath& dir_path,
SavePackage::SavePackageType save_type) {
// Stop the page from navigating.
Stop();
- save_package_ = new SavePackage(this, save_type,
- FilePath::FromWStringHack(main_file),
- FilePath::FromWStringHack(dir_path));
+ save_package_ = new SavePackage(this, save_type, main_file, dir_path);
return save_package_->Init();
}
