Always use the asynchronous codepaths in core.
(LP: #862430) (LP: #892012) (LP: #923683) (LP: #932520)

 * CompWindow::move is now just a wrapper around CompWindow::configureXWindow
 * CompWindow::configureXWindow will always call through to moveNotify
 * moveNotify and resizeNotify are only ever called pre-request in the case of
   managed windows (SubstructureRedirectMask guaruntees they will end up in
   the right place) and post ConfigureNotify for override redirect windows
 * resizeNotify is always called post ConfigureNotify regardless of whether
   or not the window is managed or not
 * various const correctness where it made sense
 * cleaned up the whole geometry.width () + geometry.border () * 2 nonsense
 * removed priv->width and priv->height , which was just redundant data used
   for window shading and honestly, is a ticking time bomb for future
   maintainers.
 * CompWindow::syncPosition is now deprecated and a no-op
 * Removed the majority of PrivateWindow::updateFrameWindow - this function
   had a lot of redundant code which PrivateWindow::reconfigureXWindow had
   anyways - the big work there was to send synthetic ConfigureNotify events
   to ourselves to notify windows moving within frame windows that don't move
   themselves. Its safer to use reconfigureXWindow in this case and probably
   more sane too.  We should look into removing the other update* functions.