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

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

Issue 2903083002: DevTools: shorten console links to 40 chars (Closed)
Patch Set: Created 3 years, 7 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 e1e45a8719ab74cad140ff119d1a9c3c5e46e55b..368c7cc71b5ab5e6c2e25724f36c60b51cd669e9 100644
--- a/third_party/WebKit/Source/devtools/front_end/console/ConsoleViewMessage.js
+++ b/third_party/WebKit/Source/devtools/front_end/console/ConsoleViewMessage.js
@@ -284,7 +284,9 @@ Console.ConsoleViewMessage = class {
anchorElement = this._linkifyLocation(this._message.url, this._message.line, this._message.column);
}
} else if (this._message.url) {
- anchorElement = Components.Linkifier.linkifyURL(this._message.url, undefined);
+ anchorElement = Components.Linkifier.linkifyURL(
+ this._message.url, undefined, undefined, undefined, undefined, undefined,
+ Console.ConsoleViewMessage.MaxLengthForLinks);
}
// Append a space to prevent the anchor text from being glued to the console message when the user selects and copies the console messages.
@@ -1276,3 +1278,9 @@ Console.ConsoleGroupViewMessage = class extends Console.ConsoleViewMessage {
return this._element;
}
};
+
+/**
+ * @const
+ * @type {number}
+ */
+Console.ConsoleViewMessage.MaxLengthForLinks = 40;

Powered by Google App Engine
This is Rietveld 408576698