Fix content-encoding handling with buggy servers.

net/filter/filter.cc

 // Copyright 2014 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 "net/filter/filter.h"
 
 #include "base/files/file_path.h"
 #include "base/strings/string_util.h"
 #include "net/base/io_buffer.h"
 #include "net/base/mime_util.h"
@@ skipping 152 matching lines @@
     if (LowerCaseEqualsASCII(mime_type, kApplicationXGzip) ||
         LowerCaseEqualsASCII(mime_type, kApplicationGzip) ||
         LowerCaseEqualsASCII(mime_type, kApplicationXGunzip))
       // The server has told us that it sent us gziped content with a gzip
       // content encoding.  Sadly, Apache mistakenly sets these headers for all
       // .gz files.  We match Firefox's nsHttpChannel::ProcessNormal and ignore
       // the Content-Encoding here.
       encoding_types->clear();
 
     GURL url;
+    std::string server_filename;
     success = filter_context.GetURL(&url);
     DCHECK(success);
-    base::FilePath filename =
-        base::FilePath().AppendASCII(url.ExtractFileName());
+    base::FilePath filename;
+    if (filter_context.GetFilename(&server_filename))
+      filename = base::FilePath().AppendASCII(server_filename);

[asanka, 2014/03/20 23:07:16]

Nit: server_filename is UTF-8, not necessarily ASCII, if it's obtained via
HttpContentDisposition. The encoding of url may not be ASCII either. 

Passing a non-ASCII string to AppendASCII isn't terrible since we are only
interested in extracting the extension and matching it against ASCII strings.
However, it'll trip a DCHECK on a DCHECK enabled build and relies on
implementation quirks to work.

+    else
+      filename = base::FilePath().AppendASCII(url.ExtractFileName());
     base::FilePath::StringType extension = filename.Extension();
 
     if (filter_context.IsDownload()) {
       // We don't want to decompress gzipped files when the user explicitly
       // asks to download them.
       // For the case of svgz files, we use the extension to distinguish
       // between svgz files and svg files compressed with gzip by the server.
       // When viewing a .svgz file, we need to uncompress it, but we don't
       // want to do that when downloading.
       // See Firefox's nonDecodableExtensions in nsExternalHelperAppService.cpp
@@ skipping 207 matching lines @@
 
 void Filter::PushDataIntoNextFilter() {
   IOBuffer* next_buffer = next_filter_->stream_buffer();
   int next_size = next_filter_->stream_buffer_size();
   last_status_ = ReadFilteredData(next_buffer->data(), &next_size);
   if (FILTER_ERROR != last_status_)
     next_filter_->FlushStreamBuffer(next_size);
 }
 
 }  // namespace net