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

Unified Diff: LayoutTests/http/tests/inspector/console-test.js

Issue 23530041: DevTools: Make VM script names less noisy. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebaselined Created 7 years, 3 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: LayoutTests/http/tests/inspector/console-test.js
diff --git a/LayoutTests/http/tests/inspector/console-test.js b/LayoutTests/http/tests/inspector/console-test.js
index f372c644788bbd0bd0b330793b85d17e829e7006..4045891389111b41e295ed1c117ad0db73af34c3 100644
--- a/LayoutTests/http/tests/inspector/console-test.js
+++ b/LayoutTests/http/tests/inspector/console-test.js
@@ -10,9 +10,9 @@ InspectorTest.prepareConsoleMessageText = function(messageElement)
{
var messageText = messageElement.textContent.replace(/\u200b/g, "");
// Replace scriptIds with generic scriptId string to avoid flakiness.
- messageText = messageText.replace(/\[VM\]\s+\(\d+\)/g, "[VM] (scriptId)");
+ messageText = messageText.replace(/VM\d+/g, "VM");
// Strip out InjectedScript from stack traces to avoid rebaselining each time InjectedScriptSource is edited.
- messageText = messageText.replace(/InjectedScript[\.a-zA-Z_]+ \[VM\] \(scriptId\):\d+/g, "");
+ messageText = messageText.replace(/InjectedScript[\.a-zA-Z_]+ VM:\d+/g, "");
// The message might be extremely long in case of dumping stack overflow message.
messageText = messageText.substring(0, 1024);
return messageText;

Powered by Google App Engine
This is Rietveld 408576698