scroll: run panBy sync from drawSections

- reverted code from commit 623e2f5
  "Scroll Section: Move move event into layouting tasks."
  It even mention: This will change in the future but for now, move event modifies map css prpoerty.
- fixes regression after commit 5a672a7
  "canvas: flush LayoutingService before canvas render"
- makes order of processing again good for current frame:
  Original order (draw -> flush):
  1. drawSections() - ScrollSection queues panBy
  2. flushLayoutingTasks() - runs panBy synchronously

  Currently we flush tasks first, then will execute doMove() in
  ScrollSection which appends new task

Signed-off-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I82cb889ea0e87d8fd5b0588e484d19dec77bc63d

Author: caolanm

browser/src/canvas/sections/ScrollSection.ts

@@ -338,16 +338,14 @@ export class ScrollSection extends CanvasSectionObject {
 	private doMove() {
 		const scrollProps: ScrollProperties = (app.activeDocument as DocumentBase).activeLayout.scrollProperties;
 
-		app.layoutingService.appendLayoutingTask(() => {
-			this.map.panBy(new cool.Point(scrollProps.moveBy[0] / app.dpiScale, scrollProps.moveBy[1] / app.dpiScale));
-			scrollProps.moveBy = null;
-			this.onUpdateScrollOffset();
+		this.map.panBy(new cool.Point(scrollProps.moveBy[0] / app.dpiScale, scrollProps.moveBy[1] / app.dpiScale));
+		scrollProps.moveBy = null;
+		this.onUpdateScrollOffset();
 
-			if (app && app.file.fileBasedView === true)
-				app.map._docLayer._checkSelectedPart();
+		if (app && app.file.fileBasedView === true)
+			app.map._docLayer._checkSelectedPart();
 
-			app.activeDocument.activeLayout.refreshScrollProperties();
-		});
+		app.activeDocument.activeLayout.refreshScrollProperties();
 	}
 
 	public onDraw(frameCount: number, elapsedTime: number): void {