Add SafeBrowsing support for checking downloaded zip files that contain executables.

chrome/common/safe_browsing/csd.proto

 // 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.
 //
 // Client side phishing and malware detection request and response
 // protocol buffers.  Those protocol messages should be kept in sync
 // with the server implementation.
 //
 // If you want to change this protocol definition or you have questions
 // regarding its format please contact chrome-anti-phishing@googlegroups.com.
@@ skipping 150 matching lines @@
   // Name of the file where the download would be stored if the
   // download completes.  E.g., "bla.exe".
   optional string file_basename = 9;
 
   // Starting with Chrome M19 we're also sending back pings for Chrome
   // extensions that get downloaded by users.
   enum DownloadType {
     WIN_EXECUTABLE = 0;    // Currently all .exe, .cab and .msi files.
     CHROME_EXTENSION = 1;  // .crx files.
     ANDROID_APK = 2;       // .apk files.
+    // .zip files containing one of the above executable types.
+    ZIPPED_WIN_EXECUTABLE = 3;

[mattm, 2012/05/11 22:42:06]

APK isn't a win executable?

[Brian Ryner, 2012/05/11 23:09:41]

It's not, no.  Are you suggesting that we don't ping on APK's inside ZIP files,
or to update this comment to not say "executable"?

[mattm, 2012/05/11 23:12:32]

Oh just that "zipped win executable" contains "one of the above" types, when one
of those types isn't a win executable.  So, more that it shouldn't say "win". 

[Brian Ryner, 2012/05/12 00:06:59]

Sounds reasonable, renamed the enum.

   }
   optional DownloadType download_type = 10 [default = WIN_EXECUTABLE];
 }
 
 message ClientDownloadResponse {
   enum Verdict {
     // Download is considered safe.
     SAFE = 0;
     // Download is considered dangerous.  Chrome should show a warning to the
     // user.
     DANGEROUS = 1;
     // Download is unknown.  Chrome should display a less severe warning.
     UNCOMMON = 2;
   }
   required Verdict verdict = 1;
 }