Ensure multiple views work

Without this, rendering the same scene in different views (renderers)
would sometimes miss things like shadow maps being enabled, etc. Added a
change check to ensure we don't keep setting `needsUpdate` on the shadow
map unless we actually have changes.

Author: vidartf

js/src/_base/Renderable.js

@@ -211,8 +211,10 @@ var RenderableView = widgets.DOMWidgetView.extend({
                 return;
             }
             if (!converterName) {
-                obj[propName] = this.model.get(propName);
-                changes = true;
+                if (obj[propName] !== this.model.get(propName)) {
+                    obj[propName] = this.model.get(propName);
+                    changes = true;
+                }
                 return;
             }
             var converterFn;
@@ -226,8 +228,11 @@ var RenderableView = widgets.DOMWidgetView.extend({
             if (!converterFn) {
                 throw new Error('invalid converter name: ' + converterName);
             }
-            obj[propName] = converterFn(model.get(propName), propName);
-            changes = true;
+            var converted = converterFn(model.get(propName), propName);
+            if (obj[propName] !== converted) {
+                obj[propName] = converted;
+                changes = true;
+            }
         }, this);
         if (changes) {
             obj.needsUpdate = true;

js/src/core/Renderer.js

@@ -165,7 +165,7 @@ var RendererView = RenderableView.extend({
 
         // We need to ensure that renderer properties are applied
         // (we have no idea where the renderer has been...)
-        this.applyBackground();
+        this.updateProperties(true);
     },
 
 });