Description Marc Dumais 2013-03-28 10:52:35 EDT

When threads are selected and/or unselected in the multicore visualizer, the selection is synchronized in the debug view.   I noticed that this only seems to work for the first debug session, however.   Not the first session created, but whatever session is the first (topmost in the debug view) at any given moment.

After a bit of investigation, I think the reason is how DebugViewTreeWalker.getRootPath() is implemented.

Consider the following line from that method: 

Object root = (items == null || items.length == 0) ? null : items[0].getData();

It seems that each debug session has its entry in the "items" array above.  So, in the current code, only the first session is considered.

To give a bit of context, here is the whole method:

	public TreePath getRootPath()
	{
		// (?) This doesn't always return the expected root element.
		//return m_viewer.getTopElementPath();
		
		TreePath path = null;
		if (m_viewer != null) {
			Tree tree = (Tree) m_viewer.getControl();
			TreeItem[] items = tree.getItems();
			Object root = (items == null || items.length == 0) ? null : items[0].getData();
			if (root != null) path = new TreePath(new Object[] {root});
		}
		return path;
	}