run Axes.expand on autorange: false axes

- this way `relayout(gd, 'ax.autorange', true)` does not have
  to go through costly editType "calc"
- this makes the 'calcIfAutorange' edit type obsolete,
  which the following few commits will cope with

Author: etpinard

src/plots/cartesian/autorange.js

@@ -210,10 +210,6 @@ function doAutoRange(ax) {
     }
 }
 
-function needsAutorange(ax) {
-    return ax.autorange || ax._rangesliderAutorange;
-}
-
 /*
  * expand: if autoranging, include new data in the outer limits for this axis.
  * Note that `expand` is called during `calc`, when we don't yet know the axis
@@ -236,8 +232,6 @@ function needsAutorange(ax) {
  *          and make it a tight bound if possible
  */
 function expand(ax, data, options) {
-    if(!needsAutorange(ax) || !data) return;
-
     if(!ax._min) ax._min = [];
     if(!ax._max) ax._max = [];
     if(!options) options = {};

src/plots/cartesian/layout_attributes.js

@@ -78,7 +78,7 @@ module.exports = {
         values: [true, false, 'reversed'],
         dflt: true,
         role: 'info',
-        editType: 'calc',
+        editType: 'plot',
         impliedEdits: {'range[0]': undefined, 'range[1]': undefined},
         description: [
             'Determines whether or not the range of this axis is',

test/jasmine/tests/axes_test.js

@@ -1858,24 +1858,21 @@ describe('Test axes', function() {
             expect(ax._max).toEqual([{val: 6, pad: 10, extrapad: true}]);
         });
 
-        it('should return early if no data is given', function() {
+        it('should fail if no data is given', function() {
             ax = getDefaultAx();
-
-            expand(ax);
-            expect(ax._min).toBeUndefined();
-            expect(ax._max).toBeUndefined();
+            expect(function() { expand(ax); }).toThrow();
         });
 
-        it('should return early if `autorange` is falsy', function() {
+        it('should return even if `autorange` is false', function() {
             ax = getDefaultAx();
             data = [2, 5];
 
             ax.autorange = false;
             ax.rangeslider = { autorange: false };
 
             expand(ax, data, {});
-            expect(ax._min).toBeUndefined();
-            expect(ax._max).toBeUndefined();
+            expect(ax._min).toEqual([{val: 2, pad: 0, extrapad: false}]);
+            expect(ax._max).toEqual([{val: 5, pad: 0, extrapad: false}]);
         });
 
         it('should consider range slider `autorange`', function() {

test/jasmine/tests/gl2d_pointcloud_test.js

@@ -181,21 +181,18 @@ describe('@gl pointcloud traces', function() {
 
             return Plotly.relayout(gd, 'xaxis.range', [3, 6]);
         }).then(function() {
-
-            expect(scene2d.xaxis._min).toEqual([]);
-            expect(scene2d.xaxis._max).toEqual([]);
+            expect(scene2d.xaxis._min).toEqual(xBaselineMins);
+            expect(scene2d.xaxis._max).toEqual(xBaselineMaxes);
 
             return Plotly.relayout(gd, 'xaxis.autorange', true);
         }).then(function() {
-
             expect(scene2d.xaxis._min).toEqual(xBaselineMins);
             expect(scene2d.xaxis._max).toEqual(xBaselineMaxes);
 
             return Plotly.relayout(gd, 'yaxis.range', [8, 20]);
         }).then(function() {
-
-            expect(scene2d.yaxis._min).toEqual([]);
-            expect(scene2d.yaxis._max).toEqual([]);
+            expect(scene2d.yaxis._min).toEqual(yBaselineMins);
+            expect(scene2d.yaxis._max).toEqual(yBaselineMaxes);
 
             return Plotly.relayout(gd, 'yaxis.autorange', true);
         }).then(function() {