Add: check exploded time is properly converted

base/time/time.cc

-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright (c) 2016 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 "base/time/time.h"
 
 #include <cmath>
 #include <ios>
 #include <limits>
 #include <ostream>
 #include <sstream>
@@ skipping 325 matching lines @@
 bool Time::Exploded::HasValidValues() const {
   return is_in_range(month, 1, 12) &&
          is_in_range(day_of_week, 0, 6) &&
          is_in_range(day_of_month, 1, 31) &&
          is_in_range(hour, 0, 23) &&
          is_in_range(minute, 0, 59) &&
          is_in_range(second, 0, 60) &&
          is_in_range(millisecond, 0, 999);
 }
 
+bool Time::Exploded::operator!=(const Exploded& rhs) {
+  return (month != rhs.month || day_of_month != rhs.day_of_month ||

[Mark Mentovai, 2016/05/18 17:42:49]

year?

[maksims (do not use this acc), 2016/05/19 09:57:19]

Done.

+          hour != rhs.hour || minute != rhs.minute || second != rhs.second ||
+          millisecond != rhs.millisecond);

[Mark Mentovai, 2016/05/18 17:42:49]

The outer (parentheses) surround the entire expression and aren’t necessary.

[maksims (do not use this acc), 2016/05/19 09:57:19]

Done.

+}
+
 }  // namespace base