OpenNeo/impress-2020
OpenNeo/impress-2020
1
0
Fork 0
Code Pull requests Activity

maybe fix races in movie script load order?

Browse source
This commit is contained in:
Emi Matchu 2020-09-22 05:12:59 -07:00
parent d50d1ecacc
commit 30ec18e85e
1 changed files with 19 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
src/app/components/OutfitCanvas.js
View file

@ -375,12 +375,26 @@ async function loadMovieLibrary(librarySrc) {
// like, get by a name or ID that we know by this point. So, here we go, just // like, get by a name or ID that we know by this point. So, here we go, just
// try to grab it once it arrives! // try to grab it once it arrives!
// //
// TODO: How reliable is the timing on this? My assumption is that, the // I'm not _sure_ this method is reliable, but it seems to be stable so far
// scripts will trigger their onloads in order of arrival, and my // in Firefox for me. The things I think I'm observing are:
// _hope_ is that the onload will execute before the next script to // - Script execution order should match insert order,
// arrive executes. Let's, ah, find out! // - Onload execution order should match insert order,
// - BUT, script executions might be batched before onloads.
// - So, each script grabs the _first_ composition from the list, and
// deletes it after grabbing. That way, it serves as a FIFO queue!
// I'm not suuure this is happening as I'm expecting, vs I'm just not seeing
// the race anymore? But fingers crossed!
await loadScriptTag(safeImageUrl(librarySrc)); await loadScriptTag(safeImageUrl(librarySrc));
const composition = Object.values(window.AdobeAn.compositions).pop(); const [compositionId, composition] = Object.entries(
window.AdobeAn.compositions
)[0];
if (Object.keys(window.AdobeAn.compositions).length > 1) {
console.warn(
`Grabbing composition ${compositionId}, but there are >1 here: `,
Object.keys(window.AdobeAn.compositions).length
);
}
delete window.AdobeAn.compositions[compositionId];
const library = composition.getLibrary(); const library = composition.getLibrary();
// One more loading step as part of loading this library is loading the // One more loading step as part of loading this library is loading the