From dad1e3bc1d2894be09c2a5bcb98708044aa1664d Mon Sep 17 00:00:00 2001 From: "levin@chromium.org" Date: Fri, 13 Nov 2009 01:30:15 +0000 Subject: Revert 31862 - Ensure AutomationMsg_SetCookieAsync is sent on IO thread. These tests started failing with this checkin: AuthenticateSuccess AuthenticateWithTokenSuccess Review URL: http://codereview.chromium.org/387042 TBR=stoyan@chromium.org Review URL: http://codereview.chromium.org/389031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@31873 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/automation/automation_profile_impl.cc | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) (limited to 'chrome/browser') diff --git a/chrome/browser/automation/automation_profile_impl.cc b/chrome/browser/automation/automation_profile_impl.cc index bb9b106..4d217d6 100644 --- a/chrome/browser/automation/automation_profile_impl.cc +++ b/chrome/browser/automation/automation_profile_impl.cc @@ -63,7 +63,7 @@ class AutomationCookieStore : public net::CookieStore { if (cookie_set) { // TODO(eroman): Should NOT be accessing the profile from here, as this // is running on the IO thread. - SendIPCMessageOnIOThread(new AutomationMsg_SetCookieAsync(0, + automation_client_->Send(new AutomationMsg_SetCookieAsync(0, profile_->tab_handle(), url, cookie_line)); } return cookie_set; @@ -106,16 +106,6 @@ class AutomationCookieStore : public net::CookieStore { } protected: - void SendIPCMessageOnIOThread(IPC::Message* m) { - if (ChromeThread::CurrentlyOn(ChromeThread::IO)) { - automation_client_->Send(m); - } else { - Task* task = NewRunnableMethod(this, - &AutomationCookieStore::SendIPCMessageOnIOThread, m); - ChromeThread::PostTask(ChromeThread::IO, FROM_HERE, task); - } - } - AutomationProfileImpl* profile_; net::CookieStore* original_cookie_store_; IPC::Message::Sender* automation_client_; -- cgit v1.1