Fix FullTabModeIE_ChromeFrameDeleteCookieTest. The problem was that a domain...

chrome_frame/urlmon_url_request.cc

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome_frame/urlmon_url_request.h"
 
 #include <wininet.h>
 #include <urlmon.h>
 
 #include "base/scoped_ptr.h"
@@ skipping 632 matching lines @@
     }
   }
 
   return hr;
 }
 
 STDMETHODIMP UrlmonUrlRequest::OnResponse(DWORD dwResponseCode,
     const wchar_t* response_headers, const wchar_t* request_headers,
     wchar_t** additional_headers) {
   DCHECK(worker_thread_ != NULL);
+  DLOG(INFO) << __FUNCTION__ << " " << url() << std::endl << " headers: " <<
+      std::endl << response_headers;
   DCHECK_EQ(PlatformThread::CurrentId(), worker_thread_->thread_id());
 
   if (!binding_) {
     DCHECK(redirect_url_.empty() == false);
     DLOG(WARNING) << __FUNCTION__
                   << ": Ignoring as the binding was aborted due to a redirect";
     return S_OK;
   }
 
   std::string raw_headers = WideToUTF8(response_headers);
@@ skipping 23 matching lines @@
       return E_FAIL;
     }
   }
 
   std::wstring url_for_persistent_cookies =
       redirect_url_.empty() ? UTF8ToWide(url()) : redirect_url_;
 
   std::string persistent_cookies;
 
   DWORD cookie_size = 0;  // NOLINT
+  // Note that there's really no way for us here to distinguish session cookies
+  // from persistent cookies here.  Session cookies should get filtered
+  // out on the chrome side as to not be added again.
   InternetGetCookie(url_for_persistent_cookies.c_str(), NULL, NULL,
                     &cookie_size);
   if (cookie_size) {
     scoped_array<wchar_t> cookies(new wchar_t[cookie_size + 1]);
     if (!InternetGetCookie(url_for_persistent_cookies.c_str(), NULL,
                            cookies.get(), &cookie_size)) {
       NOTREACHED() << "InternetGetCookie failed. Error: " << GetLastError();
     } else {
       persistent_cookies = WideToUTF8(cookies.get());
     }
@@ skipping 378 matching lines @@
       ret = net::ERR_ACCESS_DENIED;
       break;
 
     default:
       DLOG(WARNING)
           << StringPrintf("TODO: translate HRESULT 0x%08X to net::Error", hr);
       break;
   }
   return ret;
 }