CREDENTIAL: Rework the integration with Fetch (2/2)

third_party/WebKit/Source/platform/exported/WebURLRequest.cpp

 /*
  * Copyright (C) 2009 Google 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 190 matching lines @@
 
 void WebURLRequest::visitHTTPHeaderFields(WebHTTPHeaderVisitor* visitor) const
 {
     const HTTPHeaderMap& map = m_private->m_resourceRequest->httpHeaderFields();
     for (HTTPHeaderMap::const_iterator it = map.begin(); it != map.end(); ++it)
         visitor->visitHeader(it->key, it->value);
 }
 
 WebHTTPBody WebURLRequest::httpBody() const
 {
-    // TODO(mkwst): This is wrong, as it means that we're producing the body
-    // before any ServiceWorker has a chance to operate, which means we're
-    // revealing data to the SW that we ought to be hiding. Baby steps.
-    // https://crbug.com/599597
-    if (m_private->m_resourceRequest->attachedCredentialBody())
-        return WebHTTPBody(m_private->m_resourceRequest->attachedCredentialBody());
     return WebHTTPBody(m_private->m_resourceRequest->httpBody());
 }
 
 void WebURLRequest::setHTTPBody(const WebHTTPBody& httpBody)
 {
     m_private->m_resourceRequest->setHTTPBody(httpBody);
 }
 
 WebHTTPBody WebURLRequest::attachedCredentialBody() const
 {
@@ skipping 259 matching lines @@
     // Subclasses may call this directly so a self-assignment check is needed
     // here as well as in the public assign method.
     if (m_private == p)
         return;
     if (m_private)
         m_private->dispose();
     m_private = p;
 }
 
 } // namespace blink