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

Issue 6091006: Use FilePath::BaseName instead of the deprecated file_util::GetFilenameFromPath. Part 2. (Closed)

Created:
10 years ago by tfarina
Modified:
9 years, 7 months ago
Reviewers:
Evan Martin, evanm
CC:
chromium-reviews, amit
Visibility:
Public.

Description

Use FilePath::BaseName instead of the deprecated file_util::GetFilenameFromPath. Part 2. BUG=24672 TEST=trybots Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=70128

Patch Set 1 #

Total comments: 2

Patch Set 2 : value #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M chrome_frame/utils.cc View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
tfarina
Please, take a look.
10 years ago (2010-12-23 14:54:26 UTC) #1
evanm
LGTM http://codereview.chromium.org/6091006/diff/1/chrome_frame/utils.cc File chrome_frame/utils.cc (right): http://codereview.chromium.org/6091006/diff/1/chrome_frame/utils.cc#newcode418 chrome_frame/utils.cc:418: std::wstring exe_name = FilePath(exe_path).BaseName().ToWStringHack(); Since this is Windows-specific ...
10 years ago (2010-12-23 18:13:33 UTC) #2
tfarina
10 years ago (2010-12-23 18:27:00 UTC) #3
http://codereview.chromium.org/6091006/diff/1/chrome_frame/utils.cc
File chrome_frame/utils.cc (right):

http://codereview.chromium.org/6091006/diff/1/chrome_frame/utils.cc#newcode418
chrome_frame/utils.cc:418: std::wstring exe_name =
FilePath(exe_path).BaseName().ToWStringHack();
On 2010/12/23 18:13:33, evanm wrote:
> Since this is Windows-specific code, could you use .value() instead of
> .ToWStringHack()?

Done.

Powered by Google App Engine
This is Rietveld 408576698