Comment #5 on issue 8348 by [email protected]: unfork pe_image.h / pe_image.cc http://code.google.com/p/chromium/issues/detail?id=8348
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=25611
------------------------------------------------------------------------
r25611 | [email protected] | 2009-09-07 22:34:25 -0700 (Mon, 07 Sep 2009)
| 10 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/pe_image.cc?r1=25611&r2=25610
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/pe_image.h?r1=25611&r2=25610
M
http://src.chromium.org/viewvc/chrome/trunk/src/sandbox/sandbox.gyp?r1=25611&r2=25610
M
http://src.chromium.org/viewvc/chrome/trunk/src/sandbox/src/eat_resolver.cc?r1=25611&r2=25610
M
http://src.chromium.org/viewvc/chrome/trunk/src/sandbox/src/interception.cc?r1=25611&r2=25610
D /trunk/src/sandbox/src/pe_image.cc
D /trunk/src/sandbox/src/pe_image.h
D /trunk/src/sandbox/src/pe_image_unittest.cc
M
http://src.chromium.org/viewvc/chrome/trunk/src/sandbox/src/policy_broker.cc?r1=25611&r2=25610
M
http://src.chromium.org/viewvc/chrome/trunk/src/sandbox/src/resolver.cc?r1=25611&r2=25610
M
http://src.chromium.org/viewvc/chrome/trunk/src/sandbox/src/sandbox_nt_util.cc?r1=25611&r2=25610
M
http://src.chromium.org/viewvc/chrome/trunk/src/sandbox/src/service_resolver.cc?r1=25611&r2=25610
M
http://src.chromium.org/viewvc/chrome/trunk/src/sandbox/src/sidestep_resolver.cc?r1=25611&r2=25610
M
http://src.chromium.org/viewvc/chrome/trunk/src/sandbox/src/target_process.cc?r1=25611&r2=25610
Fix issue 8348: unfork pe_image.h / pe_image.cc
Moved versions of those files from sandbox/src/ to base/ (overwrite
versions in
base/ to avoid 64-bit warning).
Removed 'sandbox' namespace, adapted other files as necessary.
BUG=8348
TEST=none
Original review URL: http://codereview.chromium.org/179039
Patch by rsteiner
------------------------------------------------------------------------
--
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
-~----------~----~----~----~------~----~------~--~---
