Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3652)

Unified Diff: chrome/browser/jumplist.cc

Issue 164537: Renames the function CreateTemporaryFilename to CreateTemporaryFile and track... (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: CommandLine fix Created 11 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/jumplist.cc
===================================================================
--- chrome/browser/jumplist.cc (revision 23574)
+++ chrome/browser/jumplist.cc (working copy)
@@ -243,18 +243,18 @@
// Retrieve the path to a temporary file.
// We don't have to care about the extension of this temporary file because
// JumpList does not care about it.
- std::wstring path;
- if (!file_util::CreateTemporaryFileNameInDir(icon_dir, &path))
+ FilePath path;
+ if (!file_util::CreateTemporaryFileInDir(FilePath(icon_dir), &path))
return false;
// Create an icon file from the favicon attached to the given |page|, and
// save it as the temporary file.
- if (!IconUtil::CreateIconFileFromSkBitmap(bitmap, path))
+ if (!IconUtil::CreateIconFileFromSkBitmap(bitmap, path.value()))
return false;
// Add this icon file to the list and return its absolute path.
// The IShellLink::SetIcon() function needs the absolute path to an icon.
- icon_path->assign(path);
+ icon_path->assign(path.value());
return true;
}
« no previous file with comments | « chrome/browser/extensions/extension_updater.cc ('k') | chrome/browser/renderer_host/file_system_accessor_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698