Change-Id: I4ebeec7ac72152c134dc87c8d69d0f4c2f0d3cfc

Task-number: QTBUG-62715
Change-Id: I6eaa7d8e697de147533090242a95479d6b81463f

Change-Id: I2cdf41d3d0c60774a9b5cb48eef3ee7850b2cadf
Reviewed-by: Christian Stromme <christian.stromme@qt.io>
Reviewed-by: Andy Shaw <andy.shaw@qt.io>

In QAlsaAudioInput::open(), one should check against the available
audio inputs and not the audio outputs.

Regression introduced in 5473621b

.

Task-number: QTBUG-62435
Change-Id: I5db0a092a98c17964c55c9358ea1010ca05e9d6f
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

Task-number: QTBUG-52673
Change-Id: Ia093b2836b0f137e06624f557f4ee1746726e3e3
Reviewed-by: Simon Hausmann <simon.hausmann@qt.io>

The tests for boot2qt were disabled with commit
b9bfe241

. The tests can
be enabled since qtdeclarative now fallbacks to software
renderer if OpenGL is not supported.

Two changes were needed to pass tests on boot2qt
1. Add option to disable tests requiring WAV format and use
that in the boot2qt environment
2. Skip some tst_QPainterVideoSurface tests if GLContext is
not supported by the platform

Task-number: QTBUG-60268
Change-Id: I236207fedb505124a2c7edf0caccd85011ab1416
Reviewed-by: Simon Hausmann <simon.hausmann@qt.io>

Change-Id: I0e0b41d2c3539b46b6464812c3910e54cb966ee3

Since the slot connected to the frameAvailable signal may be in another
thread there is a bigger risk of it being invalid by the time the slot
is invoked. Therefore we copy the data and emit with the copy to ensure
that we don't lose the data.

Task-number: QTBUG-61817
Change-Id: I2888661d8a7f97105a85f87b08cc9ec25f8ce8c7
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

This is the result of running the (experimental) clang-tidy check
   qt-modernize-qsharedpointer-create

Discarded changes: none.

This run probably does not cover all of qtmultimedia.

Change-Id: I86a51b6a1edfe254f3fa9bff12dda359ac413df0
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Reviewed-by: Lars Knoll <lars.knoll@qt.io>

Change-Id: I8d96dea9955d4c749b99fffd14cde44ebf1f7290
Reviewed-by: Jędrzej Nowacki <jedrzej.nowacki@qt.io>

By default pa_context_connect() tries to spawn a new pulseaudio
instance if it the connection fails. This is not always desirable.

Add a new environment variable: QT_PULSE_NOAUTOSPAWN which - when
set - will make QSoundEffect pass the PA_CONTEXT_NOAUTOSPAWN flag
to pa_context_connect() thus disabling the autospawn feature.

Change-Id: I623639fd5a262c8fefbd683fa9481916e79bdf0e
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

Currently we call pa_context_connect() with an empty server string.
This works fine if we only want to connect to the default pulseaudio
server, but we may want to specify the server string on systems with
custom configuration.

Add a new environment variable: QT_PULSE_SERVER_STRING which allows to
specify the server string passed to pa_context_connect().

Change-Id: I6805ff5e941d13bc91d306e5df3ab5d04a8a90af
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

Change-Id: Ifda1f02b8458a4fbe85d8bc27e61d4646147d82c

Change-Id: I979c7d8d01b00689269a7969b628864ca5eab991
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>

It have the virtue of documenting its meaning directly.

Change-Id: I42f1be6b4c8e69dcf8b8711b42b19759a6aebe78
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>

QSoundEffect (pulseaudio) uses a workaround for stability issues:
although the pulseaudio mainloop mutex is recursive, it utilizes a
separate lock counting. This is not the best solution, but it is a
part of a larger set of changes which improved the stability in CI
under heavy load.

QSoundEffect always calls pa_threaded_mainloop_lock/unlock() from the
same thread so the additional lock counting works in normal situation
even though it doesn't use atomic types.

However if pa_context_connect() fails, pa_threaded_mainloop_unlock()
is called without regard to current lock count. This leads to random
double-unlock aborts if pa_context_connect() fails more than once
(e.g. after the reconnect scheduled from onContextFailed()).

Fix this by always using the PulseDaemon wrappers around
pa_threaded_mainloop_lock/unlock().

Task-number: QTBUG-61725
Change-Id: I41eb9a76892a6646fd5620ef8f686473b339464f
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

Conflicts:
	.qmake.conf

Change-Id: I5f9bed6cb611471db61b9c69004f16786822625d

Change-Id: Ib4f0ad04f54dca3c71fe09a25629d2c5b604ca5b

Change-Id: Ifcf6bf24ffe1e1762796ff330f6ae1357aa83870

Change-Id: Icd0e0d4b27cb4e5eb892fffd14b5285d43f4afbf
Reviewed-by: Lars Knoll <lars.knoll@qt.io>

Binary compatibility files updated.

Change-Id: Idfb4d256ad582a601538b584dcddb71dce480227
Reviewed-by: Sergio Ahumada <sahumada@texla.cl>

We only need to wait for a new viewfinder frame if we have a
videorenderer control. If there's no videorender control, then there's
no preview set and we can therefore just save the capture image
immediately.

