The reformat API locking semantics are challenging to understand,
efectiveky browser_window_reformat should never be called by frontend
code directly except explicitly on the reformat callback. Otherwise
the content may already be locked and the system winds up asserting.
This changes the LOG macro to be varadic removing the need for all
callsites to have double bracketing and allows for future improvement
on how we use the logging macros.
The callsites were changed with coccinelle and the changes checked by
hand. Compile tested for several frontends but not all.
A formatting annotation has also been added which allows the compiler
to check the parameters and types passed to the logging.
This splits up a great deal of the win32 window code out from other
gui code. It also remove large quantities of unused and junk
variables and functions.
This cleans up this header and moves the functionality into more
useful places while reducing the include complexity but only pulling
in whats required.