Call update after running commands in the idle handler.

[jneem]

I'm not too sure about the invalidate_and_finalize, but at the very least I can't see why it would make sense for commands but not for ext events.

[cmyr]

do_update calls invalidate_and_finalize. This does have me confused, though.

Thinking this through: most of the time, a command is issued during another event, and so invalidate_and_finalize is going to get called at the end of the parent event.

In certain situations (such as when a command is submitted during layout, in which case we've already called update) the command is queued, and then we schedule an idle task to run the command after paint returns.

Now: is there some reason not to update in this case? I can't think of one.

In this case I think the patch we want, here, is one that just replaces invalidate_and_finalize with do_update when handling RUN_COMMANDS_TOKEN.

[cmyr]

thanks!