Converting geom-obj to object literal notation.

Signed-off-by: Nivesh Rajbhandari <mqg734@motorola.com>

8 files changed, 471 insertions, 412 deletions

diff --git a/js/lib/drawing/world.js b/js/lib/drawing/world.js
index 7ce23921..0dde9af4 100755
--- a/js/lib/drawing/world.js
+++ b/js/lib/drawing/world.js
@@ -874,17 +874,17 @@ World.prototype.importObjectJSON = function( jObj, parentGeomObj )
         switch (type)
         {
                 case 1:
-                        obj = new Rectangle();
+                        obj = Object.create(Rectangle, {});
                         obj.importJSON( jObj );
                         break;
 
                 case 2:         // circle
-                        obj = new Circle();
+                        obj = Object.create(Circle, {});
                         obj.importJSON( jObj );
                         break;
 
                 case 3:         // line
-            obj = new Line();
+            obj = Object.create(Line, {});
             obj.importJSON( jObj );
             break;
 
diff --git a/js/lib/geom/brush-stroke.js b/js/lib/geom/brush-stroke.js
index 1fae0c1d..6facdd5d 100755
--- a/js/lib/geom/brush-stroke.js
+++ b/js/lib/geom/brush-stroke.js
@@ -768,7 +768,7 @@ var BrushStroke = function GLBrushStroke() {
 
 }; //function BrushStroke ...class definition
 
-BrushStroke.prototype = new GeomObj();
+BrushStroke.prototype = Object.create(GeomObj, {});
 
 BrushStroke.prototype._CatmullRomSplineInterpolate = function(ctrlPts, t)
 {
diff --git a/js/lib/geom/geom-obj.js b/js/lib/geom/geom-obj.js
index 7cb9b80f..4cb21a25 100755
--- a/js/lib/geom/geom-obj.js
+++ b/js/lib/geom/geom-obj.js
@@ -10,184 +10,239 @@ var MaterialsModel = require("js/models/materials-model").MaterialsModel;
 // Class GLGeomObj
 //      Super class for all geometry classes
 ///////////////////////////////////////////////////////////////////////
-var GeomObj = function GLGeomObj() {
+exports.GeomObj = Object.create(Object.prototype, {
     ///////////////////////////////////////////////////////////////////////
     // Constants
     ///////////////////////////////////////////////////////////////////////
-    this.GEOM_TYPE_RECTANGLE = 1;
-    this.GEOM_TYPE_CIRCLE = 2;
-    this.GEOM_TYPE_LINE = 3;
-    this.GEOM_TYPE_PATH = 4;
-    this.GEOM_TYPE_CUBIC_BEZIER = 5;
-    this.GEOM_TYPE_BRUSH_STROKE = 6;
-    this.GEOM_TYPE_UNDEFINED = -1;
-
-    // Needed for calculating dashed/dotted strokes
-    this.DASH_LENGTH = 0.15;
-    this.DOT_LENGTH = 0.05;
-    this.GAP_LENGTH = 0.05;
+    // TODO - Is there a way to make these static constants?
+    GEOM_TYPE_RECTANGLE: { value : 1, writable: false },
+    GEOM_TYPE_CIRCLE: { value : 2, writable: false },
+    GEOM_TYPE_LINE: { value : 3, writable: false },
+    GEOM_TYPE_PATH: { value : 4, writable: false },
+    GEOM_TYPE_CUBIC_BEZIER: { value : 5, writable: false },
+    GEOM_TYPE_BRUSH_STROKE: { value : 6, writable: false },
+    GEOM_TYPE_UNDEFINED: { value : -1, writable: false },
 
     ///////////////////////////////////////////////////////////////////////
     // Instance variables
     ///////////////////////////////////////////////////////////////////////
-    this._matrix = Matrix.I(4);
+    _matrix: { value : Matrix.I(4), writable: true },
 
-    this._next = undefined;
-    this._prev = undefined;
-    this._child = undefined;
-    this._parent = undefined;
+    _next: { value : undefined, writable: true },
+    _prev: { value : undefined, writable: true },
+    _child: { value : undefined, writable: true },
+    _parent: { value : undefined, writable: true },
 
-    this.m_world = null;
+    m_world: { value : null, writable: true },
 
     // stroke and fill colors
-    this._strokeColor = [0, 0, 0, 0];
-    this._fillColor = [0, 0, 0, 0];
+    _strokeColor: { value : [0, 0, 0, 0], writable: true },
+    _fillColor: { value : [0, 0, 0, 0], writable: true },
 
     // stroke and fill materials
-    this._fillMaterial = null;
-    this._strokeMaterial = null;
+    _fillMaterial: { value : null, writable: true },
+    _strokeMaterial: { value : null, writable: true },
 
     // Shapes (such as lines) that don't support fill should set this to false
-    this.canFill = true;
+    canFill: { value : true, writable: true },
 
     // array of primitives - used in RDGE
-    this._primArray = [];
-    this._materialNodeArray = [];
-    this._materialArray = [];
-    this._materialTypeArray = [];
+    _primArray: { value : [], writable: true },
+    _materialNodeArray: { value : [], writable: true },
+    _materialArray: { value : [], writable: true },
+    _materialTypeArray: { value : [], writable: true },
 
     // the transform node used by RDGE
-    this._trNode = null;
+    _trNode: { value : null, writable: true },
 
     ///////////////////////////////////////////////////////////////////////
     // Property accessors
     ///////////////////////////////////////////////////////////////////////
-    this.setWorld = function (world) {
-        this.m_world = world;
-    };
+    getWorld: {
+        value: function() {
+            return this.m_world;
+        }
+    },
 
-    this.getWorld = function () {
-        return this.m_world;
-    };
+    setWorld: {
+        value: function(world) {
+            this.m_world = world;
+        }
+    },
 
-    this.getMatrix = function () {
-        return this._matrix.slice(0);
-    };
+    getMatrix: {
+        value: function() {
+            return this._matrix.slice(0);
+        }
+    },
 
-    this.setMatrix = function (m) {
-        this._matrix = m.slice(0);
-    };
+    setMatrix: {
+        value: function(m) {
+            this._matrix = m.slice(0);
+        }
+    },
 
-    this.setNext = function (next) {
-        this._next = next;
-    };
+    getNext: {
+        value: function() {
+            return this._next;
+        }
+    },
 
-    this.getNext = function () {
-        return this._next;
-    };
+    setNext: {
+        value: function(next) {
+            this._next = next;
+        }
+    },
 
-    this.setPrev = function (prev) {
-        this._prev = prev;
-    };
+    getPrev: {
+        value: function() {
+            return this._prev;
+        }
+    },
 
-    this.getPrev = function () {
-        return this._prev;
-    };
+    setPrev: {
+        value: function(prev) {
+            this._prev = prev;
+        }
+    },
 
-    this.setChild = function (child) {
-        this._child = child;
-    };
+    getChild: {
+        value: function() {
+            return this._child;
+        }
+    },
 
-    this.getChild = function () {
-        return this._child;
-    };
+    setChild: {
+        value: function(child) {
+            this._child = child;
+        }
+    },
 
-    this.setParent = function (parent) {
-        this._parent = parent;
-    };
+    getParent: {
+        value: function() {
+            return this._parent;
+        }
+    },
 
-    this.getParent = function () {
-        return this._parent;
-    };
+    setParent: {
+        value: function(parent) {
+            this._parent = parent;
+        }
+    },