Revert #788 – Interface State Runloop Queue

[Adlai-Holler]

Unfortunately, diff #788 has a few issues that need to be resolved before we can re-land it.

• When the run loop queue is disabled, the unit tests need to pass (Interface state tests fail when transaction queue is disabled #853). It appears that the diff affected the normal behavior of the framework, and Pinterest and other users ([ASPagerNode] Layout bug introduced by 2618c5 #788 #845) have seen issues when the feature turned on.
• The experimental feature should be opt-in, not opt-out. We now have ASConfiguration to make this easy.

This reverts the diff until we can get these issues resolved.

[ghost]

	1 Warning
⚠️	This is a big PR, please consider splitting it up to ease code review.

Generated by 🚫 Danger

[appleguy]

@Adlai-Holler thanks for putting this up — it sounds like there is a clear need to disable this by default.

To that end, I've asked my team to make this a P0 and we'll either submit a PR today that accomplishes this (with all the coalescing code bypassed by default, and original timing unchanged) — or alternatively we can land this revert if the path to disable has any complications.

I'll let you guys know within a few hours what we plan to do. Sorry to hear this has created some issues!

[Adlai-Holler]

OK @appleguy that's super reasonable IMO. To disable the coalescing by default, modify the + (ASConfiguration *)defaultConfiguration method in ASConfigurationInternal.m and remove the Coalescing flag there.

[Adlai-Holler]

Superseded by #862

[appleguy]

Thanks for being on top of this - glad we could resolve quickly with a minimum of risk on the other PR.