Video does not return to the first frame after finishing

Bug #1324622 reported by Bill Filler
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Undecided
Unassigned
gallery-app
Fix Released
High
Arthur Mello
gallery-app (Ubuntu)
Fix Released
High
Unassigned
gallery-app (Ubuntu RTM)
Fix Released
High
Unassigned

Bug Description

when play back of video ends, we should show the preview (first framew) with the Play button overlayed (same as if you have just navigated to video from Photos view). Currently we show black screen after video ends.

Related branches

Bill Filler (bfiller)
Changed in gallery-app:
importance: Undecided → Medium
assignee: nobody → Arthur Mello (artmello)
status: New → Confirmed
tags: added: rtm14
Bill Filler (bfiller)
Changed in gallery-app:
assignee: Arthur Mello (artmello) → Ugo Riboni (uriboni)
Ugo Riboni (uriboni)
Changed in gallery-app:
status: Confirmed → In Progress
Revision history for this message
Ugo Riboni (uriboni) wrote :

In the gallery video player we load through a Loader the Video element (to save memory) and destroy it when browsing away from the video.

This somehow causes the media hub client not receiving the EndOfStream signal from the server, after the first time the Video component is re-created.

summary: - on video playback finished, return to the first frame
+ Video does not return to the first frame after finishing
Changed in gallery-app:
importance: Medium → High
Revision history for this message
Ugo Riboni (uriboni) wrote :

This problem is caused by a bug in the media hub client (which is used by qtubuntu-media which is the backend of the QML Video element). When an instance of the Video is destroyed and re-created the media hub client stops receiving events from the server.
This is reported in bug 1378311

Since we are using Video elements inside of ListView delegates, the ListView may destroy these elements at any time, unless we force all elements to stay in memory, which wouldn't be very good on a phone.

Revision history for this message
Sebastien Bacher (seb128) wrote :

@Ugo, the media hub issue has been fixed, gallery still display an empty screen after playing a video, is there work needed on that side as well?

Bill Filler (bfiller)
Changed in gallery-app (Ubuntu):
importance: Undecided → High
Changed in gallery-app (Ubuntu RTM):
importance: Undecided → High
tags: added: ota-future
Revision history for this message
Bill Filler (bfiller) wrote :

we are going to modify gallery-app to play back videos the same way as camera-app. That is, it will invoke mediaplayer app rather than try to play them inline. Once mediaplayer-app is made a trusted session the playback will be embedded in the gallery via the same mechanism.

Changed in gallery-app:
assignee: Ugo Riboni (uriboni) → Arthur Mello (artmello)
tags: added: ww07
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gallery-app - 0.0.67+15.04.20150202-0ubuntu1

---------------
gallery-app (0.0.67+15.04.20150202-0ubuntu1) vivid; urgency=medium

  [ Arthur Mello ]
  * Makes gallery-app to call MediaPlayer to play videos instead of
    playing by itself (LP: #1324622)
  * Make fullscreen mode dynamic when on device (LP: #1415140)

  [ Ubuntu daily release ]
  * New rebuild forced

gallery-app (0.0.67+15.04.20150126.2~rtm-0ubuntu1) 14.09; urgency=low

  [ Arthur Mello ]
  * Do not create objects if the file was removed from the time the
    create was requested and when it was started

  [ Florian Boucault ]
  * Do not create objects if the file was removed from the time the
    create was requested and when it was started

  [ Ubuntu daily release ]
  * New rebuild forced
 -- Ubuntu daily release <email address hidden> Mon, 02 Feb 2015 23:41:37 +0000

Changed in gallery-app (Ubuntu):
status: New → Fix Released
Bill Filler (bfiller)
Changed in canonical-devices-system-image:
milestone: none → ww09-2015
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gallery-app - 0.0.67+15.04.20150219~rtm-0ubuntu1

---------------
gallery-app (0.0.67+15.04.20150219~rtm-0ubuntu1) 14.09; urgency=medium

  [ Olivier Tilloy ]
  * Remove a useless runtime dependency. The dependency on
    Ubuntu.Components.Extras was removed at revision 1008 (on 2014-07-
    03) but debian/control wasn’t updated accordingly back then.

gallery-app (0.0.67+15.04.20150217-0ubuntu1) vivid; urgency=medium

  [ Olivier Tilloy ]
  * Fix failing autopilot tests that were broken by the changes
    introduced at revision 1143 (fix for bug #1416043). (LP: #1416043)
 -- CI Train Bot <email address hidden> Thu, 19 Feb 2015 09:10:33 +0000

Changed in gallery-app (Ubuntu RTM):
status: New → Fix Released
Changed in canonical-devices-system-image:
milestone: ww09-2015 → ww13-ota
status: New → Fix Released
Changed in gallery-app:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.