Remove extra code from Circle/Triangle (fabricjs#4547)

* removed code

* removed code

* removed code

src/shapes/circle.class.js

@@ -49,16 +49,6 @@
 
     cacheProperties: fabric.Object.prototype.cacheProperties.concat('radius'),
 
-    /**
-     * Constructor
-     * @param {Object} [options] Options object
-     * @return {fabric.Circle} thisArg
-     */
-    initialize: function(options) {
-      this.callSuper('initialize', options);
-      this.set('radius', options && options.radius || 0);
-    },
-
     /**
      * @private
      * @param {String} key

src/shapes/triangle.class.js

@@ -26,15 +26,18 @@
     type: 'triangle',
 
     /**
-     * Constructor
-     * @param {Object} [options] Options object
-     * @return {Object} thisArg
+     * Width is set to 100 to compensate the old initialize code that was setting it to 100
+     * @type Number
+     * @default
      */
-    initialize: function(options) {
-      this.callSuper('initialize', options);
-      this.set('width', options && options.width || 100)
-        .set('height', options && options.height || 100);
-    },
+    width: 100,
+
+    /**
+     * Height is set to 100 to compensate the old initialize code that was setting it to 100
+     * @type Number
+     * @default
+     */
+    height: 100,
 
     /**
      * @private

test/unit/circle.js

@@ -13,6 +13,13 @@
     assert.deepEqual(circle.type, 'circle');
   });
 
+  QUnit.test('constructor with radius', function(assert) {
+    assert.ok(fabric.Circle);
+    var circle = new fabric.Circle({ radius: 20 });
+    assert.equal(circle.width, 40, 'width is set');
+    assert.equal(circle.height, 40, 'height is set');
+  });
+
   QUnit.test('getRadiusX, getRadiusY', function(assert) {
     var circle = new fabric.Circle({ radius: 10 });