Comment #1 on issue 25272 by [email protected]: Cleanup: make sure we use appropriate types for PIDs http://code.google.com/p/chromium/issues/detail?id=25272
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=29511
------------------------------------------------------------------------
r29511 | [email protected] | 2009-10-20 08:09:55 -0700 (Tue, 20 Oct
2009) | 8 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/process_util.h?r1=29511&r2=29510
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/process_util_posix.cc?r1=29511&r2=29510
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/chrome_dll_main.cc?r1=29511&r2=29510
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/memory_details.h?r1=29511&r2=29510
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/memory_details_win.cc?r1=29511&r2=29510
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/process_watcher_posix.cc?r1=29511&r2=29510
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/render_thread.cc?r1=29511&r2=29510
Cleanup: change PIDs to base::ProcessId (or pid_t, as appropriate).
We probably want to discourage the use of ints for PIDs. This is a start;
there are many other places where we should fix this.
BUG=25272
TEST=none
Review URL: http://codereview.chromium.org/300010
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
