Comment #3 on issue 5323 by [email protected]: Remove platform ifdefs  
from sandbox.h
http://code.google.com/p/chromium/issues/detail?id=5323

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=7825

------------------------------------------------------------------------
r7825 | [email protected] | 2009-01-09 12:28:44 -0800 (Fri, 09 Jan 2009)  
| 3 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/chrome_dll_main.cc?r1=7825&r2=7824
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/scoped_ole_initializer.h
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser_main.cc?r1=7825&r2=7824
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome.xcodeproj/project.pbxproj?r1=7825&r2=7824
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/common.scons?r1=7825&r2=7824
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/common.vcproj?r1=7825&r2=7824
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/main_function_params.h
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/sandbox_init_wrapper.cc
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/sandbox_init_wrapper.h
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/plugin/plugin_main.cc?r1=7825&r2=7824
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/renderer_main.cc?r1=7825&r2=7824
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/sandbox/src/sandbox.h?r1=7825&r2=7824

remove chrome dependencies from win sandboxing headers. Wrap sandbox code to
make the main routine a little cleaner. Unify the parameters of each of  
the "main" entry points so we can more easily abstract platform differences  
in the future. BUG=5323
Review URL: http://codereview.chromium.org/17426
------------------------------------------------------------------------


--
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
-~----------~----~----~----~------~----~------~--~---

Reply via email to