Make callers of FromUTC(Local)Exploded in net/ use new time API.

net/cert/x509_cert_types.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 "net/cert/x509_cert_types.h"
 
 #include <cstdlib>
 #include <cstring>
 
 #include "base/logging.h"
@@ skipping 53 matching lines @@
 
   exploded.year =         ParseIntAndAdvance(&field, year_length, &valid);
   exploded.month =        ParseIntAndAdvance(&field, 2, &valid);
   exploded.day_of_month = ParseIntAndAdvance(&field, 2, &valid);
   exploded.hour =         ParseIntAndAdvance(&field, 2, &valid);
   exploded.minute =       ParseIntAndAdvance(&field, 2, &valid);
   exploded.second =       ParseIntAndAdvance(&field, 2, &valid);
   if (valid && year_length == 2)
     exploded.year += exploded.year < 50 ? 2000 : 1900;
 
-  valid &= exploded.HasValidValues();
-
   if (!valid)
     return false;
 
-  *time = base::Time::FromUTCExploded(exploded);
-  return true;
+  // FromUTCExploded() can fail even though exploded.HasValidValues()

[eroman, 2016/12/08 01:39:19]

This comment is no longer relevant, please remove

[maksims (do not use this acc), 2016/12/13 08:56:09]

Done.

+  // returned true.
+  return base::Time::FromUTCExploded(exploded, time);
 }
 
 }  // namespace net