diff options
| author | maruel@chromium.org <maruel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-29 17:44:42 +0000 |
|---|---|---|
| committer | maruel@chromium.org <maruel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-29 17:44:42 +0000 |
| commit | b6128aaba9e7c148d1a71710b1d7de5d101f9967 (patch) | |
| tree | dca19e0d51b91edcd5e2203de2f3158c64bcdb5c /chrome_frame | |
| parent | a65b0ba084b3b197f817b88bd21b275e1790a629 (diff) | |
| download | chromium_src-b6128aaba9e7c148d1a71710b1d7de5d101f9967.zip chromium_src-b6128aaba9e7c148d1a71710b1d7de5d101f9967.tar.gz chromium_src-b6128aaba9e7c148d1a71710b1d7de5d101f9967.tar.bz2 | |
Move common code into process_util.cc.
Fix namespace usage.
Change ProcessEntry to have a common interface accross platforms and change ProcessFilter::Includes() to make use of it.
Split NamedProcessIterator in two.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/1689012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45953 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome_frame')
| -rw-r--r-- | chrome_frame/test_utils.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome_frame/test_utils.cc b/chrome_frame/test_utils.cc index 319aa54..e1377c7 100644 --- a/chrome_frame/test_utils.cc +++ b/chrome_frame/test_utils.cc @@ -244,10 +244,10 @@ class ArgumentFilter : public base::ProcessFilter { // Returns true to indicate set-inclusion and false otherwise. This method // should not have side-effects and should be idempotent. - virtual bool Includes(base::ProcessId pid, base::ProcessId parent_pid) const { + virtual bool Includes(const base::ProcessEntry& entry) const { bool found = false; std::wstring command_line; - if (GetCommandLineForProcess(pid, &command_line)) { + if (GetCommandLineForProcess(entry.pid(), &command_line)) { std::wstring::const_iterator it = std::search(command_line.begin(), command_line.end(), |