diff options
author | Lars Knoll <lars.knoll@qt.io> | 2021-04-26 14:09:48 +0200 |
---|---|---|
committer | Lars Knoll <lars.knoll@qt.io> | 2021-04-28 08:46:00 +0000 |
commit | 2dceee374cad71ad204f04acd967cc939fa93b5f (patch) | |
tree | 9974b7210a0181935da7388d7f3430c5992ff8e8 /src/multimediawidgets/qvideowidget.cpp | |
parent | 14f960eeb6bf99e6de4bfd41b0d5a6bf2de2fe67 (diff) |
Fix sizeHint() of QVideoWidget/QVideoGraphicsItem
Doesn't yet fix all test failures, but it's an
improvement.
Change-Id: I775857f1fb580b78ae137dc71ee445a15eafd0a7
Reviewed-by: Doris Verria <doris.verria@qt.io>
Reviewed-by: Lars Knoll <lars.knoll@qt.io>
Diffstat (limited to 'src/multimediawidgets/qvideowidget.cpp')
-rw-r--r-- | src/multimediawidgets/qvideowidget.cpp | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/src/multimediawidgets/qvideowidget.cpp b/src/multimediawidgets/qvideowidget.cpp index eebda495b..ff7b488d6 100644 --- a/src/multimediawidgets/qvideowidget.cpp +++ b/src/multimediawidgets/qvideowidget.cpp @@ -308,11 +308,13 @@ void QVideoWidget::setSaturation(float saturation) */ QSize QVideoWidget::sizeHint() const { -// Q_D(const QVideoWidget); + Q_D(const QVideoWidget); -// ##### -// if (d->video) -// return d->backend->sizeHint(); + if (d->videoSink) { + auto size = d->videoSink->videoSize(); + if (size.isValid()) + return size; + } return QWidget::sizeHint(); } |