Merge branch 'refs/heads/masterDomArc' into binding

28 files changed, 2525 insertions, 2515 deletions

diff --git a/js/components/layout/bread-crumb.reel/bread-crumb.js b/js/components/layout/bread-crumb.reel/bread-crumb.js
index e396bdbf..c1b021a3 100755
--- a/js/components/layout/bread-crumb.reel/bread-crumb.js
+++ b/js/components/layout/bread-crumb.reel/bread-crumb.js
@@ -23,7 +23,7 @@ exports.Breadcrumb = Montage.create(Component, {
         value: function(){
             if(!this.application.ninja.documentController.activeDocument) {
                 this.disabled = true;
-                this.application.ninja.currentSelectedContainer = this.application.ninja.currentDocument.documentRoot;
+                this.application.ninja.currentSelectedContainer = (this.application.ninja.currentDocument ? this.application.ninja.currentDocument.documentRoot : null);
             }
         }
     },
diff --git a/js/controllers/document-controller.js b/js/controllers/document-controller.js
index 803519a7..2be2e214 100755
--- a/js/controllers/document-controller.js
+++ b/js/controllers/document-controller.js
@@ -427,10 +427,6 @@ var DocumentController = exports.DocumentController = Montage.create(Component,
 
     closeDocument: {
         value: function(id) {
-            if(this.activeDocument.needsSave === true){
-                //if file dirty then alert user to save
-        }
-
             var doc = this._findDocumentByUUID(id);
 
             var closeDocumentIndex = this._findIndexByUUID(id);
diff --git a/js/controllers/styles-controller.js b/js/controllers/styles-controller.js
index ec4314f9..2ff3e235 100755
--- a/js/controllers/styles-controller.js
+++ b/js/controllers/styles-controller.js
@@ -795,7 +795,9 @@ var stylesController = exports.StylesController = Montage.create(Component, {
             ///// method to apply/test the new value
             dec.setProperty(property, value, priority);
 
-            this.styleSheetModified(rule.parentStyleSheet);
+            if(rule.parentStyleSheet) {
+                this.styleSheetModified(rule.parentStyleSheet);
+            }
 
             ///// Return browser value for value we just set
             return dec.getPropertyValue(property);
@@ -1102,7 +1104,7 @@ var stylesController = exports.StylesController = Montage.create(Component, {
                 ///// The dominant rule might not have the style property defined - why?
                 ///// If no rules have the property defined, we can use the
                 ///// most-specific single-target rule as the dominant rule (for setting styles)
-                return element.ownerDocument.defaultView.getComputedStyle(element).getPropertyValue(property);
+                return (element.ownerDocument.defaultView ? element.ownerDocument.defaultView.getComputedStyle(element).getPropertyValue(property) : null);
             }
 
             return value;
diff --git a/js/controllers/tree-controller.js b/js/controllers/tree-controller.js
deleted file mode 100644
index 2b2e910c..00000000
--- a/js/controllers/tree-controller.js
+++ /dev/null
@@ -1,185 +0,0 @@
-/* <copyright>
- This file contains proprietary software owned by Motorola Mobility, Inc.<br/>
- No rights, expressed or implied, whatsoever to this software are provided by Motorola Mobility, Inc. hereunder.<br/>
- (c) Copyright 2011 Motorola Mobility, Inc.  All Rights Reserved.
- </copyright> */
-/**
- @module montage/ui/controller/tree-controller
- @requires montage/core/core
- @requires montage/ui/controller/object-controller
- @requires montage/core/event/mutable-event
- */
-var Montage = require("montage").Montage,
-    ObjectController = require("montage/ui/controller/object-controller").ObjectController,
-    ArrayController = require("montage/ui/controller/array-controller").ArrayController,
-    MutableEvent = require("montage/core/event/mutable-event").MutableEvent;
-/**
-    TODO: Write description like the array controllers: The ArrayController helps with organizing a hierarchical
-    collection of objects, and managing user selection within that collection.
-    You can assign a TreeController instance as the <code>contentProvider</code> property for a TreeView object.
-    @class module:montage/ui/controller/tree-controller.TreeController
-    @classdesc
-    @extends module:montage/ui/controller/object-controller.ObjectController
-*/
-var TreeController = exports.TreeController = Montage.create(ObjectController, /** @lends module:montage/ui/controller/tree-controller.TreeController# */ {
-
-    rootKey : {
-        value: null
-    },
-
-    branchKey : {
-        value: 'children'
-    },
-
-    _root : {
-        value : null
-    },
-    root : {
-        get: function() {
-            return this._root;
-        },
-        set: function(value) {
-            this._root = value;
-
-            this.initArrayControllers();
-        }
-    },
-
-    rootController: {
-        value: null
-    },
-
-    initArrayControllers : {
-        value: function() {
-            var self = this;
-
-            ///// Recursive function that finds all branch nodes and initializes
-            ///// sets the tree node type to "branch" or "leaf"
-
-            function walk(node, init, depth) {
-                var branch = node[self.branchKey];
-
-                if(branch) {
-                    branch.forEach(function(node) {
-                        walk(node, init, ++depth);
-                    });
-
-                    node['treeNodeType'] = 'branch';
-                } else {
-                    node['treeNodeType'] = 'leaf';
-                }
-            }
-
-            walk(this._root, 0);
-
-        }
-    },
-
-    /**
-        @private
-     */
-    _selectedIndexes: {
-        value: null,
-        enumerable: false
-    },
-
-   /**
-        Description TODO
-        @type {Function}
-        @default null
-    */
-    selectedIndexes: {
-        get: function() {
-
-        },
-        set: function(value) {
-
-        }
-    },
-
-    branchControllers: {
-        value: []
-    },
-
-    addBranchController : {
-        value: function(controller) {
-            if(this.delegate) {
-                controller.delegate = this.delegate;
-            }
-
-            this.branchControllers.push(controller);
-        }
-    },
-
-    /**
-     @private
-     */
-    _content: {
-        enumerable: false,
-        value: null
-    },
-    /**
-        The content managed by the TreeController.
-        @type {Function}
-        @default {String} null
-    */
-    content: {
-        get: function() {