More specialized typing, more DRY code

Author: davidbrochart

packages/base-manager/src/manager-base.ts

@@ -157,14 +157,7 @@ export abstract class ManagerBase implements IWidgetManager {
     }
     const data = (msg.content.data as unknown) as ISerializedState;
     const buffer_paths = data.buffer_paths || [];
-    // Make sure the buffers are DataViews
-    const buffers = (msg.buffers || []).map(b => {
-      if (b instanceof DataView) {
-        return b;
-      } else {
-        return new DataView(b instanceof ArrayBuffer ? b : b.buffer);
-      }
-    });
+    const buffers = msg.buffers || [];
     put_buffers(data.state, buffer_paths, buffers);
     return this.new_model(
       {

packages/base/src/utils.ts

@@ -100,7 +100,12 @@ export function reject(message: string, log: boolean) {
 export function put_buffers(
   state: Dict<BufferJSON>,
   buffer_paths: (string | number)[][],
-  buffers: any[]
+  buffers: (
+    | DataView
+    | ArrayBuffer
+    | ArrayBufferView
+    | { buffer: ArrayBuffer }
+  )[]
 ): void {
   for (let i = 0; i < buffer_paths.length; i++) {
     const buffer_path = buffer_paths[i];

packages/base/src/widget.ts

@@ -216,15 +216,7 @@ export class WidgetModel extends Backbone.Model {
           .then(() => {
             const state = data.state;
             const buffer_paths = data.buffer_paths || [];
-            // Make sure the buffers are DataViews
-            const buffers = (msg.buffers || []).map(b => {
-              if (b instanceof DataView) {
-                return b;
-              } else {
-                return new DataView(b instanceof ArrayBuffer ? b : b.buffer);
-              }
-            });
-
+            const buffers = msg.buffers || [];
             utils.put_buffers(state, buffer_paths, buffers);
             return (this.constructor as typeof WidgetModel)._deserialize_state(
               state,