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

Unified Diff: Source/devtools/front_end/ConsoleModel.js

Issue 185713007: DevTools: Add timestamp support in the console (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Make repeat count work Created 6 years, 10 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/devtools/front_end/ConsoleModel.js
diff --git a/Source/devtools/front_end/ConsoleModel.js b/Source/devtools/front_end/ConsoleModel.js
index a97dcff6092ed219b0f6fe167dae8639dd2846d6..3760056de96fc38ffe473d521f2dff60bf5ef07c 100644
--- a/Source/devtools/front_end/ConsoleModel.js
+++ b/Source/devtools/front_end/ConsoleModel.js
@@ -169,9 +169,10 @@ WebInspector.ConsoleModel.prototype = {
* @param {!NetworkAgent.RequestId=} requestId
* @param {!Array.<!RuntimeAgent.RemoteObject>=} parameters
* @param {!Array.<!ConsoleAgent.CallFrame>=} stackTrace
+ * @param {number=} timestamp
* @param {boolean=} isOutdated
*/
-WebInspector.ConsoleMessage = function(source, level, text, type, url, line, column, repeatCount, requestId, parameters, stackTrace, isOutdated)
+WebInspector.ConsoleMessage = function(source, level, text, type, url, line, column, repeatCount, requestId, parameters, stackTrace, timestamp, isOutdated)
{
this.source = source;
this.level = level;
@@ -182,6 +183,7 @@ WebInspector.ConsoleMessage = function(source, level, text, type, url, line, col
this.column = column || 0;
this.parameters = parameters;
this.stackTrace = stackTrace;
+ this.timestamp = timestamp || Date.now() / 1000;
this.isOutdated = isOutdated;
repeatCount = repeatCount || 1;
@@ -205,7 +207,7 @@ WebInspector.ConsoleMessage.prototype = {
*/
clone: function()
{
- return new WebInspector.ConsoleMessage(this.source, this.level, this.text, this.type, this.url, this.line, this.column, this.repeatCount, this.requestId, this.parameters, this.stackTrace, this.isOutdated);
+ return new WebInspector.ConsoleMessage(this.source, this.level, this.text, this.type, this.url, this.line, this.column, this.repeatCount, this.requestId, this.parameters, this.stackTrace, this.timestamp, this.isOutdated);
},
/**
@@ -233,6 +235,7 @@ WebInspector.ConsoleMessage.prototype = {
}
}
+ // Timestamp doesn't affect equality.
return (this.source === msg.source)
&& (this.type === msg.type)
&& (this.level === msg.level)
@@ -311,6 +314,7 @@ WebInspector.ConsoleDispatcher.prototype = {
payload.networkRequestId,
payload.parameters,
payload.stackTrace,
+ payload.timestamp,
this._console._enablingConsole);
this._console.addMessage(consoleMessage, true);
},

Powered by Google App Engine
This is Rietveld 408576698