[Offline Pages] Create a event/activity logger

components/offline_pages/offline_page_model.h

Index: components/offline_pages/offline_page_model.h
diff --git a/components/offline_pages/offline_page_model.h b/components/offline_pages/offline_page_model.h
index e869f2c99209e55b5822b8f635af09ec09fc9f60..d6e77f9af8624ccaa6cdd77faef8825457561d7a 100644
--- a/components/offline_pages/offline_page_model.h
+++ b/components/offline_pages/offline_page_model.h
@@ -14,6 +14,7 @@
 
 #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"
 
@@ -193,6 +194,15 @@ class OfflinePageModel : public base::SupportsUserData {
 
   // 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