From 55ed9e4f62b5add55bc8efbc1ffbd264a44eb322 Mon Sep 17 00:00:00 2001
From: Nathan Sobo <nathan@github.com>
Date: Tue, 28 Feb 2017 20:19:06 -0700
Subject: [PATCH] Pre-assign measuremets.scrollTop when autoscrolling

This avoids work when the scroll event happens asynchronously because
we'll treat the event as a no-op since the measurements didn't change.
---
 src/text-editor-component.js | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/text-editor-component.js b/src/text-editor-component.js
index 1b85025f5..610933556 100644
--- a/src/text-editor-component.js
+++ b/src/text-editor-component.js
@@ -665,6 +665,10 @@ class TextEditorComponent {
         this.autoscrollTop = desiredScrollBottom - this.measurements.clientHeight
       }
     }
+
+    if (this.autoscrollTop != null) {
+      this.measurements.scrollTop = this.autoscrollTop
+    }
   }
 
   getVerticalScrollMargin () {
@@ -849,9 +853,7 @@ class TextEditorComponent {
   }
 
   getScrollTop () {
-    if (this.autoscrollTop != null) {
-      return this.autoscrollTop
-    } else if (this.measurements != null) {
+    if (this.measurements != null) {
       return this.measurements.scrollTop
     }
   }