VideoOutput plays as soon as loaded even if the MediaPlayer is not set to autoPlay

Bug #1457972 reported by Nick Dedekind
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Medium
Pat McGowan
qtubuntu-media (Ubuntu)
Fix Released
Undecided
Canonical Phone Foundations
qtubuntu-media (Ubuntu RTM)
Fix Released
Undecided
Jim Hodapp

Bug Description

When using qml components (not sure about Qt)

MediaPlayer { id: mp; source: "test-file.ogg"; autoPlay: false }
VideoOutput { source: mp }

The media plays as soon as the media is loaded even though the autoPlay is set to false.
There is a play command going out as soon as we get a glConsumerSet signal.

Related branches

Jim Hodapp (jhodapp)
Changed in qtubuntu-media (Ubuntu):
assignee: nobody → Canonical Phone Foundations (canonical-phonedations-team)
Revision history for this message
Jim Hodapp (jhodapp) wrote :

I tried an example like this to reproduce it with no luck:

import QtQuick 2.0
import QtQuick.Window 2.0
import QtMultimedia 5.0
import Ubuntu.Unity.Action 1.1 as UnityActions
import Ubuntu.Components 1.1
import Ubuntu.Components.Popups 1.0 as Popups

import QtQuick 2.0
import QtMultimedia 5.0

Text {
    text: "Should NOT auto-play media";
    font.pointSize: 48;
    width: 150; height: 50;

    MediaPlayer {
        id: playMusic
        autoPlay: false
        source: "/home/phablet/Music/testfile.ogg"
    }
    MouseArea {
        id: playArea
        anchors.fill: parent
        onPressed: { playMusic.play() }
    }
}

Revision history for this message
Jim Hodapp (jhodapp) wrote :

Nevermind, was testing audio only and misread the bug that this should be video-based.

Changed in qtubuntu-media (Ubuntu):
status: New → In Progress
Revision history for this message
Jim Hodapp (jhodapp) wrote :

Updated test QML that confirmed the bug:

import QtQuick 2.0
import QtQuick.Window 2.0
import QtMultimedia 5.0
import Ubuntu.Unity.Action 1.1 as UnityActions
import Ubuntu.Components 1.1
import Ubuntu.Components.Popups 1.0 as Popups

Text {
    text: "Should NOT auto-play media";
    font.pointSize: 20;
    width: 150; height: 50;

    MediaPlayer {
        id: playVideo
        autoPlay: false
        source: "/home/phablet/Videos/testfile.mp4"
    }
    VideoOutput { source: playVideo }
    MouseArea {
        id: playArea
        anchors.fill: parent
        onPressed: { playVideo.play() }
    }
}

Jim Hodapp (jhodapp)
Changed in qtubuntu-media (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qtubuntu-media - 0.7.1+15.10.20150602.3-0ubuntu1

---------------
qtubuntu-media (0.7.1+15.10.20150602.3-0ubuntu1) wily; urgency=medium

  [ CI Train Bot ]
  * New rebuild forced.

  [ Jim Hodapp ]
  * Don't auto play when playing a video unless
    QMediaPlayerControl::play() was actually called. (LP: #1457972)

 -- CI Train Bot <email address hidden> Tue, 02 Jun 2015 19:37:22 +0000

Changed in qtubuntu-media (Ubuntu):
status: Fix Committed → Fix Released
Jim Hodapp (jhodapp)
Changed in qtubuntu-media (Ubuntu RTM):
status: New → In Progress
assignee: nobody → Jim Hodapp (jhodapp)
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

This bug was fixed in the package qtubuntu-media 0.7.1+15.04.20150828-0ubuntu1 in https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/stable-phone-overlay

---------------

qtubuntu-media (0.7.1+15.04.20150828-0ubuntu1) vivid; urgency=medium

  [ Jim Hodapp ]
  * Bump version for vivid sync

  [ CI Train Bot ]
  * New rebuild forced.

qtubuntu-media (0.7.1+15.04.20150710-0ubuntu1) vivid; urgency=medium

  [ CI Train Bot ]
  * New rebuild forced.

  [ Jim Hodapp ]
  * Make sure video can play a different media source using the same
    player instance and not produce a black screen.

qtubuntu-media (0.7.1+15.04.20150602.3-0ubuntu1) vivid; urgency=medium

  [ CI Train Bot ]
  * New rebuild forced.

  [ Jim Hodapp ]
  * Don't auto play when playing a video unless
    QMediaPlayerControl::play() was actually called. (LP: #1457972)

qtubuntu-media (0.7.1+15.04.20150602.1-0ubuntu1) vivid; urgency=medium

  [ CI Train Bot ]
  * New rebuild forced.

  [ Jim Hodapp ]
  * Get rid of the gcc-4.9 build dependency. (LP: #1452340)

 -- CI Train Bot <email address hidden> Fri, 28 Aug 2015 13:52:24 +0000

Changed in qtubuntu-media (Ubuntu RTM):
status: In Progress → Fix Released
Changed in canonical-devices-system-image:
assignee: nobody → Pat McGowan (pat-mcgowan)
importance: Undecided → Medium
milestone: none → ww40-2015
status: New → Fix Released
status: Fix Released → Fix Committed
Changed in canonical-devices-system-image:
status: Fix Committed → 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.