From f32e48cd0864af2f57c249fbd5d85f8ffb0ce60f 2014-01-16 10:57:13
From: Jason Grout <jason.grout@drake.edu>
Date: 2014-01-16 10:57:13
Subject: [PATCH] Make the widget keys property traverse the superclasses and accumulate the _keys attributes.

This caches the result, overwriting the property.
---

diff --git a/IPython/html/widgets/widget.py b/IPython/html/widgets/widget.py
index 11d8147..3a9e4b1 100644
--- a/IPython/html/widgets/widget.py
+++ b/IPython/html/widgets/widget.py
@@ -84,14 +84,22 @@ class BaseWidget(LoggingConfigurable):
         removed from the frontend."""
         self._close_communication()
     
-    
+    _keys = ['default_view_name']
+
     # Properties
     @property
     def keys(self):
-        keys = ['default_view_name']
-        keys.extend(self._keys)
+        """Lazily accumulate _keys from all superclasses and cache them in this class"""
+        keys=[]
+        for c in self.__class__.mro():
+            if hasattr(c, '_keys'):
+                keys.extend(getattr(c,'_keys'))
+            else:
+                break
+        # cache so future lookups are fast
+        self.__class__.x = keys
         return keys
-    
+
     @property
     def comm(self):
         if self._comm is None:
@@ -346,12 +354,7 @@ class Widget(BaseWidget):
     # Private/protected declarations
     _css = Dict() # Internal CSS property dict
 
-    # Properties
-    @property
-    def keys(self):
-        keys = ['visible', '_css']
-        keys.extend(super(Widget, self).keys)
-        return keys
+    _keys = ['visible', '_css']
 
     def get_css(self, key, selector=""):
         """Get a CSS property of the widget.  Note, this function does not