Avoid having every frame of a multipart response create a

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 "webkit/glue/multipart_response_delegate.h"
 
 #include "base/logging.h"
 #include "base/string_util.h"
 #include "net/base/net_util.h"
 #include "net/http/http_util.h"
@@ skipping 46 matching lines @@
     WebURLLoaderClient* client,
     WebURLLoader* loader,
     const WebURLResponse& response,
     const std::string& boundary)
     : client_(client),
       loader_(loader),
       original_response_(response),
       boundary_("--"),
       first_received_data_(true),
       processing_headers_(false),
-      stop_sending_(false) {
+      stop_sending_(false),
+      has_sent_first_response_(false) {
   // 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) {
@@ skipping 142 matching lines @@
   original_response_.visitHTTPHeaderFields(&copier);
 
   for (size_t i = 0; i < arraysize(kReplaceHeaders); ++i) {
     std::string name(kReplaceHeaders[i]);
     std::string value = net::GetSpecificHeader(headers, name);
     if (!value.empty()) {
       response.setHTTPHeaderField(WebString::fromUTF8(name),
                                   WebString::fromUTF8(value));
     }
   }
+  // To avoid recording every multipart load as a separate visit in
+  // the history database, we want to keep track of whether the response
+  // is part of a multipart payload.  We do want to record the first visit,
+  // so we only set isMultipartPayload to true after the first visit.
+  response.setIsMultipartPayload(has_sent_first_response_);
+  has_sent_first_response_ = true;
   // Send the response!
   client_->didReceiveResponse(loader_, response);
 
   return true;
 }
 
 // Boundaries are supposed to be preceeded with --, but it looks like gecko
 // doesn't require the dashes to exist.  See nsMultiMixedConv::FindToken.
 size_t MultipartResponseDelegate::FindBoundary() {
   size_t boundary_pos = data_.find(boundary_);
@@ skipping 92 matching lines @@
                            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;
 }
 
 }  // namespace webkit_glue