Task-number: QTBUG-60329
Change-Id: I9ba34919f7cd82258482507c65db6367e330e231
Reviewed-by: Christian Stromme <christian.stromme@qt.io>
Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>

Use Qt 5 signal/slot syntax, range based for, member initialization.
Add message for captured images and fix some minor issues.

Task-number: QTBUG-60627
Change-Id: I1011fc95d928a9d2edaad16120acb0dc41f987d9
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

Task-number: QTBUG-60627
Change-Id: Ib7268d2b939614e713ea6ac209f1c162002d6142
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

It's possible to initiate an asynchronous image capture
(via AVCaptureStillImageOutput object) and immediately
stop AVCaptureSession. In this case image capture's callback
can potentially block forever, waiting for a semaphore: (if)
no new frames arrive after 'stop' (on whatever thread
AVFoundation/GDC chooses), this semaphore is never released.
To avoid this we try to acquire a semaphore with a (reasonable)
timeout and report an error in case of failure. To make sure
we are not leaking a semaphore and not creating a danling pointer,
we use a QSharedPointer now.

Task-number: QTBUG-61367
Change-Id: I208cd463f843bc807b53b23ac9651aab0382775a
Reviewed-by: Christian Stromme <christian.stromme@qt.io>
Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>

Change-Id: Ife0ddc6788ddfe64e6e27e5b7403064cddcf636a
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

Fixes g++ warnings:
..\common\evr\evrhelpers.cpp: In function 'QVideoFrame::PixelFormat qt_evr_pixelFormatFromD3DFormat(D3DFORMAT)':
..\common\evr\evrhelpers.cpp:143:5: warning: case value '842094158' not in enumerated type 'D3DFORMAT {aka _D3DFORMAT}' [-Wswitch]
     case D3DFMT_NV12:
..\common\evr\evrhelpers.cpp:145:5: warning: case value '842094169' not in enumerated type 'D3DFORMAT {aka _D3DFORMAT}' [-Wswitch]
     case D3DFMT_YV12:

Change-Id: I11869144b34d848bd68434d3a062852608518978
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

Fix g++ 7.1 warning:
player\directshowplayerservice.cpp:724:9: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
         if (SUCCEEDED(DirectShowUtils::connectFilters(m_graph, m_source, m_videoSampleGrabber->filter(), true)))

player\directshowplayerservice.cpp:726:13: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if'
             break;

Amends 8e4d966f

.

Change-Id: I491e14e0116140d2ebb870441c2f048a2d7ddc31
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

Change-Id: I9f764077d257dadfe1641e9bd446b6939c381173
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

Though it's a pointer.

Task-number: QTBUG-60464
Change-Id: Ia3e896da908f42939148fffd14c660fd8da43aa2
Reviewed-by: Maurice Kalinowski <maurice.kalinowski@qt.io>

When m_loopCount is Infinite (i.e. -1) m_runningCount would go from -2
and down, eventuall wrapping and reaching 0 and thus stopping.

This is a theorical problem since even if we had a 1 second video it would
take lots of years to trigger but i guess it's better than
having the value of m_runningCount be unbounded

Change-Id: I340ec2157ece3334e58ab4afd77c34b478a256e9
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

Sometimes m_format.bytesForDuration(delta) is larger than the QByteArray
containing the data. Therefore select the minimum of these.

Task-number: QTBUG-61085
Change-Id: I20a8ffb77095aecd2711f53d4c9fc7da7f9ae8e2
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

This patch moves the QGstTools lib to a private Qt module and marks
the needed classes and functions with Q_DECL*, to improve
cross-platform abilities. It's the first step to use the GStreamer
backend on macOS and Windows platform.

Change-Id: Idc23c72bcccb52dd501e169e9dfdc227856787fa
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

Conflicts:
	.qmake.conf

Change-Id: I20470ed193e818eefedd975d89f755fb9d673241

Change-Id: I5e0887cd2b1837d1e3de93bbabd751212488ae0a

6.6.0 and 7.0.0 mm-renderer don't appear to produce spurious stopped
state updates.  From the MMR_STOPPED in one of the comments, it looks
like this was a problem with the BlackBerry mm-renderer.

Change-Id: Ia336195b48df342c376b7fb3c8eb2a65ef748032
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

It was only used by the BlackBerry code which is gone.

Change-Id: I9311824b643c3c97205e1f13656ea3a2eeebfe7a
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

Used wrong class name.  Probably a Copy-FailToChange.

Get rid of a warning about the conversion specification that didn't
match argument type.

Change-Id: I4f24b737dd5f72cc1dc61c58de6ad21d9a07c8ed
Reviewed-by: Dan Cape <dcape@qnx.com>
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

Where possible, we should be emitting the stateChanged() signal to
StoppedState when we know the file is no longer being written to. The
finializing status can be used to indicate it is finishing and when it
is actually finished then StoppedState should be used.

Task-number: QTBUG-50588
Change-Id: Ie3ac1c5cd00a6a36978e72b5485622e3302054ce
Reviewed-by: Christian Stromme <christian.stromme@qt.io>

Change-Id: I1d646cf7d8f68a7ec5c713735c233c71bc3f338b
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>