Findit identified the culprit r614119 as introducing flaky test(s) summarized in https://findit-for-me.appspot.com/waterfall/flake/flake-culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyQwsSDEZsYWtlQ3VscHJpdCIxY2hyb21pdW0vZTNjZWRhNzQwYzZjZmQxYjg2YjZjNDMyMDI0MjQ1MDVmZmNlZTQ2Ngw Please revert the culprit, or disable the test(s) and find the appropriate owner to fix or delete. If the culprit above is wrong, please file a bug using this link: https://bugs.chromium.org/p/chromium/issues/entry?status=Unconfirmed&labels=Pri-1,Test-Findit-Wrong&components=Tools%3ETest%3EFindit%3EFlakiness&summary=%5BFindit%5D%20Flake%20Analyzer%20-%20Wrong%20culprit%20r614119&comment=Link%20to%20Culprit%3A%20https://findit-for-me.appspot.com/waterfall/flake/flake-culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyQwsSDEZsYWtlQ3VscHJpdCIxY2hyb21pdW0vZTNjZWRhNzQwYzZjZmQxYjg2YjZjNDMyMDI0MjQ1MDVmZmNlZTQ2Ngw Automatically posted by the findit-for-me app (https://goo.gl/Ot9f7N).
The CL added the test. I think "testRunner.log('Before await. Resolved: ' + resolved);" could be true since there are two awaits before it. I'll tentatively mark the test as flaky. Assigned to the author of the original CL.
Let me paste a few builds where the test fails for reference: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/linux-xenial-rel/5613 https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/WebKit%20Win10/43873 and flakiness dashbaord https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests&tests=inspector-protocol%2Fworker%2Fworker-autoattach-order.js
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2c9f823f2b2296fe6a18195de6f0ade373270ad4 commit 2c9f823f2b2296fe6a18195de6f0ade373270ad4 Author: Makoto Shimazu <shimazu@chromium.org> Date: Mon Dec 17 04:23:47 2018 Mark inspector-protocol/worker/worker-autoattach-order.js as flaky TBR=dgozman@chromium.org No-Try: true Bug: 915472 Change-Id: Icb714ce155704962e8a7b137f74c7b88c2de783f Reviewed-on: https://chromium-review.googlesource.com/c/1379620 Reviewed-by: Makoto Shimazu <shimazu@chromium.org> Commit-Queue: Makoto Shimazu <shimazu@chromium.org> Cr-Commit-Position: refs/heads/master@{#617046} [modify] https://crrev.com/2c9f823f2b2296fe6a18195de6f0ade373270ad4/third_party/blink/web_tests/TestExpectations
Removing from sheriff queue
Comment 1 by shimazu@chromium.org
, Dec 17Owner: dgozman@chromium.org
Status: Assigned (was: Untriaged)
The CL added the test. I think "testRunner.log('Before await. Resolved: ' + resolved);" could be true since there are two awaits before it. I'll tentatively mark the test as flaky. Assigned to the author of the original CL.