[Offline Pages] Create a event/activity logger

components/offline_pages/offline_page_model.h

 // Copyright 2015 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.
 
 #ifndef COMPONENTS_OFFLINE_PAGES_OFFLINE_PAGE_MODEL_H_
 #define COMPONENTS_OFFLINE_PAGES_OFFLINE_PAGE_MODEL_H_
 
 #include <stdint.h>
 
 #include <memory>
 #include <set>
 #include <string>
 #include <vector>
 
 #include "base/supports_user_data.h"
 #include "components/offline_pages/offline_page_archiver.h"
+#include "components/offline_pages/offline_page_model_event_logger.h"
 #include "components/offline_pages/offline_page_storage_manager.h"
 #include "components/offline_pages/offline_page_types.h"
 
 class GURL;
 namespace base {
 class Time;
 }  // namespace base
 
 namespace offline_pages {
 
@@ skipping 159 matching lines @@
   // archive files.
   virtual void ExpirePages(const std::vector<int64_t>& offline_ids,
                            const base::Time& expiration_time,
                            const base::Callback<void(bool)>& callback) = 0;
 
   // Returns the policy controller.
   virtual ClientPolicyController* GetPolicyController() = 0;
 
   // TODO(dougarnett): Remove this and its uses.
   virtual bool is_loaded() const = 0;
+
+  // Puts the current recorded logs into |storage_vector|.
+  virtual void GetLogs(std::vector<std::string>& storage_vector) = 0;
+
+  // Set whether to record logs.
+  virtual void SetIsLogging(bool should_record) = 0;
+
+ protected:
+  // Logger to facilitate recording of events.

[fgorski, 2016/06/23 16:34:06]

? I am not sure I follow.
Did you mean to add a logger here?
Is there any reason to duplicate the logging interface?

[chili, 2016/06/24 02:45:51]

I had a Logger here with all its implementations so child classes don't have to
copy.  I was advised against that, and didn't clear it up all the way.

 };
 
 }  // namespace offline_pages
 
 #endif  // COMPONENTS_OFFLINE_PAGES_OFFLINE_PAGE_MODEL_H_