no need to re-calc 'regl' traces in-and-out of arrayOk values

Author: etpinard

src/plot_api/plot_api.js

@@ -1572,8 +1572,10 @@ function _restyle(gd, aobj, traces) {
             else {
                 if(valObject) {
                     // must redo calcdata when restyling array values of arrayOk attributes
-                    if(valObject.arrayOk && (
-                        Lib.isArrayOrTypedArray(newVal) || Lib.isArrayOrTypedArray(oldVal))
+                    // ... but no need to this for regl-based traces
+                    if(valObject.arrayOk &&
+                        !Registry.traceIs(contFull, 'regl') &&
+                        (Lib.isArrayOrTypedArray(newVal) || Lib.isArrayOrTypedArray(oldVal))
                     ) {
                         flags.calc = true;
                     }

test/jasmine/tests/splom_test.js

@@ -901,6 +901,27 @@ describe('Test splom update switchboard:', function() {
 
             return Plotly.restyle(gd, 'marker.color', [['red', 'green', 'blue']]);
         })
+        .then(function() {
+            var msg = 'after arrayOk marker.color restyle';
+
+            assertSpies(msg, [
+                ['supplyDefaults', 1],
+                ['doCalcdata', 0],
+                ['doTicks', 0],
+                ['clear', 1],
+                ['update', 1],
+                ['draw', 1]
+            ]);
+
+            expect(toPlainArray(scene.matrixOptions.colors[0]))
+                .toBeCloseToArray([1, 0, 0, 1], 1, msg + '- 0');
+            expect(toPlainArray(scene.matrixOptions.colors[1]))
+                .toBeCloseToArray([0, 0.501, 0, 1], 1, msg + '- 1');
+            expect(toPlainArray(scene.matrixOptions.colors[2]))
+                .toBeCloseToArray([0, 0, 1, 1], 1, msg + '- 2');
+
+            return Plotly.restyle(gd, 'marker.size', 20);
+        })
         .catch(failTest)
         .then(done);
     });