Handle cases where the multipart boundary is improperly prefixed with "--"....

webkit/glue/multipart_response_delegate.cc

 // Copyright (c) 2006-2008 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 "config.h"
 #include <string>
 
 #include "base/compiler_specific.h"
 
 MSVC_PUSH_WARNING_LEVEL(0);
@@ skipping 15 matching lines @@
     WebCore::ResourceHandle* job,
     const WebCore::ResourceResponse& response,
     const std::string& boundary)
     : client_(client),
       job_(job),
       original_response_(response),
       boundary_("--"),
       first_received_data_(true),
       processing_headers_(false),
       stop_sending_(false) {
-  boundary_.append(boundary);
+  // Some servers report a boundary prefixed with "--".  See bug 5786.
+  if (StartsWithASCII(boundary, "--", true)) {
+    boundary_.assign(boundary);
+  } else {
+    boundary_.append(boundary);
+  }
 }
 
 void MultipartResponseDelegate::OnReceivedData(const char* data, int data_len) {
   // stop_sending_ means that we've already received the final boundary token. 
   // The server should stop sending us data at this point, but if it does, we
   // just throw it away.
   if (stop_sending_)
     return;
 
   // TODO(tc): Figure out what to use for length_received.  Maybe we can just
@@ skipping 262 matching lines @@
   std::string byte_range_upper_bound =
       content_range.substr(byte_range_upper_bound_start_offset,
                            byte_range_upper_bound_characters);
 
   if (!StringToInt(byte_range_lower_bound, content_range_lower_bound))
     return false;
   if (!StringToInt(byte_range_upper_bound, content_range_upper_bound))
     return false;
   return true;
 }