Address review comments

IPython/html/static/services/kernels/comm.js

@@ -92,12 +92,12 b' define(['
     
     CommManager.prototype.comm_close = function(msg) {
         var content = msg.content;
-        if (!this.comms[content.comm_id]) {
+        if (this.comms[content.comm_id] === undefined) {
             console.error('Comm promise not found for comm id ' + content.comm_id);
             return;
         }
 
-        this.comms[content.comm_id].then(function(comm) {
+        this.comms[content.comm_id] = this.comms[content.comm_id].then(function(comm) {
             this.unregister_comm(comm);
             try {
                 comm.handle_close(msg);
@@ -109,12 +109,12 b' define(['
     
     CommManager.prototype.comm_msg = function(msg) {
         var content = msg.content;
-        if (!this.comms[content.comm_id]) {
+        if (this.comms[content.comm_id] === undefined) {
             console.error('Comm promise not found for comm id ' + content.comm_id);
             return;
         }
 
-        this.comms[content.comm_id].then(function(comm) {
+        this.comms[content.comm_id] = this.comms[content.comm_id].then(function(comm) {
             try {
                 comm.handle_msg(msg);
             } catch (e) {
@@ -174,7 +174,7 b' define(['
     
     // methods for handling incoming messages
     
-    Comm.prototype._maybe_callback = function (key, msg) {
+    Comm.prototype._callback = function (key, msg) {
         var callback = this['_' + key + '_callback'];
         if (callback) {
             try {
@@ -186,11 +186,11 b' define(['
     };
     
     Comm.prototype.handle_msg = function (msg) {
-        that._maybe_callback('msg', msg);
+        this._callback('msg', msg);
     };
     
     Comm.prototype.handle_close = function (msg) {
-        this._maybe_callback('close', msg);
+        this._callback('close', msg);
     };
     
     // For backwards compatability.

IPython/html/static/widgets/js/manager.js

@@ -186,7 +186,7 b' define(['
         return this.create_model({
             model_name: msg.content.data.model_name, 
             model_module: msg.content.data.model_module, 
-            comm: comm}).catch($.proxy(console.error, console));
+            comm: comm}).catch(utils.reject("Couldn't create a model."));
     };
 
     WidgetManager.prototype.create_model = function (options) {

IPython/html/static/widgets/js/widget_int.js

@@ -150,7 +150,7 b' define(['
                     this.$label.hide();
                 } else {
                     this.$label.text(description);
-                    // MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
+                    MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
                     this.$label.show();
                 }
                 
@@ -308,7 +308,7 b' define(['
                     this.$label.hide();
                 } else {
                     this.$label.text(description);
-                    // MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
+                    MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
                     this.$label.show();
                 }
             }
@@ -416,7 +416,7 b' define(['
                 this.$label.hide();
             } else {
                 this.$label.text(description);
-                // MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
+                MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
                 this.$label.show();
             }
             return ProgressView.__super__.update.apply(this);

IPython/html/static/widgets/js/widget_selection.js

@@ -94,7 +94,7 b' define(['
                     this.$label.hide();
                 } else {
                     this.$label.text(description);
-                    // MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
+                    MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
                     this.$label.show();
                 }
             }
@@ -219,7 +219,7 b' define(['
                     this.$label.hide();
                 } else {
                     this.$label.text(description);
-                    // MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
+                    MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
                     this.$label.show();
                 }
             }
@@ -326,7 +326,7 b' define(['
                     this.$label.hide();
                 } else {
                     this.$label.text(description);
-                    // MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
+                    MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
                     this.$label.show();
                 }
             }
@@ -441,7 +441,7 b' define(['
                     this.$label.hide();
                 } else {
                     this.$label.text(description);
-                    // MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
+                    MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
                     this.$label.show();
                 }
             }

IPython/html/static/widgets/js/widget_string.js

@@ -101,7 +101,7 b' define(['
                     this.$label.hide();
                 } else {
                     this.$label.text(description);
-                    // MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
+                    MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
                     this.$label.show();
                 }
             }
@@ -177,7 +177,7 b' define(['
                     this.$label.hide();
                 } else {
                     this.$label.text(description);
-                    // MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
+                    MathJax.Hub.Queue(["Typeset",MathJax.Hub,this.$label.get(0)]);
                     this.$label.show();
                 }
             }

IPython/html/tests/widgets/widget.js

@@ -131,7 +131,7 b' casper.notebook_test(function () {'
         multiset.model_id = this.get_output_cell(index).text.trim();
     });
 
-    this.wait_for_widget(multiset); 
+    this.wait_for_widget(multiset);
 
     index = this.append_cell(
         'print("%d%d%d" % (multiset.a, multiset.b, multiset.c))');