commit a1eabcdc93d118146639139257c2f5ccc3ffb061
author Gilad Arnold <garnold@chromium.org> Wed Jul 09 15:42:40 2014 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Fri Jul 11 17:00:26 2014 +0000
tree b69fe72fe71b85e88b59bea85677f68bb08ddf41
parent bfc44f70a99a74bdb714c84fc65be2ab50806bab

UpdateManager: Do not schedule update checks if OOBE did not complete.

This abides by the current logic, as found in
UpdateCheckScheduler::StaticCheck().  New unit test added to verify this
behavior.

BUG=chromium:358269
TEST=Unit tests.

Change-Id: I747e8a59408d6b93ceea62ef741b36e09f937a08
Reviewed-on: https://chromium-review.googlesource.com/207241
Reviewed-by: Alex Deymo <deymo@chromium.org>
Tested-by: Gilad Arnold <garnold@chromium.org>
Commit-Queue: Gilad Arnold <garnold@chromium.org>

update_manager/fake_config_provider.h

diff --git a/update_manager/fake_config_provider.h b/update_manager/fake_config_provider.h
index 373ff0a..fab01d1 100644
--- a/update_manager/fake_config_provider.h
+++ b/update_manager/fake_config_provider.h
@@ -15,13 +15,12 @@
  public:
   FakeConfigProvider() {}
 
- protected:
   virtual FakeVariable<bool>* var_is_oobe_enabled() override {
     return &var_is_oobe_enabled_;
   }
 
  private:
-  FakeVariable<bool> var_is_oobe_enabled_{
+  FakeVariable<bool> var_is_oobe_enabled_{  // NOLINT(whitespace/braces)
       "is_oobe_enabled", kVariableModeConst};
 
   DISALLOW_COPY_AND_ASSIGN(FakeConfigProvider);