AU: Skip all automatic update checks if OOBE is not complete.

update_check_scheduler.cc

 // Copyright (c) 2010 The Chromium OS 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 "update_engine/update_check_scheduler.h"
 
 #include "update_engine/utils.h"
 
 namespace chromeos_update_engine {
 
@@ skipping 32 matching lines @@
   // Kicks off periodic update checks. The first check is scheduled
   // |kTimeoutOnce| seconds from now. Subsequent checks are scheduled by
   // ScheduleNextCheck, normally at |kTimeoutPeriodic|-second intervals.
   ScheduleCheck(kTimeoutOnce, kTimeoutRegularFuzz);
 }
 
 bool UpdateCheckScheduler::IsBootDeviceRemovable() {
   return utils::IsRemovableDevice(utils::RootDevice(utils::BootDevice()));
 }
 
+bool UpdateCheckScheduler::IsOOBEComplete() {
+  return utils::IsOOBEComplete();
+}
+
 bool UpdateCheckScheduler::IsOfficialBuild() {
   return utils::IsOfficialBuild();
 }
 
 guint UpdateCheckScheduler::GTimeoutAddSeconds(guint interval,
                                                GSourceFunc function) {
   return g_timeout_add_seconds(interval, function, this);
 }
 
 void UpdateCheckScheduler::ScheduleCheck(int interval, int fuzz) {
   if (!CanSchedule()) {
     return;
   }
   last_interval_ = interval;
   interval = utils::FuzzInt(interval, fuzz);
   if (interval < 0) {
     interval = 0;
   }
   GTimeoutAddSeconds(interval, StaticCheck);
   scheduled_ = true;
   LOG(INFO) << "Next update check in " << interval << " seconds.";
 }
 
 gboolean UpdateCheckScheduler::StaticCheck(void* scheduler) {
   UpdateCheckScheduler* me = reinterpret_cast<UpdateCheckScheduler*>(scheduler);
   CHECK(me->scheduled_);
   me->scheduled_ = false;
-  me->update_attempter_->Update("", "");
+  if (me->IsOOBEComplete()) {
+    me->update_attempter_->Update("", "");
+  } else {
+    // Skips all automatic update checks if the OOBE process is not complete and
+    // schedules a new check as if it is the first one.
+    LOG(WARNING) << "Skipping update check because OOBE is not complete.";
+    me->ScheduleCheck(kTimeoutOnce, kTimeoutRegularFuzz);
+  }
   // This check ensures that future update checks will be or are already
   // scheduled. The check should never fail. A check failure means that there's
   // a bug that will most likely prevent further automatic update checks. It
   // seems better to crash in such cases and restart the update_engine daemon
   // into, hopefully, a known good state.
   CHECK(me->update_attempter_->status() != UPDATE_STATUS_IDLE ||
         !me->CanSchedule());
   return FALSE;  // Don't run again.
 }
 
@@ skipping 34 matching lines @@
   ScheduleCheck(interval, fuzz);
 }
 
 void UpdateCheckScheduler::SetUpdateStatus(UpdateStatus status) {
   if (status == UPDATE_STATUS_IDLE) {
     ScheduleNextCheck();
   }
 }
 
 }  // namespace chromeos_update_engine