parseHTTPHeaders should return the number of consumed bytes.

Source/platform/network/HTTPRequest.cpp

 /*
  * Copyright (C) 2011 Apple Inc. All Rights Reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 63 matching lines @@
     m_url = KURL(KURL(), url);
     return result;
 }
 
 size_t HTTPRequest::parseHeaders(const char* data, size_t length, String& failureReason)
 {
     const char* p = data;
     const char* end = data + length;
     AtomicString name;
     AtomicString value;
-    for (; p < data + length; p++) {
+    while (p < data + length) {
         size_t consumedLength = parseHTTPHeader(p, end - p, failureReason, name, value);
         if (!consumedLength)
             return 0;
         p += consumedLength;
         if (name.isEmpty())
             break;

[tyoshino (SeeGerritForStatus), 2014/06/05 08:24:15]

we never reached here before your last fix?

[yhirano, 2014/06/05 08:33:20]

Yes, when "\r\n" is consumed.

         m_headerFields.add(name, value);
     }
     return p - data;
 }
 
 size_t HTTPRequest::parseRequestBody(const char* data, size_t length)
 {
     return parseHTTPRequestBody(data, length, m_body);
 }
 
 HTTPRequest::HTTPRequest()
     : m_httpVersion(Unknown)
 {
 }
 
 HTTPRequest::HTTPRequest(const String& requestMethod, const KURL& url, HTTPVersion version)
     : m_url(url)
     , m_httpVersion(version)
     , m_requestMethod(requestMethod)
 {
 }
 
 HTTPRequest::~HTTPRequest()
 {
 }
 
 } // namespace WebCore