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

Unified Diff: mojo/public/cpp/environment/lib/default_logger.cc

Issue 334263010: Mojo: Add GetMinimumLogLevel() to the logger interface. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebased Created 6 years, 6 months 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: mojo/public/cpp/environment/lib/default_logger.cc
diff --git a/mojo/public/cpp/environment/lib/default_logger.cc b/mojo/public/cpp/environment/lib/default_logger.cc
index 100b0d185ef792ef6466a0f2647f5af4651268a4..7f0116a92f0638cc43fe5c9212326fcacdcf3f6f 100644
--- a/mojo/public/cpp/environment/lib/default_logger.cc
+++ b/mojo/public/cpp/environment/lib/default_logger.cc
@@ -7,9 +7,13 @@
#include <stdio.h>
#include <stdlib.h> // For |abort()|.
+#include <algorithm>
+
namespace mojo {
namespace {
+MojoLogLevel g_minimum_log_level = MOJO_LOG_LEVEL_INFO;
+
const char* GetLogLevelString(MojoLogLevel log_level) {
if (log_level < MOJO_LOG_LEVEL_VERBOSE)
return "VERBOSE";
@@ -28,18 +32,36 @@ const char* GetLogLevelString(MojoLogLevel log_level) {
}
void LogMessage(MojoLogLevel log_level, const char* message) {
+ if (log_level < g_minimum_log_level)
+ return;
+
// TODO(vtl): Add timestamp also?
fprintf(stderr, "%s:%s\n", GetLogLevelString(log_level), message);
if (log_level >= MOJO_LOG_LEVEL_FATAL)
abort();
}
+MojoLogLevel GetMinimumLogLevel() {
+ return g_minimum_log_level;
+}
+
const MojoLogger kDefaultLogger = {
- LogMessage
+ LogMessage,
+ GetMinimumLogLevel
};
} // namespace
+namespace internal {
+
+// Declared in mojo/public/cpp/environment/lib/default_logger_internal.h:
+void SetMinimumLogLevel(MojoLogLevel minimum_log_level) {
+ g_minimum_log_level = std::min(minimum_log_level, MOJO_LOG_LEVEL_FATAL);
+}
+
+} // namespace internal
+
+// Declared in mojo/public/cpp/environment/default_logger.h:
const MojoLogger* GetDefaultLogger() {
return &kDefaultLogger;
}
« no previous file with comments | « mojo/public/cpp/environment/environment.h ('k') | mojo/public/cpp/environment/lib/default_logger_internal.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698