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

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

Issue 2270033003: DevTools: include traces when exporting console log via 'Save as' (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Use contentElement instead of formattedMessage Created 4 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: 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 1a77e9750f7711779fb9fda5e39656600c8e6b52..2489bb08886533872b9cba9136520abb36173e11 100644
--- a/third_party/WebKit/Source/devtools/front_end/console/ConsoleView.js
+++ b/third_party/WebKit/Source/devtools/front_end/console/ConsoleView.js
@@ -657,7 +657,7 @@ WebInspector.ConsoleView.prototype = {
var lines = [];
for (var i = 0; i < chunkSize && i + messageIndex < this.itemCount(); ++i) {
var message = this.itemElement(messageIndex + i);
- lines.push(message.searchableElement().deepTextContent());
+ lines.push(message.formattedMessage().deepTextContent());
lushnikov 2016/09/06 19:30:29 why do we still use formatMessage?
luoe 2016/09/06 21:50:29 My bad, I didn't remove formattedMessage() uses in
}
messageIndex += i;
stream.write(lines.join("\n") + "\n", writeNextChunk.bind(this));
@@ -1235,11 +1235,15 @@ WebInspector.ConsoleCommand = function(message, linkifier, nestingLevel)
WebInspector.ConsoleCommand.prototype = {
/**
* @override
- * @return {!Element})
+ * @return {!Element}
*/
- searchableElement: function()
+ formattedMessage: function()
{
- return this.contentElement();
+ if (this._formattedMessage)
+ return this._formattedMessage;
+ this._formattedMessage = createElementWithClass("span", "console-message-text source-code");
+ this._formattedMessage.textContent = this.text.replaceControlCharacters();
+ return this._formattedMessage;
},
/**
@@ -1251,14 +1255,11 @@ WebInspector.ConsoleCommand.prototype = {
if (!this._element) {
this._element = createElementWithClass("div", "console-user-command");
this._element.message = this;
+ this._element.appendChild(this.formattedMessage());
- this._formattedCommand = createElementWithClass("span", "console-message-text source-code");
- this._formattedCommand.textContent = this.text.replaceControlCharacters();
- this._element.appendChild(this._formattedCommand);
-
- if (this._formattedCommand.textContent.length < WebInspector.ConsoleCommand.MaxLengthToIgnoreHighlighter) {
+ if (this._formattedMessage.textContent.length < WebInspector.ConsoleCommand.MaxLengthToIgnoreHighlighter) {
var javascriptSyntaxHighlighter = new WebInspector.DOMSyntaxHighlighter("text/javascript", true);
- javascriptSyntaxHighlighter.syntaxHighlightNode(this._formattedCommand).then(this._updateSearch.bind(this))
+ javascriptSyntaxHighlighter.syntaxHighlightNode(this._formattedMessage).then(this._updateSearch.bind(this))
} else {
this._updateSearch();
}

Powered by Google App Engine
This is Rietveld 408576698