Update JTree when model changes

Question

Hello!
I'am doing a webbrowser that stores the history in a JTree that is visible in EAST in the browserFrame.
The history is updated when a link is clicked or it is typed in the addressbar. Everything works for the moment but it's quite ugly i guess...
For now i do tree.updateUI(); in the view everytime I have added a node, but I want to remove tree.updateUI() and update the view automaticly whenever the model has changed.
Is nodesWereInserted() the right function to use?
I have tried to use this listener in the HistoryModel(after removing tree.updateUI()) but the JTree acts very strange then...
thanks for any help :)
Fredrik
////////////THE LISTENER THAT I HAVE TRIED TO USE IN THE HistoryModel
    class MyTreeModelListener implements TreeModelListener {
        public void treeNodesChanged(TreeModelEvent e) {
            DefaultMutableTreeNode node;
            node = (DefaultMutableTreeNode)
                     (e.getTreePath().getLastPathComponent());
             * If the event lists children, then the changed
             * node is the child of the node we've already
             * gotten.  Otherwise, the changed node and the
             * specified node are the same.
            try {
                int index = e.getChildIndices()[0];
                node = (DefaultMutableTreeNode)
                       (node.getChildAt(index));
            } catch (NullPointerException exc) {}
            System.out.println("The user has finished editing the node.");
            System.out.println("New value: " + node.getUserObject());
        public void treeNodesInserted(TreeModelEvent e) {
            DefaultMutableTreeNode node;
            node = (DefaultMutableTreeNode)
                     (e.getTreePath().getLastPathComponent());           
            int index = treeModel.getIndexOfChild(node.getParent(), node);
            treeModel.nodesWereInserted(node.getParent(), new int[] {index});
        public void treeNodesRemoved(TreeModelEvent e) {
        public void treeStructureChanged(TreeModelEvent e) {
        }////////////////THIS IS THE VIEW CLASS
package freebrowser.gui;
import java.net.URL;
import java.util.ArrayList;
import java.util.Iterator;
import javax.swing.JTree;
import javax.swing.event.TreeSelectionEvent;
import javax.swing.event.TreeSelectionListener;
import javax.swing.tree.DefaultMutableTreeNode;
import javax.swing.tree.TreeSelectionModel;
import freebrowser.model.HistoryModel;
import freebrowser.model.LinkInfo;
import freebrowser.types.Browser;
* This class represents a JTree with a domainname as children and url:s in its leafs.
* @author dalen
public class HistoryTree implements TreeSelectionListener {
    private JTree tree;
    private Browser browser;
    private ArrayList currentPageLinkInfo;
    private URL browserCurrentUrl;
    private HistoryModel historyModel;
    public HistoryTree(Browser browser, HistoryModel historyModel){
        this.browser = browser;
        this.historyModel = historyModel;
        //Create a tree that allows one selection at a time.
        tree = historyModel.createJtree();
        tree.getSelectionModel().setSelectionMode(TreeSelectionModel.SINGLE_TREE_SELECTION);
        //Listen for when the selection changes.
        tree.addTreeSelectionListener(this);
     * This function is called when a link has been clicked.
     * @param url - The <code>url</code> that has been clicked
    public void newPageClicked(URL url){
        Iterator it = currentPageLinkInfo.iterator();
        while(it.hasNext()){
            LinkInfo info = (LinkInfo)it.next();
            if(info.getUrl().equals(url)){
                historyModel.addNode(info, browserCurrentUrl);
                tree.updateUI(); /// this is what I want to remove!
                break;
     * Adds a node to the tree if a url has been entered in the addresfield
     * @param url
     * @param title
    public void addressBarPageEntered(URL url, String title) {
        LinkInfo info = new LinkInfo(url, title);
        browserCurrentUrl = url;
        historyModel.addNode(info, browserCurrentUrl);
        tree.updateUI();/// this is what I want to remove!
     * Helps the <code>HistoryBar</code> to be updated with the current url.
     * @param url - The curren <code>url</code> in the browser
    public void setHistoryCurrentPage(URL url){
        LinkInfo info = new LinkInfo(url, null);
        currentPageLinkInfo = info.getLinkInfos();
        browserCurrentUrl = url;
     * Invoked by TreeSelectionListener when a node has been clicked
    public void valueChanged(TreeSelectionEvent e) {
        DefaultMutableTreeNode node = (DefaultMutableTreeNode)tree.getLastSelectedPathComponent();
        if (node == null) return;
        Object nodeInfo = node.getUserObject();
        if (node.isLeaf()) {
            LinkInfo linkInfo = (LinkInfo)nodeInfo;
            System.out.println(linkInfo.getUrl());
            browser.setURL(linkInfo.getUrl());
     * Function to recieve the JTree
     * @return The <code>tree</code>
    public JTree getTree() {
        return this.tree;
}/////////////THIS IS THE MODEL CLASS
package freebrowser.model;
import java.net.URL;
import java.util.Enumeration;
import javax.swing.JTree;
import javax.swing.tree.DefaultMutableTreeNode;
import javax.swing.tree.DefaultTreeModel;
* This class holds the infomation of the historytree. It is stored in an array containing <code>Nodes</code>.
* @author dalen
public class HistoryModel {
    private DefaultTreeModel treeModel;
    private DefaultMutableTreeNode rootNode;
    public HistoryModel() {
        rootNode =  new DefaultMutableTreeNode("L�nkHistorik");
        treeModel = new DefaultTreeModel(rootNode);
     * Adds a node in the right place in the model
     * @param info - The <code>info</code> to be added.
    public void addNode(LinkInfo info, URL browserCurrentUrl) {
        //l�gg till noden p� r�tt plats i modellen
        Enumeration e = rootNode.children();
        DefaultMutableTreeNode category = null;
        DefaultMutableTreeNode urlLeaf = null;
        boolean newNodeSet = false;
        while(e.hasMoreElements()) {
            DefaultMutableTreeNode childToCheck = (DefaultMutableTreeNode)e.nextElement();
            //check if the domainname already is in the tree
            if(childToCheck.toString().equals( browserCurrentUrl.getHost()) ){
                urlLeaf = new DefaultMutableTreeNode(info);
                treeModel.insertNodeInto(urlLeaf, childToCheck, childToCheck.getChildCount());
                newNodeSet = true;
                break;
        if(!newNodeSet) {
            category = new DefaultMutableTreeNode(new LinkInfo(null,browserCurrentUrl.getHost()));
            DefaultMutableTreeNode parent = (DefaultMutableTreeNode)treeModel.getRoot();
            treeModel.insertNodeInto(category, parent, parent.getChildCount());
            urlLeaf = new DefaultMutableTreeNode(info);
            treeModel.insertNodeInto(urlLeaf, category, category.getChildCount());
        ((DefaultMutableTreeNode)treeModel.getRoot()).setUserObject("L�nkhistorik (dom�ner: " + rootNode.getChildCount() + "  l�nkar: " + rootNode.getLeafCount() +")");
     * Creates and returns a JTree built on the model from createJTreeModel()
     * @return The new JTree
    public JTree createJtree() {
        return new JTree(treeModel);
}

Answer 1

Nevermind, I solved it by my self :)
added treeModel.nodeChanged(node); after insertNodeInto()