Various data classes in QtContacts have an unnessarily virtual destructor

These are:
QContactAbstractRequest
QContactDetail
QContactFilter

These are unnecessary because all subclasses of them are expected to hold all data in the private object (d pointer), which is cleaned up by the base class's destructor anyway.