MS Edge compatibility issue with processQueue() about freezer HOT 6 CLOSED

@arqex That seems to have done the trick for the case we were seeing. Thanks for the quick turnaround here!

from freezer.

I work on the Microsoft Edge team and came here to report this same issue. We have a bug logged on our side to track the investigation of this, but it would be great if a workaround could be implemented in Freezer for the time being.

from freezer.

Hey guys sorry for the delay, I've been out for some days.

This issue is interesting. The message passing here is used because in browsers we don't have a simple way of executing things on the next JS execution loop. By default freezer doesn't emit update events immediately, it allow us to make multiple changes to its data and the event is emmited in the "next tick".

Using timeouts to delay that event triggering is too slow (a timeout of 0ms is not really executing the function on the next tick), so the alternative was using the browser's message system: Freezer emits a message nexttick to the current window, and listen for that message that arrives on the next tick.

If the message is not coming from the same window (event.source === global) don't emit the event. If we remove that condition, we run the risk of triggering events by messages produced by other freezer instances inside of iframes or browser's plugins.

If we move the flushQueue call outside the if block, we are just flushing on every browser message, and that is very often (especially in browsers with many plugins) and may lead to unexpected behaviours.

The right solution would be creating an id for every freezer instance, and check if the message comes from ourselves before flushing the queue. That way we can skip the current window check.

I'll have a look at the problem this evening, but it would be great to fix the bug at MS Edge too.

Thanks for your feedback guys!

from freezer.

I have just published a new version that it's not using event.source anymore to check what's the origin of the message. Let me know if it fixed your issue.

from freezer.

I was finally able to test this today. It works for me. Thanks for the quick turnaround!

from freezer.

@arqex Could you make this an official release? It would make it easier for our npm config to pick it up.

from freezer.