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

Unified Diff: third_party/WebKit/Source/devtools/front_end/console/ConsoleView.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/ConsoleView.js
diff --git a/third_party/WebKit/Source/devtools/front_end/console/ConsoleView.js b/third_party/WebKit/Source/devtools/front_end/console/ConsoleView.js
index efca992edf38f63c0a8ac82e9f4076a2ab930499..59263a55479ceeb4b8f63754f9b431c125bc2235 100644
--- a/third_party/WebKit/Source/devtools/front_end/console/ConsoleView.js
+++ b/third_party/WebKit/Source/devtools/front_end/console/ConsoleView.js
@@ -144,7 +144,7 @@ Console.ConsoleView = class extends UI.VBox {
this._consoleHistoryAutocompleteChanged();
this._updateFilterStatus();
- Common.moduleSetting('consoleTimestampsEnabled').addChangeListener(this._consoleTimestampsSettingChanged, this);
+ Common.moduleSetting('consoleTimestampFormat').addChangeListener(this._consoleTimestampsSettingChanged, this);
this._registerWithMessageSink();
SDK.targetManager.observeTargets(this);
@@ -321,15 +321,9 @@ Console.ConsoleView = class extends UI.VBox {
this._addConsoleMessage(consoleMessage);
}
- /**
- * @param {!Common.Event} event
- */
- _consoleTimestampsSettingChanged(event) {
- var enabled = /** @type {boolean} */ (event.data);
+ _consoleTimestampsSettingChanged() {
this._updateMessageList();
- this._consoleMessages.forEach(function(viewMessage) {
- viewMessage.updateTimestamp(enabled);
- });
+ this._consoleMessages.forEach(viewMessage => viewMessage.updateTimestamp());
}
_executionContextChanged() {
@@ -684,8 +678,9 @@ Console.ConsoleView = class extends UI.VBox {
* @return {boolean}
*/
_tryToCollapseMessages(lastMessage, viewMessage) {
- if (!Common.moduleSetting('consoleTimestampsEnabled').get() && viewMessage &&
- !lastMessage.consoleMessage().isGroupMessage() &&
+ var timestampFormat = Common.moduleSetting('consoleTimestampFormat').get();
+ var timestampsShown = timestampFormat !== Console.ConsoleViewMessage.TimestampFormat.None;
+ if (!timestampsShown && viewMessage && !lastMessage.consoleMessage().isGroupMessage() &&
lastMessage.consoleMessage().isEqual(viewMessage.consoleMessage())) {
viewMessage.incrementRepeatCount();
return true;
@@ -1206,6 +1201,8 @@ Console.ConsoleCommand = class extends Console.ConsoleViewMessage {
} else {
this._updateSearch();
}
+
+ this.updateTimestamp();
}
return this._contentElement;
}
@@ -1248,7 +1245,6 @@ Console.ConsoleCommandResult = class extends Console.ConsoleViewMessage {
element.insertBefore(icon, element.firstChild);
}
}
- this.updateTimestamp(false);
return element;
}
};

Powered by Google App Engine
This is Rietveld 408576698