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

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

Issue 2571713005: DevTools: untruncate links on console export (Closed)
Patch Set: test desc Created 4 years 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 73aedc4e179d8c807b396adce17e0a2844e34dd4..c1010ff1cf3a6b09bb3a729efd9780d3bd8ca5f3 100644
--- a/third_party/WebKit/Source/devtools/front_end/console/ConsoleViewMessage.js
+++ b/third_party/WebKit/Source/devtools/front_end/console/ConsoleViewMessage.js
@@ -292,13 +292,17 @@ Console.ConsoleViewMessage = class {
anchorElement = this._linkifyLocation(consoleMessage.url, consoleMessage.line, consoleMessage.column);
}
} else if (consoleMessage.url) {
- anchorElement = Components.Linkifier.linkifyURL(consoleMessage.url, undefined, 'console-message-url');
+ anchorElement = Components.Linkifier.linkifyURL(consoleMessage.url, undefined);
}
// Append a space to prevent the anchor text from being glued to the console message when the user selects and copies the console messages.
- if (anchorElement)
- anchorElement.appendChild(createTextNode(' '));
- return anchorElement;
+ if (anchorElement) {
+ var anchorWrapperElement = createElementWithClass('span', 'console-message-anchor');
+ anchorWrapperElement.appendChild(anchorElement);
+ anchorWrapperElement.createTextChild(' ');
+ return anchorWrapperElement;
+ }
+ return null;
}
/**
@@ -356,7 +360,7 @@ Console.ConsoleViewMessage = class {
var target = this._target();
if (!target)
return null;
- return this._linkifier.linkifyScriptLocation(target, null, url, lineNumber, columnNumber, 'console-message-url');
+ return this._linkifier.linkifyScriptLocation(target, null, url, lineNumber, columnNumber);
}
/**
@@ -367,7 +371,7 @@ Console.ConsoleViewMessage = class {
var target = this._target();
if (!target)
return null;
- return this._linkifier.linkifyStackTraceTopFrame(target, stackTrace, 'console-message-url');
+ return this._linkifier.linkifyStackTraceTopFrame(target, stackTrace);
}
/**
@@ -381,8 +385,7 @@ Console.ConsoleViewMessage = class {
var target = this._target();
if (!target)
return null;
- return this._linkifier.linkifyScriptLocation(
- target, scriptId, url, lineNumber, columnNumber, 'console-message-url');
+ return this._linkifier.linkifyScriptLocation(target, scriptId, url, lineNumber, columnNumber);
}
/**
@@ -1091,6 +1094,23 @@ Console.ConsoleViewMessage = class {
}
/**
+ * @return {string}
+ */
+ toExportString() {
+ var lines = [];
+ var messageContent = this.contentElement()
dgozman 2017/01/10 01:03:30 Let's reformat somehow...
luoe 2017/01/10 22:21:44 Done.
+ .childTextNodes()
+ .map(node => {
dgozman 2017/01/10 01:03:30 var nodes = ...; for (var index = 0; index < nodes
luoe 2017/01/10 22:21:44 Done.
+ var originalLinkText = Components.Linkifier.originalLinkText(node.parentElement);
+ return typeof originalLinkText === 'string' ? originalLinkText : node.textContent;
+ })
+ .join('');
+ for (var i = 0; i < this.repeatCount(); ++i)
+ lines.push(messageContent);
+ return lines.join('\n');
+ }
+
+ /**
* @param {?RegExp} regex
*/
setSearchRegex(regex) {

Powered by Google App Engine
This is Rietveld 408576698