Certificate Transparency: Remove the obsolete invalid sct status.

content/common/ssl_status_serialization.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/common/ssl_status_serialization.h"
 
 #include <stdint.h>
 
 #include "base/logging.h"
 #include "base/numerics/safe_conversions.h"
@@ skipping 13 matching lines @@
       return true;
   }
   return false;
 }
 
 // Checks that an integer |sct_status| is a valid net::ct::SCTVerifyStatus enum
 // value. Returns true if valid, false otherwise.
 bool CheckSCTStatus(uint32_t sct_status) {
   switch (sct_status) {
     case net::ct::SCT_STATUS_LOG_UNKNOWN:
-    // INVALID is deprecated and should not be used anymore, but it
-    // might have been previously written into the disk cache.
-    case net::ct::SCT_STATUS_INVALID:
     case net::ct::SCT_STATUS_INVALID_SIGNATURE:
     case net::ct::SCT_STATUS_OK:
     case net::ct::SCT_STATUS_INVALID_TIMESTAMP:
       return true;
     case net::ct::SCT_STATUS_NONE:
       // SCT_STATUS_NONE should never happen, so it isn't valid to
       // receive a status of NONE in a serialized SSLStatus.
       return false;
+    case 2:

[estark, 2016/08/31 22:03:33]

Is this necessary? Can we just fall through to line 46?

[Ryan Sleevi, 2016/09/01 00:01:29]

+1

[Eran Messeri, 2016/09/01 14:16:34]

Done.

+      // Was SCT_STATUS_INVALID, which was removed.
+      return false;
   }
   return false;
 }
 
 }  // namespace
 
 namespace content {
 
 std::string SerializeSecurityInfo(const SSLStatus& ssl_status) {
   base::Pickle pickle;
@@ skipping 67 matching lines @@
   // Sanity check |key_exchange_info|: 0 or greater.
   if (ssl_status->key_exchange_info < 0) {
     *ssl_status = SSLStatus();
     return false;
   }
 
   return true;
 }
 
 }  // namespace content