Encapsulate SceneHistory. A little. (#1016)

src/components/App.tsx

@@ -2432,10 +2432,7 @@ export class App extends React.Component<any, AppState> {
       this.broadcastSceneUpdate();
     }
 
-    if (history.isRecording()) {
+    history.record(this.state, globalSceneState.getAllElements());
-      history.pushEntry(this.state, globalSceneState.getAllElements());
-      history.skipRecording();
-    }
   }
 }
 

src/history.ts

@@ -123,17 +123,17 @@ export class SceneHistory {
     return null;
   }
 
-  isRecording() {
+  // Suspicious that this is called so many places. Seems error-prone.
-    return this.recording;
-  }
-
-  skipRecording() {
-    this.recording = false;
-  }
-
   resumeRecording() {
     this.recording = true;
   }
+
+  record(state: AppState, elements: readonly ExcalidrawElement[]) {
+    if (this.recording) {
+      this.pushEntry(state, elements);
+      this.recording = false;
+    }
+  }
 }
 
 export const createHistory: () => { history: SceneHistory } = () => {