Summary: | wasm/function-tests/memory-multiagent times out sometimes | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Filip Pizlo <fpizlo> | ||||
Component: | JavaScriptCore | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | NEW --- | ||||||
Severity: | Normal | CC: | jfbastien, keith_miller | ||||
Priority: | P2 | ||||||
Version: | WebKit Nightly Build | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Filip Pizlo
2017-04-18 11:41:33 PDT
Created attachment 307399 [details]
patch to skip the test for now
Comment on attachment 307399 [details]
patch to skip the test for now
r=me.
Comment on attachment 307399 [details] patch to skip the test for now View in context: https://bugs.webkit.org/attachment.cgi?id=307399&action=review > JSTests/wasm/function-tests/memory-multiagent.js:51 > + // prevents any JS-heap-modifying things from happening because JS is a synchrinous typo: synchronous (In reply to Keith Miller from comment #3) > Comment on attachment 307399 [details] > patch to skip the test for now > > View in context: > https://bugs.webkit.org/attachment.cgi?id=307399&action=review > > > JSTests/wasm/function-tests/memory-multiagent.js:51 > > + // prevents any JS-heap-modifying things from happening because JS is a synchrinous > > typo: synchronous Fixed! Comment on attachment 307399 [details] patch to skip the test for now View in context: https://bugs.webkit.org/attachment.cgi?id=307399&action=review > JSTests/wasm/function-tests/memory-multiagent.js:53 > + // https://bugs.webkit.org/show_bug.cgi?id=170958 My expectation was that receiveBroadcast could change the value of `state`. Is that not the case? (In reply to JF Bastien from comment #6) > Comment on attachment 307399 [details] > patch to skip the test for now > > View in context: > https://bugs.webkit.org/attachment.cgi?id=307399&action=review > > > JSTests/wasm/function-tests/memory-multiagent.js:53 > > + // https://bugs.webkit.org/show_bug.cgi?id=170958 > > My expectation was that receiveBroadcast could change the value of `state`. > Is that not the case? It could. But how would it run? JS is runloop-based. You're doing a while loop. So the runloop doesn't run. (In reply to Filip Pizlo from comment #7) > (In reply to JF Bastien from comment #6) > > Comment on attachment 307399 [details] > > patch to skip the test for now > > > > View in context: > > https://bugs.webkit.org/attachment.cgi?id=307399&action=review > > > > > JSTests/wasm/function-tests/memory-multiagent.js:53 > > > + // https://bugs.webkit.org/show_bug.cgi?id=170958 > > > > My expectation was that receiveBroadcast could change the value of `state`. > > Is that not the case? > > It could. But how would it run? JS is runloop-based. You're doing a while > loop. So the runloop doesn't run. Seemed to work in the jsc shell. I'll do a follow-up soon. |