Merge pull request #3590 from brave/fix-p3a-dcheck-error

Account for initial run having a null value
brave · Oct 3, 2019 · d17efcf · d17efcf
2 parents 5524e2d + 2af7a57
commit d17efcf
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/components/p3a/brave_p3a_service.cc b/components/p3a/brave_p3a_service.cc
@@ -156,11 +156,15 @@ void BraveP3AService::Init() {
   // Do rotation if needed.
   const base::Time last_rotation =
       local_state_->GetTime(kLastRotationTimeStampPref);
-  const base::TimeDelta last_rotation_interval =
-      rotation_interval_.is_zero() ? TimeDeltaTillMonday(last_rotation)
-                                   : rotation_interval_;
-  if (base::Time::Now() - last_rotation > last_rotation_interval) {
+  if (last_rotation.is_null()) {
     DoRotation();
+  } else {
+    const base::TimeDelta last_rotation_interval =
+        rotation_interval_.is_zero() ? TimeDeltaTillMonday(last_rotation)
+                                     : rotation_interval_;
+    if (base::Time::Now() - last_rotation > last_rotation_interval) {
+      DoRotation();
+    }
   }
 
   // Init other components.