Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(455)

Unified Diff: components/offline_pages/offline_event_logger.h

Issue 2489443002: Move all components/offline_pages/ files into component/offline_pages/core (Closed)
Patch Set: rebase Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/offline_pages/offline_event_logger.h
diff --git a/components/offline_pages/offline_event_logger.h b/components/offline_pages/offline_event_logger.h
deleted file mode 100644
index 0466c8fd28e8ad232abc2207452025919791f7e8..0000000000000000000000000000000000000000
--- a/components/offline_pages/offline_event_logger.h
+++ /dev/null
@@ -1,75 +0,0 @@
-// 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.
-
-#ifndef COMPONENTS_OFFLINE_PAGES_OFFLINE_EVENT_LOGGER_H_
-#define COMPONENTS_OFFLINE_PAGES_OFFLINE_EVENT_LOGGER_H_
-
-#include <deque>
-#include <string>
-#include <vector>
-
-#include "base/macros.h"
-
-namespace offline_pages {
-
-// Maximum number of recorded Logs to keep track of at any moment. Defined in
-// offline_event_logger.cc.
-extern const size_t kMaxLogCount;
-
-// Facilitates the logging of events. Subclasses should create methods that
-// call RecordActivity to write into the log. |SetIsLogging|, |GetLogs|, and
-// |Clear| are called from the chrome://offline-internals page.
-//
-// Logging should be done by calling the corresponding subclass methods when
-// a loggable event occurs (i.e. when status has changed for a save request
-// or when an offlined page has been accessed/saved).
-//
-// This log only keeps track of the last |kMaxLogCount| events.
-class OfflineEventLogger {
- public:
- // This client interface should be implemented by the class which provides the
- // ability to pipe the log somewhere else (Eg. a java class which can write
- // logs into a file). It's optional and uses SetClient() to attach the client
- // to the event logger instance.
- class Client {
- public:
- virtual ~Client(){};
- virtual void CustomLog(const std::string& message) = 0;
- };
-
- OfflineEventLogger();
-
- ~OfflineEventLogger();
-
- // Clears the recorded activities.
- void Clear();
-
- // Turns logging on/off.
- void SetIsLogging(bool is_logging);
-
- // Returns whether we are currently logging.
- bool GetIsLogging();
-
- // Dumps the current activity list into |records|.
- void GetLogs(std::vector<std::string>* records);
-
- // Write the activity into the cycling log if we are currently logging.
- void RecordActivity(const std::string& activity);
-
- // Sets the client for custom logging process if needed.
- void SetClient(Client* client);
-
- private:
- // Recorded offline page activities.
- std::deque<std::string> activities_;
-
- // Whether we are currently recording logs or not.
- bool is_logging_;
-
- // Not owned.
- Client* client_;
-};
-} // namespace offline_pages
-
-#endif // COMPONENTS_OFFLINE_PAGES_OFFLINE_EVENT_LOGGER_H_
« no previous file with comments | « components/offline_pages/downloads/offline_page_download_notifier.h ('k') | components/offline_pages/offline_event_logger.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698