PB-265: Video background for Banner

- fix CR comments

Author: omiroshnichenko

app/code/Magento/PageBuilder/view/adminhtml/web/js/content-type/banner/preview.js

@@ -466,7 +466,7 @@ export default class Preview extends BasePreview {
             nestingLinkDialog(this.contentType.dataStore, this.wysiwyg, "message", "link_url");
         });
         this.contentType.dataStore.subscribe(function(data: DataObject) {
-            if (this.isVideoShouldBeUpdated(data)) {
+            if (this.shouldUpdateVideo(data)) {
                 this.buildJarallax();
             }
         }.bind(this));
@@ -492,12 +492,12 @@ export default class Preview extends BasePreview {
     }
 
     /**
-     * Adjust textarea's height based on scrollHeight
+     * Check if video options has been updated.
      *
      * @return boolean
      */
-    private isVideoShouldBeUpdated(state: DataObject): boolean {
-        const previousState = this.contentType.dataStore.previousState;
+    private shouldUpdateVideo(state: DataObject): boolean {
+        const previousState = this.contentType.dataStore.getPreviousState();
         const diff = mageUtils.compare(previousState, state).changes;
 
         if (diff.length > 0) {

app/code/Magento/PageBuilder/view/adminhtml/web/js/data-store.js

@@ -15,9 +15,9 @@ export interface DataObject<T = any> {
 }
 
 export default class DataStore {
-    public previousState: DataObject = {};
     private state: DataObject = {};
     private events: JQuery = $({});
+    private previousState: DataObject = {};
 
     /**
      * Retrieve specific data from the data store
@@ -39,6 +39,15 @@ export default class DataStore {
         return this.state;
     }
 
+    /**
+     * Retrieve the entire previous state of the data object
+     *
+     * @returns {DataObject}
+     */
+    public getPreviousState() {
+        return this.previousState;
+    }
+
     /**
      * Set a specific keys value in the data store
      *