Issue #2833: Returning a new object instead returning an object's reference for some Engine class to avoid some mistake.

Author: dingpinglv

cocos2d/actions/CCActionGrid3D.js

@@ -366,7 +366,7 @@ cc.Lens3D = cc.Grid3DAction.extend(/** @lends cc.Lens3D# */{
      * @return {cc.Point}
      */
     getPosition:function () {
-        return this._position;
+        return cc.p(this._position.x,this._position.y);
     },
 
     /**

cocos2d/effects/CCGrid.js

@@ -97,7 +97,7 @@ cc.GridBase = cc.Class.extend(/** @lends cc.GridBase# */{
      * @return {cc.Size}
      */
     getGridSize:function () {
-        return this._gridSize;
+        return cc.size(this._gridSize.width, this._gridSize.height);
     },
 
     /**
@@ -114,15 +114,16 @@ cc.GridBase = cc.Class.extend(/** @lends cc.GridBase# */{
      * @return {cc.Point}
      */
     getStep:function () {
-        return this._step;
+        return cc.p(this._step.x, this._step.y);
     },
 
     /**
      * set pixels between the grids
      * @param {cc.Point} step
      */
     setStep:function (step) {
-        this._step = step;
+        this._step.x = step.x;
+        this._step.y = step.y;
     },
 
     /**

cocos2d/layers_scenes_transitions_nodes/CCLayer.js

@@ -793,15 +793,9 @@ cc.LayerColor = cc.LayerRGBA.extend(/** @lends cc.LayerColor# */{
     _squareVerticesAB:null,
     _squareColorsAB:null,
 
-    /**
-     * Constructor
-     */
-    ctor: null,
-
     _ctorForCanvas: function () {
         cc.LayerRGBA.prototype.ctor.call(this);
         this._blendFunc = new cc.BlendFunc(cc.BLEND_SRC, cc.BLEND_DST);
-        this._color = new cc.Color4B(0, 0, 0, 0);
     },
 
     _ctorForWebGL: function () {
@@ -1045,7 +1039,6 @@ cc.LayerGradient = cc.LayerColor.extend(/** @lends cc.LayerGradient# */{
     ctor:function () {
         cc.LayerColor.prototype.ctor.call(this);
 
-        this._color = new cc.Color3B(0, 0, 0);
         this._startColor = new cc.Color3B(0, 0, 0);
         this._endColor = new cc.Color3B(0, 0, 0);
         this._alongVector = cc.p(0, -1);
@@ -1060,7 +1053,7 @@ cc.LayerGradient = cc.LayerColor.extend(/** @lends cc.LayerGradient# */{
      * @return {cc.Color3B}
      */
     getStartColor:function () {
-        return this._color;
+        return this._realColor;
     },
 
     /**
@@ -1135,15 +1128,16 @@ cc.LayerGradient = cc.LayerColor.extend(/** @lends cc.LayerGradient# */{
      * @param {cc.Point} Var
      */
     setVector:function (Var) {
-        this._alongVector = Var;
+        this._alongVector.x = Var.x;
+        this._alongVector.y = Var.y;
         this._updateColor();
     },
 
     /**
      * @return {cc.Point}
      */
     getVector:function () {
-        return this._alongVector;
+        return cc.p(this._alongVector.x, this._alongVector.y);
     },
 
     /** is Compressed Interpolation

cocos2d/misc_nodes/CCProgressTimer.js

@@ -77,7 +77,7 @@ cc.ProgressTimer = cc.NodeRGBA.extend(/** @lends cc.ProgressTimer# */{
      *  @return {cc.Point}
      */
     getMidpoint:function () {
-        return this._midPoint;
+        return cc.p(this._midPoint.x, this._midPoint);
     },
 
     /**
@@ -96,7 +96,7 @@ cc.ProgressTimer = cc.NodeRGBA.extend(/** @lends cc.ProgressTimer# */{
      *  @return {cc.Point}
      */
     getBarChangeRate:function () {
-        return this._barChangeRate;
+        return cc.p(this._barChangeRate.x, this._barChangeRate.y);
     },
 
     /**

cocos2d/particle_nodes/CCParticleSystem.js

@@ -727,7 +727,8 @@ cc.ParticleSystem = cc.Node.extend(/** @lends cc.ParticleSystem# */{
      */
     getGravity:function () {
         cc.Assert(this._emitterMode == cc.PARTICLE_MODE_GRAVITY, "Particle Mode should be Gravity");
-        return this.modeA.gravity;
+        var locGravity = this.modeA.gravity;
+        return cc.p(locGravity.x, locGravity.y);
     },
 
     /**

cocos2d/platform/CCEGLView.js

@@ -227,7 +227,7 @@ cc.EGLView = cc.Class.extend(/** @lends cc.EGLView# */{
      * @return {cc.Size}
      */
     getFrameSize:function () {
-        return this._screenSize;
+        return cc.size(this._screenSize.width, this._screenSize.height);
     },
 
     /**
@@ -363,7 +363,7 @@ cc.EGLView = cc.Class.extend(/** @lends cc.EGLView# */{
      * @return {cc.Size}
      */
     getDesignResolutionSize:function () {
-        return this._designResolutionSize;
+        return cc.size(this._designResolutionSize.width, this._designResolutionSize.height);
     },
 
     /**

cocos2d/sprite_nodes/CCSpriteFrame.js

@@ -88,14 +88,18 @@ cc.SpriteFrame = cc.Class.extend(/** @lends cc.SpriteFrame# */{
      * @return {cc.Rect}
      */
     getRectInPixels:function () {
-        return this._rectInPixels;
+        var locRectInPixels = this._rectInPixels;
+        return cc.rect(locRectInPixels.x, locRectInPixels.y, locRectInPixels.width, locRectInPixels.height);
     },
 
     /**
      * @param {cc.Rect} rectInPixels
      */
     setRectInPixels:function (rectInPixels) {
-        this._rectInPixels = rectInPixels;
+        this._rectInPixels.x = rectInPixels.x;
+        this._rectInPixels.y = rectInPixels.y;
+        this._rectInPixels.width = rectInPixels.width;
+        this._rectInPixels.height = rectInPixels.height;
         this._rect = cc.RECT_PIXELS_TO_POINTS(rectInPixels);
     },
 
@@ -130,7 +134,10 @@ cc.SpriteFrame = cc.Class.extend(/** @lends cc.SpriteFrame# */{
      * @param {cc.Rect} rect
      */
     setRect:function (rect) {
-        this._rect = rect;
+        this._rect.x = rect.x;
+        this._rect.y = rect.y;
+        this._rect.width = rect.width;
+        this._rect.height = rect.height;
         this._rectInPixels = cc.RECT_POINTS_TO_PIXELS(this._rect);
     },
 
@@ -147,7 +154,8 @@ cc.SpriteFrame = cc.Class.extend(/** @lends cc.SpriteFrame# */{
      * @param {cc.Point} offsetInPixels
      */
     setOffsetInPixels:function (offsetInPixels) {
-        this._offsetInPixels = offsetInPixels;
+        this._offsetInPixels.x = offsetInPixels.x;
+        this._offsetInPixels.y = offsetInPixels.y;
         this._offset = cc.POINT_PIXELS_TO_POINTS(this._offsetInPixels);
     },
 
@@ -157,15 +165,16 @@ cc.SpriteFrame = cc.Class.extend(/** @lends cc.SpriteFrame# */{
      * @return {cc.Size}
      */
     getOriginalSizeInPixels:function () {
-        return this._originalSizeInPixels;
+        return cc.size(this._originalSizeInPixels.width, this._originalSizeInPixels.height);
     },
 
     /**
      * set original size of the trimmed image
      * @param {cc.Size} sizeInPixels
      */
     setOriginalSizeInPixels:function (sizeInPixels) {
-        this._originalSizeInPixels = sizeInPixels;
+        this._originalSizeInPixels.width = sizeInPixels.width;
+        this._originalSizeInPixels.height = sizeInPixels.height;
     },
 
     /**
@@ -182,7 +191,8 @@ cc.SpriteFrame = cc.Class.extend(/** @lends cc.SpriteFrame# */{
      * @param {cc.Size} sizeInPixels
      */
     setOriginalSize:function (sizeInPixels) {
-        this._originalSize = sizeInPixels;
+        this._originalSize.width = sizeInPixels.width;
+        this._originalSize.height = sizeInPixels.height;
     },
 
     /**

cocos2d/tileMap_parallax_nodes/CCTMXLayer.js

@@ -186,29 +186,31 @@ cc.TMXLayer = cc.SpriteBatchNode.extend(/** @lends cc.TMXLayer# */{
      * @return {cc.Size}
      */
     getLayerSize:function () {
-        return this._layerSize;
+        return cc.size(this._layerSize.width, this._layerSize.height);
     },
 
     /**
      * @param {cc.Size} Var
      */
     setLayerSize:function (Var) {
-        this._layerSize = Var;
+        this._layerSize.width = Var.width;
+        this._layerSize.height = Var.height;
     },
 
     /**
      * Size of the map's tile (could be different from the tile's size)
      * @return {cc.Size}
      */
     getMapTileSize:function () {
-        return this._mapTileSize;
+        return cc.size(this._mapTileSize.width,this._mapTileSize.height);
     },
 
     /**
      * @param {cc.Size} Var
      */
     setMapTileSize:function (Var) {
-        this._mapTileSize = Var;
+        this._mapTileSize.width = Var.width;
+        this._mapTileSize.height = Var.height;
     },
 
     /**

cocos2d/tileMap_parallax_nodes/CCTMXObjectGroup.js

@@ -51,14 +51,15 @@ cc.TMXObjectGroup = cc.Class.extend(/** @lends cc.TMXObjectGroup# */{
      * @return {cc.Point}
      */
     getPositionOffset:function () {
-        return this._positionOffset;
+        return cc.p(this._positionOffset.x,this._positionOffset.y);
     },
 
     /**
      * @param {cc.Point} Var
      */
     setPositionOffset:function (Var) {
-        this._positionOffset = Var;
+        this._positionOffset.x = Var.x;
+        this._positionOffset.y = Var.y;
     },
 
     /**

cocos2d/tileMap_parallax_nodes/CCTMXTiledMap.js

@@ -126,28 +126,30 @@ cc.TMXTiledMap = cc.Node.extend(/** @lends cc.TMXTiledMap# */{
      * @return {cc.Size}
      */
     getMapSize:function () {
-        return this._mapSize;
+        return cc.size(this._mapSize.width, this._mapSize.height);
     },
 
     /**
      * @param {cc.Size} Var
      */
     setMapSize:function (Var) {
-        this._mapSize = Var;
+        this._mapSize.width = Var.width;
+        this._mapSize.height = Var.height;
     },
 
     /**
      * @return {cc.Size}
      */
     getTileSize:function () {
-        return this._tileSize;
+        return cc.size(this._tileSize.width, this._tileSize.height);
     },
 
     /**
      * @param {cc.Size} Var
      */
     setTileSize:function (Var) {
-        this._tileSize = Var;
+        this._tileSize.width = Var.width;
+        this._tileSize.height = Var.height;
     },
 
     /**

cocos2d/tileMap_parallax_nodes/CCTMXXMLParser.js

@@ -296,29 +296,31 @@ cc.TMXMapInfo = cc.SAXParser.extend(/** @lends cc.TMXMapInfo# */{
      * @return {cc.Size}
      */
     getMapSize:function () {
-        return this._mapSize;
+        return cc.size(this._mapSize.width,this._mapSize.height);
     },
 
     /**
      * @param {cc.Size} Var
      */
     setMapSize:function (Var) {
-        this._mapSize = Var;
+        this._mapSize.width = Var.width;
+        this._mapSize.height = Var.height;
     },
 
     /**
      * Tiles width & height
      * @return {cc.Size}
      */
     getTileSize:function () {
-        return this._tileSize;
+        return cc.size(this._tileSize.width, this._tileSize.height);
     },
 
     /**
      * @param {cc.Size} Var
      */
     setTileSize:function (Var) {
-        this._tileSize = Var;
+        this._tileSize.width = Var.width;
+        this._tileSize.height = Var.height;
     },
 
     /**