NodeInterface subclass event firing causes stackoverflow

[SamuraiJack1]

Found this code in the NodeInterface:

                fireEventArgs: function(eventName, args) {
                    var rootNode,
                        fireEventArgs = this.superclass.fireEventArgs,
                        result;

                    // The event bubbles (all native NodeInterface events do)...
                    if (bubbledEvents[eventName]) {
                        for (var eventSource = this; result !== false && eventSource; eventSource = (rootNode = eventSource).parentNode) {

And colored line causes infinite recursion if tree model class is a subclass of the another tree model class, since "this.superclass" in such case will still point to the class with NodeInterface applied.

A test case attached (replace the content of /examples/tree/treegrid.js with it).

Proposed solutions:

Quick and dirty:

                fireEventArgs: function(eventName, args) {
                    var rootNode,
                        fireEventArgs = Ext.data.Model.prototype.fireEventArgs,
                        result;

                    // The event bubbles (all native NodeInterface events do)...
                    if (bubbledEvents[eventName]) {
                        for (var eventSource = this; result !== false  && eventSource; eventSource = (rootNode =  eventSource).parentNode) {

This solution does not take into account that some of the intermediate subclasses may re-define `fireEventArgs` additionally, but probably is acceptable.