EngineController: ditch canDecode()
authorMatěj Laitl <matej@laitl.cz>
Fri, 13 Jul 2012 11:05:33 +0000 (13:05 +0200)
committerMatěj Laitl <matej@laitl.cz>
Thu, 19 Jul 2012 13:22:30 +0000 (15:22 +0200)
commit712873769dda415c6a21737428bd1a5305018271
tree001f76b1c66e435e65960b52284b6c5fcd5cf4ac
parenta8a57b9cff66e944bb36b4b4702b337a68105aa5
EngineController: ditch canDecode()

MetaFile::Track::isTrack() is partial replacement. Existing 3 calls to
canDecode() were in fact all related to MetaFile classes, so move the
method there, simplify it not to query phonon at all (and document it
can return false positives).

As a consequence, we show all audio and video files in file browser and
in other places, even if they wouldn't be playable by the current
phonon back-end. This is arguably a cleaner approach and at least lets
users discover where the error is.

Works quite well for me and prevents failures in many tests. This
change is propelled by Bart's and Ralf's legitimate comments on
http://git.reviewboard.kde.org/r/105524/

BUG: 303253
FIXED-IN: 2.6
ChangeLog
src/EngineController.cpp
src/EngineController.h
src/browsers/filebrowser/FileBrowser.cpp
src/browsers/filebrowser/FileView.cpp
src/core-impl/collections/support/CollectionManager.cpp
src/core-impl/meta/file/File.cpp
src/core-impl/meta/file/File.h