[Hotword] Sync Audio History pref every 24 hours, when opening chrome://settings and . . .

chrome/browser/extensions/api/hotword_private/hotword_private_apitest.cc

 // Copyright 2014 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/command_line.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/browser/extensions/api/hotword_private/hotword_private_api.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/history/web_history_service.h"
@@ skipping 47 matching lines @@
   bool expected_success_;
   bool expected_value_;
 };
 
 // Make a mock audio history handler so that the method for getting the web
 // history can be overridden.
 class MockAudioHistoryHandler : public HotwordAudioHistoryHandler {
  public:
   MockAudioHistoryHandler(content::BrowserContext* context,
                           history::WebHistoryService* web_history)
-      : HotwordAudioHistoryHandler(context),
+      : HotwordAudioHistoryHandler(context,
+                                   base::MessageLoop::current()->task_runner()),
         web_history_(web_history) {}
   ~MockAudioHistoryHandler() override {}
 
   history::WebHistoryService* GetWebHistory() override {
     return web_history_.get();
   }
 
  private:
   scoped_ptr<history::WebHistoryService> web_history_;
 };
@@ skipping 356 matching lines @@
   EXPECT_TRUE(setListenerT2.WaitUntilSatisfied());
   EXPECT_TRUE(setListenerF2.WaitUntilSatisfied());
   EXPECT_TRUE(getListener2.WaitUntilSatisfied());
 }
 
 IN_PROC_BROWSER_TEST_F(HotwordPrivateApiTest, AudioHistoryNoWebHistory) {
   MockAudioHistoryHandler* handler =
       new MockAudioHistoryHandler(profile(), nullptr);
   service()->SetAudioHistoryHandler(handler);
 
-  ExtensionTestMessageListener setListenerT("set AH: false failure", false);
-  ExtensionTestMessageListener setListenerF("set AH: false failure", false);
-  ExtensionTestMessageListener getListener("get AH: false failure", false);
+  // Set an initial value for the audio logging pref.
+  PrefService* prefs = profile()->GetPrefs();
+  prefs->SetBoolean(prefs::kHotwordAudioLoggingEnabled, true);
+
+  ExtensionTestMessageListener setListenerT("set AH: true failure", false);
+  ExtensionTestMessageListener setListenerF("set AH: true failure", false);
+  ExtensionTestMessageListener getListener("get AH: true failure", false);
 
   ASSERT_TRUE(RunComponentExtensionTest("audioHistory")) << message_;
 
   EXPECT_TRUE(setListenerT.WaitUntilSatisfied());
   EXPECT_TRUE(setListenerF.WaitUntilSatisfied());
   EXPECT_TRUE(getListener.WaitUntilSatisfied());
 }
 
 IN_PROC_BROWSER_TEST_F(HotwordPrivateApiTest, AudioHistoryWebHistoryFailure) {
   MockWebHistoryService* web_history = new MockWebHistoryService(profile());
   MockAudioHistoryHandler* handler =
       new MockAudioHistoryHandler(profile(), web_history);
   service()->SetAudioHistoryHandler(handler);
   web_history->SetFailureState();
   // It shouldn't matter if this is set to true. GetAduioHistoryEnabled should
   // still return false.
   web_history->SetExpectedValue(true);
 
   ExtensionTestMessageListener setListenerT("set AH: false failure", false);
   ExtensionTestMessageListener setListenerF("set AH: false failure", false);
   ExtensionTestMessageListener getListener("get AH: false failure", false);
 
   ASSERT_TRUE(RunComponentExtensionTest("audioHistory")) << message_;
 
   EXPECT_TRUE(setListenerT.WaitUntilSatisfied());
   EXPECT_TRUE(setListenerF.WaitUntilSatisfied());
   EXPECT_TRUE(getListener.WaitUntilSatisfied());
 }