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

chrome/utility/media_galleries/picasa_album_table_reader.cc

Index: chrome/utility/media_galleries/picasa_album_table_reader.cc
diff --git a/chrome/utility/media_galleries/picasa_album_table_reader.cc b/chrome/utility/media_galleries/picasa_album_table_reader.cc
index a02b03fed67baf3c0133010df4bb07409f76f2ab..07096d5a4dea41512cc66159501e7c3ff7145d21 100644
--- a/chrome/utility/media_galleries/picasa_album_table_reader.cc
+++ b/chrome/utility/media_galleries/picasa_album_table_reader.cc
@@ -24,7 +24,15 @@ base::Time TimeFromMicrosoftVariantTime(double variant_time) {
   base::TimeDelta variant_delta = base::TimeDelta::FromMicroseconds(
       static_cast<int64_t>(variant_time * base::Time::kMicrosecondsPerDay));
 
-  return base::Time::FromLocalExploded(kPmpVariantTimeEpoch) + variant_delta;
+  base::Time out_time;
+  if (!base::Time::FromLocalExploded(kPmpVariantTimeEpoch, &out_time)) {

[Lei Zhang, 2016/07/06 17:44:30]

Isn't |kPmpVariantTimeEpoch| also a known to be good date that should no fail?

[maksims (do not use this acc), 2016/07/07 05:48:14]

Done.

+    // TODO(maksims): implement error handling.
+    // We might just return |out_time|, which is Time(0).
+    NOTIMPLEMENTED();
+    return out_time;
+  }
+
+  return out_time + variant_delta;
 }
 
 }  // namespace