Sureshbabu pointed out something interesting in an earlier post. The TreeBacker class has the boolean values for leaf reversed. So all of the branch nodes have setLeaf(true) and visa versa.
I will fix this in CVS now, but there is another interesting point here. It doesn't matter to any of the simple examples! This is because the getType method is used to determine the facet (and hence what type of icon to use). The navigation icons (the plus/minus icons) are determined by the child count. IMO the getChildCount method and the getType methods could cover pretty much every scenario that you would need isLeaf for. I have opened a JIRA issue (MYFACES-215) to suggest we deprecate the is Leaf method (and eventually eliminate it.) Please add comments there if you have feelings on this one way or another. sean

