Comment #18 on issue 25354 by [email protected]: Need a fool-proof way  
of executing tasks on other threads without worrying about thread  
destruction order
http://code.google.com/p/chromium/issues/detail?id=25354

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

------------------------------------------------------------------------
r30688 | [email protected] | 2009-11-01 22:10:30 -0800 (Sun, 01 Nov 2009) |  
7 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/base/ref_counted.h?r1=30688&r2=30687
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/chrome_thread.h?r1=30688&r2=30687
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/chrome_thread_unittest.cc?r1=30688&r2=30687
    D /trunk/src/chrome/browser/renderer_host/file_system_accessor.cc
    D /trunk/src/chrome/browser/renderer_host/file_system_accessor.h
    D  
/trunk/src/chrome/browser/renderer_host/file_system_accessor_unittest.cc
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/resource_message_filter.cc?r1=30688&r2=30687
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/resource_message_filter.h?r1=30688&r2=30687
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome.gyp?r1=30688&r2=30687
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/ipc/ipc_channel_proxy.cc?r1=30688&r2=30687
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/ipc/ipc_channel_proxy.h?r1=30688&r2=30687

Add the ability for objects which derive from RefCountedThreadSafe to  
specify a destructor trait.  This allows browser objects to specify which  
thread they're terminated on.  The benefit is we avoid the need to do  
manual ref counting when an object posts tasks to itself on different  
threads, if an object must be destructed on a specific thread.

This patch adds initial support and only shows one example with  
ResourceMessageFilter.  I will do the rest in a follow-up patch to keep  
things small.

BUG=25354
TEST=added unit tests
Review URL: http://codereview.chromium.org/338065
------------------------------------------------------------------------


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