[buteo-mtp] Fix build break

Buteo SyncDBusConnection is no more
Merge pull request #55 from monich/build-fix

Fix build break
[tests] Added storagefactor-test to tests.xml

Also removed some inaccurate comments.
It would be nice to change the suite and set names to reflect the
new package name, but that would mess with the continuity of
test results for no good reason.
[deviceinfo] initialize m_batteryLevel

issue found by valgrind
[tests] don't use global QEventLoop in protocol_test

QEventLoop complains if it is constructed before QApplication,
and this may be involved in a hard-to-reproduce segfault in
initTestCase. Move it into the function that uses it.
[tests] Initialize m_opcode in mtpresponder_test

found by valgrind
[fsstorageplugin] Initialize the cached inotify event

Class member initialization was missing, found with valgrind.
This was causing crashes in the autotests.
[fsstorageplugin] pass ObjHandle by value

Changed methods removeFromStorage and deleteItemHelper to take
their ObjHandle by value instead of by reference, on the grounds
that they're going to remove that handle from some maps and this
is highly awkward if the calling function passed in a reference
taken from those maps.

Specifically, the autotests found an error where handleFSMove
passes m_pathNamesMap[oldPath] as the argument, and the []
operator returns a reference to the map element, which is
then deleted in removeFromStorage.

This is the kind of subtle bug that makes C++ so beloved.