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

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

Issue 464293002: [DevTools] ConsoleMessage storage moved from ConsoleAgent (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@remove-can-generate
Patch Set: Rebased Created 6 years, 4 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: Source/core/inspector/ConsoleMessageStorage.cpp
diff --git a/Source/core/inspector/ConsoleMessageStorage.cpp b/Source/core/inspector/ConsoleMessageStorage.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..9cf3b2e3e47cdee7b901608c29c2305fa51c3f21
--- /dev/null
+++ b/Source/core/inspector/ConsoleMessageStorage.cpp
@@ -0,0 +1,69 @@
+// Copyright 2014 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 "config.h"
+#include "core/inspector/ConsoleMessageStorage.h"
+
+#include "core/frame/LocalDOMWindow.h"
+#include "core/inspector/ConsoleMessage.h"
+
+namespace blink {
+
+static const unsigned maximumConsoleMessages = 1000;
+static const int expireConsoleMessagesStep = 100;
+
+ConsoleMessageStorage::ConsoleMessageStorage()
+ : m_expiredCount(0)
+{
+}
+
+void ConsoleMessageStorage::addMessage(PassRefPtr<ConsoleMessage> message)
+{
+ // TODO: it's not awesome solution, because we always first store message and second - send to frontend
vsevik 2014/08/14 08:06:22 Is there something you plan to address here? If no
kozyatinskiy1 2014/08/20 13:44:18 Done.
+ message->autogenerateMetadata();
+
+ m_messages.append(message);
+
+ if (m_messages.size() >= maximumConsoleMessages) {
+ m_expiredCount += expireConsoleMessagesStep;
+ m_messages.remove(0, expireConsoleMessagesStep);
+ }
+}
+
+void ConsoleMessageStorage::clear()
+{
+ m_messages.clear();
+ m_expiredCount = 0;
+}
+
+Vector<unsigned> ConsoleMessageStorage::argumentCounts()
+{
+ Vector<unsigned> result(m_messages.size());
+ for (size_t i = 0; i < size(); ++i)
+ result[i] = at(i)->argumentCount();
+ return result;
+}
+
+void ConsoleMessageStorage::windowCleared(LocalDOMWindow* window)
+{
+ for (size_t i = 0; i < size(); ++i)
+ at(i)->windowCleared(window);
+}
+
+size_t ConsoleMessageStorage::size() const
+{
+ return m_messages.size();
+}
+
+PassRefPtr<ConsoleMessage> ConsoleMessageStorage::at(size_t index)
+{
+ return m_messages[index];
+}
+
+int ConsoleMessageStorage::expiredCount() const
+{
+ return m_expiredCount;
+}
+
+} // namespace WebCore

Powered by Google App Engine
This is Rietveld 408576698