[runtime] Possible Broacaster leaks

[jfarcand]

From this thread

Hi,
Regarding the memory leak, in the class DefaultBroadcasterFactory

    @Override
    public Broadcaster lookup(Class<? extends Broadcaster> c, Object id, boolean createIfNull) {
        Broadcaster b = store.get(id);
        if (b != null && !c.isAssignableFrom(b.getClass())) {
            String msg = "Invalid lookup class " + c.getName() + ". Cached class is: " + b.getClass().getName();
            logger.debug(msg);
            throw new IllegalStateException(msg);
        }
        if ((b == null && createIfNull) || (b != null && b.isDestroyed())) {
            if (b != null) {
                logger.debug("Removing destroyed Broadcaster {}", b.getID());
                store.remove(b.getID(), b);
            }
            if (store.putIfAbsent(id, createBroadcaster(c, id)) == null) {
                logger.debug("Added Broadcaster {} . Factory size: {}", id, store.size());
            }
            b = store.get(id);
        }
        return b;
    }

broadcasters are created and added to the store, but if they already located in the store under the same ID, threads are created and not added to the store, so when you destroy the factory you just terminate broadcasters in the store and the others (created by the testWebapp2) still alive, thus, the memory leak.

[jfarcand]

Fixed