From 3cda26846833304e87bb0b036d8c3d0e68f002a8 2014-01-16 10:57:02
From: Jonathan Frederic <jdfreder@calpoly.edu>
Date: 2014-01-16 10:57:02
Subject: [PATCH] Privatize _handle_com_open of WidgetManager

---

diff --git a/IPython/html/static/notebook/js/widget.js b/IPython/html/static/notebook/js/widget.js
index acc1430..72cef4e 100644
--- a/IPython/html/static/notebook/js/widget.js
+++ b/IPython/html/static/notebook/js/widget.js
@@ -435,7 +435,7 @@ define(["components/underscore/underscore-min",
 
         // Register already register widget model types with the comm manager.
         for (var widget_model_name in this.widget_model_types) {
-            this.comm_manager.register_target(widget_model_name, $.proxy(this.handle_com_open, this));
+            this.comm_manager.register_target(widget_model_name, $.proxy(this._handle_com_open, this));
         }
     }
 
@@ -444,7 +444,7 @@ define(["components/underscore/underscore-min",
         // Register the widget with the comm manager.  Make sure to pass this object's context
         // in so `this` works in the call back.
         if (this.comm_manager!=null) {
-            this.comm_manager.register_target(widget_model_name, $.proxy(this.handle_com_open, this));
+            this.comm_manager.register_target(widget_model_name, $.proxy(this._handle_com_open, this));
         }
         this.widget_model_types[widget_model_name] = widget_model_type;
     }
@@ -455,12 +455,6 @@ define(["components/underscore/underscore-min",
     }
 
 
-    WidgetManager.prototype.handle_com_open = function (comm, msg) {
-        var widget_type_name = msg.content.target_name;
-        var widget_model = new this.widget_model_types[widget_type_name](this.comm_manager, comm, this);
-    }
-
-
     WidgetManager.prototype.get_msg_cell = function (msg_id) {
         if (IPython.notebook != undefined && IPython.notebook != null) {
             return IPython.notebook.get_msg_cell(msg_id);
@@ -468,6 +462,12 @@ define(["components/underscore/underscore-min",
     }
 
 
+    WidgetManager.prototype._handle_com_open = function (comm, msg) {
+        var widget_type_name = msg.content.target_name;
+        var widget_model = new this.widget_model_types[widget_type_name](this.comm_manager, comm, this);
+    }
+
+
     //--------------------------------------------------------------------
     // Init code
     //--------------------------------------------------------------------