Fixed not being able to draw linear and radial gradients. Also fixed IKNINJA-1721 - File dirty marker does not show when modifying elements and IKNinja-1581 and IKNinja-1758 - Paint bucket fill bugs with lines and brush objects. Squashed commit of the following:

commit 7cad082c89911d34a99feeef8e91d22b89cae8f1
Author: Nivesh Rajbhandari <mqg734@motorola.com>
Date:   Thu Jun 28 17:52:07 2012 -0700

    Fix for not being able to draw linear and radial gradient materials.

    I'm turning off pre-flight support of gradients for WebGL since it conflicts with the materials' color support.

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

commit d0df8a2bff052d855bafb7885792b1dfff8eab7d
Merge: 2c67d26 b4b54f6
Author: Nivesh Rajbhandari <mqg734@motorola.com>
Date:   Thu Jun 28 17:50:11 2012 -0700

    Merge branch 'refs/heads/ninja-internal' into ToolFixes

commit 2c67d264851f0897fdca8ca887c1c82d0e434217
Author: Nivesh Rajbhandari <mqg734@motorola.com>
Date:   Thu Jun 28 15:06:13 2012 -0700

    IKNINJA-1721 - File dirty marker does not show when modifying elements.

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

commit bb4b6a52eb75efb2f435cdb53f810ae3bc1f1fc3
Author: Nivesh Rajbhandari <mqg734@motorola.com>
Date:   Thu Jun 28 11:15:34 2012 -0700

    IKNinja-1581 and IKNinja-1758 - Paint bucket fill bugs with lines and brush objects.

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

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

2 files changed, 9 insertions, 1 deletions

diff --git a/js/controllers/elements/shapes-controller.js b/js/controllers/elements/shapes-controller.js
index 74353454..5920d51a 100755
--- a/js/controllers/elements/shapes-controller.js
+++ b/js/controllers/elements/shapes-controller.js
@@ -138,6 +138,10 @@ exports.ShapesController = Montage.create(CanvasController, {
                     this.application.ninja.elementMediator.replaceElement(canvas, el);
                     break;
                 case "strokeMaterial":
+                    // skip shape types that don't support WebGL
+                    if(!el.elementModel.shapeModel.GLGeomObj.useWebGl) {
+                        return;
+                    }
                     m = Object.create(MaterialsModel.getMaterial(value));
                     if(m)
                     {
@@ -152,6 +156,10 @@ exports.ShapesController = Montage.create(CanvasController, {
                     }
                     break;
                 case "fillMaterial":
+                    // skip shape types that don't support WebGL or fill color
+                    if(!el.elementModel.shapeModel.GLGeomObj.canFill || !el.elementModel.shapeModel.GLGeomObj.useWebGl) {
+                        return;
+                    }
                     m = Object.create(MaterialsModel.getMaterial(value));
                     if(m)
                     {
diff --git a/js/controllers/styles-controller.js b/js/controllers/styles-controller.js
index 0f847653..e7f95335 100755
--- a/js/controllers/styles-controller.js
+++ b/js/controllers/styles-controller.js
@@ -1437,11 +1437,11 @@ var stylesController = exports.StylesController = Montage.create(Component, {
 
             ///// Dispatch modified event
             NJevent('styleSheetModified', eventData);
+            this.currentDocument.model.needsSave = true;
 
             ///// If the sheet doesn't already exist in the list of modified
             ///// sheets, dispatch dirty event and add the sheet to the list
             if(sheetSearch.length === 0) {
-                this.currentDocument.model.needsSave = true;
                 this.dirtyStyleSheets.push({
                     document : sheet.ownerNode.ownerDocument,
                     stylesheet : sheet