Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(666)

Unified Diff: chrome/browser/automation/automation_profile_impl.cc

Issue 3066004: ChromeFrame cookie requests would incorrectly get routed to the first host br... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 10 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/automation/automation_profile_impl.cc
===================================================================
--- chrome/browser/automation/automation_profile_impl.cc (revision 53631)
+++ chrome/browser/automation/automation_profile_impl.cc (working copy)
@@ -97,16 +97,6 @@
}
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);
- }
- }
-
net::CookieStore* original_cookie_store_;
scoped_refptr<AutomationResourceMessageFilter> automation_client_;
int tab_handle_;
@@ -130,23 +120,20 @@
net::CompletionCallback* callback) {
DCHECK(ChromeThread::CurrentlyOn(ChromeThread::IO));
- if (automation_client_.get()) {
- automation_client_->GetCookiesForUrl(tab_handle_, url, callback,
- cookie_store_.get());
- return net::ERR_IO_PENDING;
- }
- return net::ERR_ACCESS_DENIED;
+ AutomationResourceMessageFilter::GetCookiesForUrl(tab_handle_, url,
+ callback,
+ cookie_store_.get());
+ return net::ERR_IO_PENDING;
}
virtual int CanSetCookie(const GURL& url,
const GURL& first_party_for_cookies,
const std::string& cookie_line,
net::CompletionCallback* callback) {
- if (automation_client_.get()) {
- automation_client_->Send(new AutomationMsg_SetCookieAsync(0,
- tab_handle_, url, cookie_line));
- }
- return net::ERR_ACCESS_DENIED;
+ AutomationResourceMessageFilter::SetCookiesForUrl(tab_handle_, url,
+ cookie_line,
+ callback);
+ return net::ERR_IO_PENDING;
}
private:

Powered by Google App Engine
This is Rietveld 408576698