Is this method really necessary any more? It's a very clunky method which locks the entire queue while it is running.
The queue now has an iterator which can be used surely, which does not lock the queue?
The list() method is used only by management classes.
You're right: the list() method can be replaced by using Queue#iterator() instead without locking the queue. I'll rewrite the management classes.