Fix ChromeFrame referrer test which was flaky due to incorrect http header parsing. To pass the referrer

to Chrome during navigation we were converting normalized HTTP headers which are terminated with \r\n to
raw headers and attempting to parse them. Converting to raw headers basically replaces \r\n with the NULL
terminator which causes the parsing to fail as a result of which we fail to retrieve the referrer.

Fix is to not convert the headers to raw and just parse the orignal headers.

The other change is to only flag success or failure in the referrer frame test if we are running in Chrome.
This page loads initially in IE as well due to the vagaries of the http equiv patch. Flagging an error
if we don't find a referrer basically causes the test to fail.

This should fix http://code.google.com/p/chromium/issues/detail?id=34812

Will remove the flaky attribute on the ChromeFrame Referrer test in a subsequent CL if all goes well.

Bug=34812


Review URL: http://codereview.chromium.org/1128013

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@42372 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 2 insertions, 5 deletions

diff --git a/chrome_frame/urlmon_moniker.cc b/chrome_frame/urlmon_moniker.cc
index 85fad9f..0873f13 100644
--- a/chrome_frame/urlmon_moniker.cc
+++ b/chrome_frame/urlmon_moniker.cc
@@ -33,11 +33,8 @@ std::string FindReferrerFromHeaders(const wchar_t* headers,
     if (!both_headers[i])
       continue;
     std::string raw_headers_utf8 = WideToUTF8(both_headers[i]);
-    std::string http_headers =
-        net::HttpUtil::AssembleRawHeaders(raw_headers_utf8.c_str(),
-                                          raw_headers_utf8.length());
-    net::HttpUtil::HeadersIterator it(http_headers.begin(),
-                                      http_headers.end(), "\r\n");
+    net::HttpUtil::HeadersIterator it(raw_headers_utf8.begin(),
+                                      raw_headers_utf8.end(), "\r\n");
     while (it.GetNext()) {
       if (LowerCaseEqualsASCII(it.name(), "referer")) {
         referrer = it.values();