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

Unified Diff: third_party/WebKit/Source/devtools/front_end/console/ConsoleViewMessage.js

Issue 2651843003: DevTools: update console timestamp style (Closed)
Patch Set: nochange Created 3 years, 11 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/devtools/front_end/console/ConsoleViewMessage.js
diff --git a/third_party/WebKit/Source/devtools/front_end/console/ConsoleViewMessage.js b/third_party/WebKit/Source/devtools/front_end/console/ConsoleViewMessage.js
index d62855f8b2de19b3f4545250da11bf685ea1ef8a..3215d2ac1e9de6663046caa125b4dcb3baf8e728 100644
--- a/third_party/WebKit/Source/devtools/front_end/console/ConsoleViewMessage.js
+++ b/third_party/WebKit/Source/devtools/front_end/console/ConsoleViewMessage.js
@@ -817,23 +817,47 @@ Console.ConsoleViewMessage = class {
return regexObject.test(text);
}
- /**
- * @param {boolean} show
- */
- updateTimestamp(show) {
+ updateTimestamp() {
if (!this._contentElement)
return;
- if (show && !this.timestampElement) {
- this.timestampElement = createElementWithClass('span', 'console-timestamp');
- this.timestampElement.textContent = (new Date(this._message.timestamp)).toConsoleTime() + ' ';
- this._contentElement.insertBefore(this.timestampElement, this._contentElement.firstChild);
- return;
+ var format = Common.moduleSetting('consoleTimestampFormat').get();
+ if (format !== Console.ConsoleViewMessage.TimestampFormat.None) {
+ var timestampText = formatTimestamp(this._message.timestamp, format);
+ if (!this._timestampElement)
+ this._timestampElement = createElementWithClass('span', 'console-timestamp');
+ this._timestampElement.textContent = timestampText + ' ';
+ this._timestampElement.title = timestampText;
+ this._contentElement.insertBefore(this._timestampElement, this._contentElement.firstChild);
+ } else if (this._timestampElement) {
+ this._timestampElement.remove();
+ delete this._timestampElement;
}
- if (!show && this.timestampElement) {
- this.timestampElement.remove();
- delete this.timestampElement;
+ /**
+ * @param {number} timestamp
+ * @param {!Console.ConsoleViewMessage.TimestampFormat} format
+ * @return {string}
+ */
+ function formatTimestamp(timestamp, format) {
+ var date = new Date(timestamp);
+ var yymmdd = date.getFullYear() + '-' + leadZero(date.getMonth() + 1, 2) + '-' + leadZero(date.getDate(), 2);
+ var hhmmssfff = leadZero(date.getHours(), 2) + ':' + leadZero(date.getMinutes(), 2) + ':' +
+ leadZero(date.getSeconds(), 2) + '.' + leadZero(date.getMilliseconds(), 3);
+ if (format === Console.ConsoleViewMessage.TimestampFormat.Full)
+ return yymmdd + ' ' + hhmmssfff;
+ return hhmmssfff;
+
+ /**
+ * @param {number} value
+ * @param {number} length
+ * @return {string}
+ */
+ function leadZero(value, length) {
+ var valueString = value.toString();
+ var padding = length - valueString.length;
+ return padding <= 0 ? valueString : '0'.repeat(padding) + valueString;
+ }
}
}
@@ -896,7 +920,7 @@ Console.ConsoleViewMessage = class {
formattedMessage = this._buildMessage(consoleMessage);
contentElement.appendChild(formattedMessage);
- this.updateTimestamp(Common.moduleSetting('consoleTimestampsEnabled').get());
+ this.updateTimestamp();
return this._contentElement;
}
@@ -987,6 +1011,8 @@ Console.ConsoleViewMessage = class {
return;
this._repeatCountElement.remove();
+ if (this._contentElement)
+ this._contentElement.classList.remove('repeated-message');
delete this._repeatCountElement;
}
@@ -1173,6 +1199,15 @@ Console.ConsoleViewMessage = class {
};
/**
+ * @enum {string}
+ */
+Console.ConsoleViewMessage.TimestampFormat = {
+ None: 'none',
+ Full: 'full',
+ Short: 'short'
+};
+
+/**
* @unrestricted
*/
Console.ConsoleGroupViewMessage = class extends Console.ConsoleViewMessage {

Powered by Google App Engine
This is Rietveld 408576698