don't initialize a string to empty using "", this actually creates a string of size 0 having the '\0' char.
leave it as QString() *or* do nothign, it's correctly default initialized.

this looks like a debug code that was forgot.

hm? why would you do this?

use qCDebug

stray space.

the order of the emits matter?
if not one can save the pointers and the url and only call emit once per method.

This should have been removed durring previous merge

I will remove this and use its default.

This is part of D12719
Its creating a QUrlList from the QVariant that the QML drop target provides.

This area needs code still. Here we should be clearing the view. The Debug and space will go when finished.