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

Author: dingpinglv

cocos2d/CCDirector.js

@@ -435,7 +435,7 @@ cc.Director = cc.Class.extend(/** @lends cc.Director# */{
      * @return {cc.Size}
      */
     getWinSize:function () {
-        return this._winSizeInPoints;
+        return cc.size(this._winSizeInPoints.width, this._winSizeInPoints.height);
     },
 
     /**
@@ -453,17 +453,15 @@ cc.Director = cc.Class.extend(/** @lends cc.Director# */{
     getVisibleSize:function () {
         if (this._openGLView) {
             return this._openGLView.getVisibleSize();
-        }
-        else {
+        } else {
             return cc.size(0,0);
         }
     },
 
     getVisibleOrigin:function () {
         if (this._openGLView) {
             return this._openGLView.getVisibleOrigin();
-        }
-        else {
+        } else {
             return cc.p(0, 0);
         }
     },
@@ -939,6 +937,7 @@ cc.Director = cc.Class.extend(/** @lends cc.Director# */{
 
     /**
      * seconds per frame
+     * @return {Number}
      */
     getSecondsPerFrame:function () {
         return this._secondsPerFrame;

cocos2d/actions/CCActionGrid3D.js

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

cocos2d/label_nodes/CCLabelTTF.js

@@ -172,7 +172,7 @@ cc.LabelTTF = cc.Sprite.extend(/** @lends cc.LabelTTF# */{
      * @return {cc.Size}
      */
     getDimensions:function () {
-        return this._dimensions;
+        return cc.size(this._dimensions.width, this._dimensions.height);
     },
 
     /**

cocos2d/layers_scenes_transitions_nodes/CCLayer.js

@@ -635,7 +635,7 @@ cc.LayerRGBA = cc.Layer.extend(/** @lends cc.LayerRGBA# */{
      * @param {Number} parentOpacity
      */
     updateDisplayedOpacity: function (parentOpacity) {
-        this._displayedOpacity = this._realOpacity * parentOpacity/255.0;
+        this._displayedOpacity = 0|(this._realOpacity * parentOpacity/255.0);
 
         if (this._cascadeOpacityEnabled){
             var locChildren = this._children;
@@ -656,37 +656,44 @@ cc.LayerRGBA = cc.Layer.extend(/** @lends cc.LayerRGBA# */{
     },
 
     getColor: function () {
-        return this._realColor;
+        var locRealColor = this._realColor;
+        return cc.c3b(locRealColor.r, locRealColor.g, locRealColor.b);
     },
 
     getDisplayedColor: function () {
-        return this._displayedColor;
+        var locDisplayedColor = this._displayedColor;
+        return cc.c3b(locDisplayedColor.r, locDisplayedColor.g, locDisplayedColor.b);
     },
 
     setColor: function (color) {
-        this._displayedColor = cc.c3b(color.r, color.g, color.b);
-        this._realColor = cc.c3b(color.r,color.g, color.b);
+        var locDisplayed = this._displayedColor, locRealColor = this._realColor;
+        locDisplayed.r = locRealColor.r = color.r;
+        locDisplayed.g = locRealColor.g = color.g;
+        locDisplayed.b = locRealColor.b = color.b;
 
         if (this._cascadeColorEnabled){
-            var parentColor = cc.white();
+            var parentColor;
             var locParent = this._parent;
             if (locParent && locParent.RGBAProtocol && locParent.isCascadeColorEnabled())
                 parentColor = locParent.getDisplayedColor();
+            else
+                parentColor = cc.white();
             this.updateDisplayedColor(parentColor);
         }
     },
 
     updateDisplayedColor: function (parentColor) {
-        this._displayedColor.r = this._realColor.r * parentColor.r/255.0;
-        this._displayedColor.g = this._realColor.g * parentColor.g/255.0;
-        this._displayedColor.b = this._realColor.b * parentColor.b/255.0;
+        var locDisplayedColor = this._displayedColor, locRealColor = this._realColor;
+        locDisplayedColor.r = 0|(locRealColor.r * parentColor.r/255.0);
+        locDisplayedColor.g = 0|(locRealColor.g * parentColor.g/255.0);
+        locDisplayedColor.b = 0|(locRealColor.b * parentColor.b/255.0);
 
         if (this._cascadeColorEnabled){
             var locChildren = this._children;
             for(var i = 0; i < locChildren.length; i++){
                 var selItem = locChildren[i];
                 if(selItem && selItem.RGBAProtocol)
-                    selItem.updateDisplayedColor(this._displayedColor);
+                    selItem.updateDisplayedColor(locDisplayedColor);
             }
         }
     },

cocos2d/platform/CCEGLView.js

@@ -257,7 +257,7 @@ cc.EGLView = cc.Class.extend(/** @lends cc.EGLView# */{
         if (this._resolutionPolicy === cc.RESOLUTION_POLICY.NOBORDER) {
             return cc.size(this._screenSize.width / this._scaleX, this._screenSize.height / this._scaleY);
         } else {
-            return this._designResolutionSize;
+            return cc.size(this._designResolutionSize.width, this._designResolutionSize.height);
         }
     },