Add UMA for measuring Content-Dispostion header use and abuse.

net/http/http_content_disposition.h

 // 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.
 
 #ifndef NET_HTTP_HTTP_CONTENT_DISPOSITION_H_
 #define NET_HTTP_HTTP_CONTENT_DISPOSITION_H_
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "net/base/net_export.h"
 
 namespace net {
 
 class NET_EXPORT HttpContentDisposition {
  public:
   enum Type {
     INLINE,
     ATTACHMENT,
   };
 
+  // Properties of the Content-Disposition header. Used for UMA.
+  struct ParseResult {
+    ParseResult();
+
+    // A valid disposition-type was present.
+    bool has_disposition_type;
+
+    // The disposition-type was not 'inline' or 'attachment'.
+    bool has_unknown_disposition_type;
+
+    // Has a valid non-empty 'name' attribute.
+    bool has_name;
+
+    // Has a valid non-empty 'filename' attribute.
+    bool has_filename;
+
+    // Has a valid non-empty 'filename*' attribute.
+    bool has_ext_filename;
+
+    // The following fields are properties of the 'filename' attribute:
+
+    // quoted-string contained non-ASCII characters.

[rvargas (doing something else), 2012/12/14 22:49:54]

nit: I know why you do it this way, but do you mind capitalizing Quoted-string?
And using contains instead of contained

[asanka, 2012/12/15 01:57:32]

Done.

+    bool has_non_ascii_strings;
+
+    // quoted-string contained percent-encoding.
+    bool has_percent_encoded_strings;
+
+    // quoted-string contained RFC 2047 encoded words.
+    bool has_rfc2047_encoded_strings;
+  };
+
   HttpContentDisposition(const std::string& header,
                          const std::string& referrer_charset);
   ~HttpContentDisposition();
 
   bool is_attachment() const { return type() == ATTACHMENT; }
 
   Type type() const { return type_; }
   const std::string& filename() const { return filename_; }
 
+  const ParseResult& parse_result() const { return parse_result_; }
+
  private:
   void Parse(const std::string& header, const std::string& referrer_charset);
   std::string::const_iterator ConsumeDispositionType(
       std::string::const_iterator begin, std::string::const_iterator end);
 
   Type type_;
   std::string filename_;
+  ParseResult parse_result_;
 
   DISALLOW_COPY_AND_ASSIGN(HttpContentDisposition);
 };
 
 }  // namespace net
 
 #endif  // NET_HTTP_HTTP_CONTENT_DISPOSITION_H_