Correctly whitelist ftp directory listings.

content/renderer/render_view_impl.cc

 // Copyright (c) 2012 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 "content/renderer/render_view_impl.h"
 
 #include <algorithm>
 #include <cmath>
 #include <string>
 #include <vector>
@@ skipping 1286 matching lines @@
     params.security_info = response.securityInfo();
   }
 
   // Set the URL to be displayed in the browser UI to the user.
   params.url = GetLoadingUrl(frame);
 
   if (frame->document().baseURL() != params.url)
     params.base_url = frame->document().baseURL();
 
   GetRedirectChain(ds, &params.redirects);
+  bool is_multipart_payload = false;
+  WebURLResponseExtraDataImpl* extra_data = GetExtraDataFromResponse(response);
+  if (extra_data)
+    is_multipart_payload = extra_data->is_multipart_payload();

[marja, 2012/05/24 13:48:40]

nit: would "bool is_multipart_payload = extra_data &&
extra_data->is_multipart_payload();" be more compact / clear?

   params.should_update_history = !ds->hasUnreachableURL() &&
-      !response.isMultipartPayload() && (response.httpStatusCode() != 404);
+      !is_multipart_payload && (response.httpStatusCode() != 404);
 
   params.searchable_form_url = document_state->searchable_form_url();
   params.searchable_form_encoding =
       document_state->searchable_form_encoding();
 
   const PasswordForm* password_form_data =
       document_state->password_form_data();
   if (password_form_data)
     params.password_form = *password_form_data;
 
@@ skipping 4153 matching lines @@
 bool RenderViewImpl::WebWidgetHandlesCompositorScheduling() const {
   return !!RenderThreadImpl::current()->compositor_thread();
 }
 
 void RenderViewImpl::OnJavaBridgeInit() {
   DCHECK(!java_bridge_dispatcher_);
 #if defined(ENABLE_JAVA_BRIDGE)
   java_bridge_dispatcher_ = new JavaBridgeDispatcher(this);
 #endif
 }