This reverts commit 1463e7cb.

Conflicts:

	modules/gui/qt4/components/controller.cpp
	modules/gui/qt4/components/controller_widget.hpp

According to https://bugzilla.mozilla.org/show_bug.cgi?id=455458
npupp.h has been renamed to npfunctions.h with libxul 1.9.1
Signed-off-by: Christophe Mutricy <xtophe@videolan.org>

 * use psz_name when an item doesn't have a title
 * proper locking
 * little speed up

(thanks fenrir for the reviews)

Now, this should work safely, although it requires heavy polling.

This API is completely idiotic. By design, it requires (very active)
polling to work. Also, it is almost impossible to make it thread-safe.
Anyway, through violating a few constant pointers, we can fix half of
that. However, message item reference counting remains broken.

It was not thread-safe, and only used by ncurses (fixed earlier).

Changing the verbosity at that point is not thread-safe (and trying to
restore the value at ext is somwehat logically flawed). Luckily all the
standard <stdio.h> functions are thread-safe.

Signed-off-by: Rémi Denis-Courmont <rdenis@simphalempin.com>

Remove this define once the functionality is implemented on MacOS X and Win32.