Use a saner check for multimedia files in infopanel.

Don't rely on the reported supported mimetypes from Phonon,
Phonon supports more than just video and audio, but if it is
video or audio there's a very good chance Phonon supports it.

(If the mime type isn't in the list returned by Phonon there's
still a good chance it supports it, because mime types are stupid.)

BUG: 252539
FIXED-IN: 4.9.1

(cherry picked from commit 2fb4419de0)
This commit is contained in:
Martin Sandsmark 2012-08-09 03:27:11 +02:00 committed by Jekyll Wu
parent e534bc08f0
commit 0751d0b018

View file

@ -197,9 +197,7 @@ void InformationPanelContent::showItem(const KFileItem& item)
if (InformationPanelSettings::previewsShown()) {
const QString mimeType = item.mimetype();
const bool usePhonon = Phonon::BackendCapabilities::isMimeTypeAvailable(mimeType) &&
(mimeType != "image/png"); // TODO: workaround, as Phonon
// thinks it supports PNG images
const bool usePhonon = mimeType.startsWith("audio/") || mimeType.startsWith("video/");
if (usePhonon) {
m_phononWidget->show();
m_phononWidget->setUrl(item.targetUrl());