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

Unified Diff: third_party/WebKit/Source/core/inspector/ConsoleMessageStorage.cpp

Issue 2151273003: [DevTools] Move browser logging from Console domain to Log domain. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@internals-method
Patch Set: protocol improvements Created 4 years, 5 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: third_party/WebKit/Source/core/inspector/ConsoleMessageStorage.cpp
diff --git a/third_party/WebKit/Source/core/inspector/ConsoleMessageStorage.cpp b/third_party/WebKit/Source/core/inspector/ConsoleMessageStorage.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..70ba51e1125066f7235ca186ce0853536ab44ed2
--- /dev/null
+++ b/third_party/WebKit/Source/core/inspector/ConsoleMessageStorage.cpp
@@ -0,0 +1,75 @@
+// 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 "core/inspector/ConsoleMessageStorage.h"
+
+#include "core/inspector/ConsoleMessage.h"
+#include "core/inspector/InspectorInstrumentation.h"
+
+namespace blink {
+
+static const unsigned maxConsoleMessageCount = 1000;
+
+ConsoleMessageStorage::ConsoleMessageStorage()
+ : m_expiredCount(0)
+ , m_mutedCount(0)
+{
+}
+
+bool ConsoleMessageStorage::addConsoleMessage(ExecutionContext* context, ConsoleMessage* message)
+{
+ if (m_mutedCount)
+ return false;
+ InspectorInstrumentation::consoleMessageAdded(context, message);
+ DCHECK(m_messages.size() <= maxConsoleMessageCount);
+ if (m_messages.size() == maxConsoleMessageCount) {
+ ++m_expiredCount;
+ m_messages.removeFirst();
+ }
+ m_messages.append(message);
+ return true;
+}
+
+void ConsoleMessageStorage::clear()
+{
+ m_messages.clear();
+ m_expiredCount = 0;
+}
+
+void ConsoleMessageStorage::mute()
+{
+ ++m_mutedCount;
+}
+
+void ConsoleMessageStorage::unmute()
+{
+ --m_mutedCount;
+}
+
+size_t ConsoleMessageStorage::size() const
+{
+ return m_messages.size();
+}
+
+ConsoleMessage* ConsoleMessageStorage::at(size_t index) const
+{
+ return m_messages[index].get();
+}
+
+int ConsoleMessageStorage::expiredCount() const
+{
+ return m_expiredCount;
+}
+
+bool ConsoleMessageStorage::isMuted() const
+{
+ return m_mutedCount;
+}
+
+DEFINE_TRACE(ConsoleMessageStorage)
+{
+ visitor->trace(m_messages);
+}
+
+} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698