Updates:
        Status: Fixed

Comment #2 on issue 29247 by [email protected]: Crash -  
AutomationRequestContext::AutomationCookieStore::SendIPCMessageOnIOThread
http://code.google.com/p/chromium/issues/detail?id=29247

New Revision: 32127

Log:
amit, please review everything, jam please review the changes to the  
tab_contents and
the
renderer_host sources.

Remove the AutomationProfileImpl class which wraps the Chrome profile for  
an external
tab
container, which hosts ChromeFrame. This object was used to carry a custom  
URL
request
context which was used to intercept HTTP requests and cookie requests  
issued by
external
tabs.

However as the life time of the automation profile class depended on the  
lifetime of
the external
tab container object this caused a number of crashes in objects which held  
on to the
automation profile
pointer retrieved from the associated tab contents. This does not happen in  
a
regualar Chrome browser
instance as the profile is deleted at the very end.

We can associate the automation URL request context with the underlying  
tab_contents
which would
eventually percolate down to the resource message filter. Doing this would  
avoid the
need for the
AutomationProfile class.

This CL achieves that.
Bug=27695,27662


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