[Offline Pages] Expanding event logger to be used by harness.

chrome/browser/android/offline_pages/evaluation/evaluation_test_scheduler.cc

 // Copyright 2016 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 "chrome/browser/android/offline_pages/evaluation/evaluation_test_scheduler.h"
 
 #include "base/logging.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "chrome/browser/android/offline_pages/request_coordinator_factory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "components/offline_pages/background/device_conditions.h"
 #include "components/offline_pages/background/request_coordinator.h"
+#include "components/offline_pages/offline_event_logger.h"
 #include "net/base/network_change_notifier.h"
 
 namespace offline_pages {
 
 namespace android {
 
 namespace {
 
 void StartProcessing();
 
@@ skipping 20 matching lines @@
   Profile* profile = ProfileManager::GetLastUsedProfile();
   RequestCoordinator* coordinator =
       RequestCoordinatorFactory::GetInstance()->GetForBrowserContext(profile);
   coordinator->GetAllRequests(base::Bind(&GetAllRequestsDone));
 }
 
 }  // namespace
 
 void EvaluationTestScheduler::Schedule(
     const TriggerConditions& trigger_conditions) {
+  Profile* profile = ProfileManager::GetLastUsedProfile();
+  RequestCoordinator* coordinator =
+      RequestCoordinatorFactory::GetInstance()->GetForBrowserContext(profile);

[dougarnett, 2016/11/28 23:02:44]

how about getting the coordinator once and holding onto it in a member variable?

[romax, 2016/11/30 00:15:10]

Done.

+  coordinator->GetLogger()->RecordActivity(OFFLINE_LOG_TAG + "Start schedule!");
   base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE,
                                                 base::Bind(&StartProcessing));
 }
 
 void EvaluationTestScheduler::BackupSchedule(
     const TriggerConditions& trigger_conditions,
     long delay_in_seconds) {}
 
 void EvaluationTestScheduler::Unschedule() {}
 
 void EvaluationTestScheduler::ImmediateScheduleCallback(bool result) {
   base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE,
                                                 base::Bind(&StartProcessing));
 }
 
 }  // namespace android
 }  // namespace offline_pages