Merge lp:~phablet-team/qtubuntu-media/fix-1533779 into lp:qtubuntu-media

Proposed by Jim Hodapp
Status: Merged
Approved by: Bill Filler
Approved revision: 99
Merged at revision: 99
Proposed branch: lp:~phablet-team/qtubuntu-media/fix-1533779
Merge into: lp:qtubuntu-media
Diff against target: 14 lines (+4/-0)
1 file modified
src/aal/aalmediaplayerservice.cpp (+4/-0)
To merge this branch: bzr merge lp:~phablet-team/qtubuntu-media/fix-1533779
Reviewer Review Type Date Requested Status
Bill Filler (community) Approve
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+282656@code.launchpad.net

Commit message

Make sure to catch and report the exception InsufficientPermissionsToAddTrack

Description of the change

Make sure to catch and report the exception InsufficientPermissionsToAddTrack

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Bill Filler (bfiller) wrote :

works

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/aal/aalmediaplayerservice.cpp'
2--- src/aal/aalmediaplayerservice.cpp 2015-12-18 15:38:48 +0000
3+++ src/aal/aalmediaplayerservice.cpp 2016-01-14 20:02:26 +0000
4@@ -318,6 +318,10 @@
5 try {
6 m_hubPlayerSession->open_uri(uri);
7 }
8+ catch (const media::Player::Errors::InsufficientAppArmorPermissions &e) {
9+ qWarning() << e.what();
10+ return;
11+ }
12 catch (const std::runtime_error &e) {
13 qWarning() << "Failed to set media " << url << ": " << e.what();
14 return;

Subscribers

People subscribed via source and target branches

to all changes: