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

Issue 102014: Rewrite plugin_tests.cc to use FilePath and file_utils.h instead... (Closed)

Created:
11 years, 7 months ago by Amanda Walker
Modified:
9 years, 7 months ago
Reviewers:
Mark Mentovai
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Rewrite plugin_tests.cc to use FilePath and file_utils.h instead of Windows file & string manipulation, enable plugin tests on the Mac. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=14968

Patch Set 1 #

Patch Set 2 : '' #

Total comments: 1

Patch Set 3 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -12 lines) Patch
M webkit/tools/test_shell/plugin_tests.cc View 1 2 5 chunks +23 lines, -11 lines 0 comments Download
M webkit/tools/test_shell/test_shell.gyp View 2 chunks +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Amanda Walker
11 years, 7 months ago (2009-04-30 17:41:45 UTC) #1
Mark Mentovai
LGTM http://codereview.chromium.org/102014/diff/2002/2004 File webkit/tools/test_shell/plugin_tests.cc (right): http://codereview.chromium.org/102014/diff/2002/2004#newcode36 Line 36: PathService::Get(base::DIR_EXE, &current_directory); current_directory is not the right ...
11 years, 7 months ago (2009-04-30 18:01:33 UTC) #2
Amanda Walker
On 2009/04/30 18:01:33, Mark Mentovai wrote: > http://codereview.chromium.org/102014/diff/2002/2004#newcode36 > Line 36: PathService::Get(base::DIR_EXE, &current_directory); > current_directory ...
11 years, 7 months ago (2009-04-30 18:15:40 UTC) #3
Amanda Walker
11 years, 7 months ago (2009-04-30 18:36:29 UTC) #4
On 2009/04/30 18:01:33, Mark Mentovai wrote:
> Line 36: PathService::Get(base::DIR_EXE, &current_directory);
> current_directory is not the right name for this, is it?

Changed to "executable_directory", also fixed plugin cleanup so that the test
will run twice in a row :-).  Will commit as soon as the tree opens.

Powered by Google App Engine
This is Rietveld 408576698