DevTools: Fix flaky test

Call console.log in an html file instead of calling it directly using evaluateInPage. Bug: 1165324 Change-Id: I3198139bd867b15f4f28724ded7d1767647b3197 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2622282 Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Reviewed-by: Simon Zünd <szuend@chromium.org> Cr-Commit-Position: refs/heads/master@{#842515}

DevTools: Fix flaky test
Call console.log in an html file instead of calling it directly using evaluateInPage. Bug: 1165324 Change-Id: I3198139bd867b15f4f28724ded7d1767647b3197 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2622282 Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Reviewed-by: Simon Zünd <szuend@chromium.org> Cr-Commit-Position: refs/heads/master@{#842515}
756c4e77 · Adithya Srinivasan · Chromium LUCI CQ · 11e256cd · 756c4e77 · 756c4e77
Commit 756c4e77 authored Jan 12, 2021 by Adithya Srinivasan Committed by Chromium LUCI CQ Jan 12, 2021
4 changed files
--- a/third_party/blink/web_tests/TestExpectations
+++ b/third_party/blink/web_tests/TestExpectations
@@ -2298,8 +2298,6 @@ crbug.com/1029514 external/wpt/html/semantics/embedded-content/the-video-element
 # css-pseudo-4 opacity not applied to ::first-line
 crbug.com/1085772 external/wpt/css/css-pseudo/first-line-opacity-001.html [ Failure ]

-crbug.com/1165324 http/tests/devtools/console/console-preserve-log-x-process-navigation.js [ Pass Failure ]
-
 # motion-1 issues
 crbug.com/641245 external/wpt/css/motion/offset-path-ray-002.html [ Failure ]
 crbug.com/641245 external/wpt/css/motion/offset-path-ray-003.html [ Failure ]

--- a/third_party/blink/web_tests/http/tests/devtools/console/console-preserve-log-x-process-navigation-expected.txt
+++ b/third_party/blink/web_tests/http/tests/devtools/console/console-preserve-log-x-process-navigation-expected.txt
 Tests that the console can preserve log messages across cross-process navigations.
-console-preserve-log…ss-navigation.js:11 before navigation
-Navigated to empty.html
-console-preserve-log…ss-navigation.js:13 after navigation
+log-message.html:5 before navigation
+Navigated to log-message.html
+log-message.html:5 after navigation

--- a/third_party/blink/web_tests/http/tests/devtools/console/console-preserve-log-x-process-navigation.js
+++ b/third_party/blink/web_tests/http/tests/devtools/console/console-preserve-log-x-process-navigation.js
@@ -6,11 +6,11 @@
  TestRunner.addResult(`Tests that the console can preserve log messages across cross-process navigations.`);
  await TestRunner.loadModule('console_test_runner');
  await TestRunner.showPanel('console');
-  await TestRunner.navigatePromise('http://devtools.oopif.test:8000/devtools/oopif/resources/empty.html')
+  await TestRunner.navigatePromise('http://devtools.oopif.test:8000/devtools/console/resources/log-message.html')
  Common.settingForTest('preserveConsoleLog').set(true);
-  await TestRunner.evaluateInPage(`console.log('before navigation')`);
-  await TestRunner.navigatePromise('http://127.0.0.1:8000/devtools/oopif/resources/empty.html')
-  await TestRunner.evaluateInPage(`console.log('after navigation')`);
+  await TestRunner.evaluateInPage(`logMessage('before navigation')`);
+  await TestRunner.navigatePromise('http://127.0.0.1:8000/devtools/console/resources/log-message.html')
+  await TestRunner.evaluateInPage(`logMessage('after navigation')`);
  await ConsoleTestRunner.dumpConsoleMessages();
  TestRunner.completeTest();
 })();
--- a/third_party/blink/web_tests/http/tests/devtools/console/resources/log-message.html
+++ b/third_party/blink/web_tests/http/tests/devtools/console/resources/log-message.html
+<!DOCTYPE html>
+<body>
+  <script>
+    function logMessage(message) {
+      console.log(message);
+    }
+  </script>
+</body>