Build is broken: webkit_tests virtual/threaded/fast/compositorworker/visual-update.html https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=virtual%2Fthreaded%2Ffast%2Fcompositorworker%2Fvisual-update.html&testType=webkit_tests Revision range: chromium 402959 : 402999 Failing builders: WebKit Linux (dbg): https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20(dbg)
Time out happened in this build: https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20%28dbg%29/builds/8196
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ba9a8fbcce17e50e5006a008041301b934cde6f0 commit ba9a8fbcce17e50e5006a008041301b934cde6f0 Author: majidvp <majidvp@chromium.org> Date: Tue Jul 05 19:54:29 2016 [compositorworker] Avoid deadlock issue caused by importScript importScripts is a sync API that runs on main thread. Using it on CW will block the compositor thread on main which can lead to deallock in the current Blink compositing architecture. So we should avoid it for now. TODO: - Find a way to throw if importScripts is used in CW. Current failure is subtle - Provide and alternative to synchronous importScripts. Note that async resource loading in already possible in workers thanks to fetch so we may be a able to leverage that. BUG= 624651 TEST=virtual/threaded/fast/compositorworker Review-Url: https://codereview.chromium.org/2119383002 Cr-Commit-Position: refs/heads/master@{#403819} [modify] https://crrev.com/ba9a8fbcce17e50e5006a008041301b934cde6f0/third_party/WebKit/LayoutTests/virtual/threaded/fast/compositorworker/resources/basic-plumbing-main-to-worker.js [modify] https://crrev.com/ba9a8fbcce17e50e5006a008041301b934cde6f0/third_party/WebKit/LayoutTests/virtual/threaded/fast/compositorworker/resources/basic-plumbing-worker-to-main.js [modify] https://crrev.com/ba9a8fbcce17e50e5006a008041301b934cde6f0/third_party/WebKit/LayoutTests/virtual/threaded/fast/compositorworker/resources/proxy-disconnect.js [modify] https://crrev.com/ba9a8fbcce17e50e5006a008041301b934cde6f0/third_party/WebKit/LayoutTests/virtual/threaded/fast/compositorworker/resources/proxy-mutate.js [modify] https://crrev.com/ba9a8fbcce17e50e5006a008041301b934cde6f0/third_party/WebKit/LayoutTests/virtual/threaded/fast/compositorworker/resources/visual-update.js [delete] https://crrev.com/3528705e8b93d3a070f1d31e2dd355db1c33f52c/third_party/WebKit/LayoutTests/virtual/threaded/fast/compositorworker/resources/worker-common.js
Issue 624023 has been merged into this issue.
Issue 624712 has been merged into this issue.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/615718afbfe78126fbf0ffe7b05e524a2fb254d8 commit 615718afbfe78126fbf0ffe7b05e524a2fb254d8 Author: majidvp <majidvp@chromium.org> Date: Fri Jul 08 20:19:06 2016 [compositorworker] stop scheduling rAF in layout tests once promise is resolved BUG= 624651 Review-Url: https://codereview.chromium.org/2124143004 Cr-Commit-Position: refs/heads/master@{#404479} [modify] https://crrev.com/615718afbfe78126fbf0ffe7b05e524a2fb254d8/third_party/WebKit/LayoutTests/virtual/threaded/fast/compositorworker/resources/basic-plumbing-main-to-worker.js [modify] https://crrev.com/615718afbfe78126fbf0ffe7b05e524a2fb254d8/third_party/WebKit/LayoutTests/virtual/threaded/fast/compositorworker/resources/basic-plumbing-worker-to-main.js [modify] https://crrev.com/615718afbfe78126fbf0ffe7b05e524a2fb254d8/third_party/WebKit/LayoutTests/virtual/threaded/fast/compositorworker/resources/proxy-disconnect.js [modify] https://crrev.com/615718afbfe78126fbf0ffe7b05e524a2fb254d8/third_party/WebKit/LayoutTests/virtual/threaded/fast/compositorworker/resources/proxy-mutate.js [modify] https://crrev.com/615718afbfe78126fbf0ffe7b05e524a2fb254d8/third_party/WebKit/LayoutTests/virtual/threaded/fast/compositorworker/resources/visual-update.js
Comment 1 by horo@chromium.org
, Jun 30 2016