Commit 5b2f6be4 authored by Sigurd Schneider's avatar Sigurd Schneider Committed by Commit Bot

Prepare for removing superfluous toMessageElement function

Bug: chromium:1135143
Change-Id: I1646dd57e0a6255d6437a58645737f2a17c9e64a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461014
Commit-Queue: Sigurd Schneider <sigurds@chromium.org>
Reviewed-by: default avatarPeter Marshall <petermarshall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#815528}
parent 60eaec95
...@@ -68,7 +68,7 @@ ...@@ -68,7 +68,7 @@
for (var i = 0; i < messages.length; ++i) { for (var i = 0; i < messages.length; ++i) {
var consoleMessage = messages[i].consoleMessage(); var consoleMessage = messages[i].consoleMessage();
var element = messages[i].toMessageElement(); var element = messages[i].element();
var node = element.traverseNextNode(element); var node = element.traverseNextNode(element);
while (node) { while (node) {
......
...@@ -34,7 +34,7 @@ ...@@ -34,7 +34,7 @@
for (var i = 0; i < messages.length; i++) { for (var i = 0; i < messages.length; i++) {
// Ordering is important here, as accessing the element the first time around // Ordering is important here, as accessing the element the first time around
// triggers live location creation and updates which we need to await properly. // triggers live location creation and updates which we need to await properly.
const element = messages[i].toMessageElement(); const element = messages[i].element();
await TestRunner.waitForPendingLiveLocationUpdates(); await TestRunner.waitForPendingLiveLocationUpdates();
TestRunner.addResult('>' + element.deepTextContent()); TestRunner.addResult('>' + element.deepTextContent());
} }
......
...@@ -70,7 +70,7 @@ ...@@ -70,7 +70,7 @@
// Ordering is important here, as accessing the element the first time around // Ordering is important here, as accessing the element the first time around
// triggers live location creation and updates which we need to await properly. // triggers live location creation and updates which we need to await properly.
const element = viewMessage.toMessageElement(); const element = viewMessage.element();
await TestRunner.waitForPendingLiveLocationUpdates(); await TestRunner.waitForPendingLiveLocationUpdates();
TestRunner.addResult(indent + delimeter + element.deepTextContent()); TestRunner.addResult(indent + delimeter + element.deepTextContent());
} }
......
...@@ -56,7 +56,7 @@ ...@@ -56,7 +56,7 @@
for (var i = 0; i < messages.length; ++i) { for (var i = 0; i < messages.length; ++i) {
// Ordering is important here. Retrieveing the message element the first time triggers // Ordering is important here. Retrieveing the message element the first time triggers
// live location creation and updates, which we need to await for correct locations. // live location creation and updates, which we need to await for correct locations.
var element = messages[i].toMessageElement(); var element = messages[i].element();
await TestRunner.waitForPendingLiveLocationUpdates(); await TestRunner.waitForPendingLiveLocationUpdates();
var anchor = element.querySelector('.console-message-anchor'); var anchor = element.querySelector('.console-message-anchor');
TestRunner.addResult(anchor.textContent.replace(/VM\d+/g, 'VM')); TestRunner.addResult(anchor.textContent.replace(/VM\d+/g, 'VM'));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment