[Merge] lp:~albaguirre/media-hub/use-new-dbus-interface-for-display into lp:media-hub

Alberto Aguirre alberto.aguirre at canonical.com
Tue Jun 24 15:18:04 UTC 2014


Jim, 

Is it throwing an exception perhaps? The com.canonical.Unity.Screen DBus interface won't exist in the existing images (and why we are attempting to land all required changes at once)

The same would happen if there was no powerd BTW...

Perhaps the code that attempts to create sessions and call interfaces on those sessions should be surrounded by try-catch blocks so that it gracefully still plays videos even if it can't find the powerd/unity screen interfaces.

But I see that as a pre-existing issue though...perhaps addressed in a different MR?




-- 
https://code.launchpad.net/~albaguirre/media-hub/use-new-dbus-interface-for-display/+merge/223338
Your team Ubuntu Phablet Team is subscribed to branch lp:media-hub.



More information about the Ubuntu-reviews mailing list