diff options
author | Kari Oikarinen <kari.oikarinen@qt.io> | 2019-09-04 09:53:10 +0300 |
---|---|---|
committer | Kari Oikarinen <kari.oikarinen@qt.io> | 2019-09-04 13:33:54 +0300 |
commit | 114600e0a4ad6481d027bb7c4f206aac09a7bc13 (patch) | |
tree | 1ec6f8cd3f2ce27d82ff657eb27e8e8fb83c5120 | |
parent | 661cbf25b3ee924700ad09865dc902d3acbb5739 (diff) | |
parent | d1d6d2c4c497fef1a57b0f3689a2074331ed44c8 (diff) |
Merge "Merge dev into 5.14 (delayed final downmerge)"
-rw-r--r-- | src/animation/backend/buildblendtreesjob.cpp | 3 | ||||
-rw-r--r-- | src/extras/text/qdistancefieldglyphcache.cpp | 4 | ||||
-rw-r--r-- | src/render/picking/qobjectpicker.cpp | 2 | ||||
-rw-r--r-- | src/render/renderers/opengl/graphicshelpers/submissioncontext.cpp | 2 | ||||
-rw-r--r-- | src/render/renderers/opengl/textures/gltexture.cpp | 7 | ||||
-rw-r--r-- | tests/auto/extras/extras.pro | 3 | ||||
-rw-r--r-- | tests/auto/extras/qtext2dentity/qtext2dentity.pro | 13 | ||||
-rw-r--r-- | tests/auto/extras/qtext2dentity/qtext2dentity.qml | 93 | ||||
-rw-r--r-- | tests/auto/extras/qtext2dentity/qtext2dentity.qrc | 5 | ||||
-rw-r--r-- | tests/auto/extras/qtext2dentity/tst_qtext2dentity.cpp | 98 | ||||
-rw-r--r-- | tests/manual/downloading/main.qml | 2 |
11 files changed, 227 insertions, 5 deletions
diff --git a/src/animation/backend/buildblendtreesjob.cpp b/src/animation/backend/buildblendtreesjob.cpp index be96889fa..3cfa81446 100644 --- a/src/animation/backend/buildblendtreesjob.cpp +++ b/src/animation/backend/buildblendtreesjob.cpp @@ -86,7 +86,8 @@ void BuildBlendTreesJob::run() // Build the format for clip results that should be used by nodes in the blend // tree when used with this animator const ChannelMapper *mapper = m_handler->channelMapperManager()->lookupResource(blendClipAnimator->mapperId()); - Q_ASSERT(mapper); + if (!mapper) + continue; const QVector<ChannelNameAndType> channelNamesAndTypes = buildRequiredChannelsAndTypes(m_handler, mapper); const QVector<ComponentIndices> channelComponentIndices diff --git a/src/extras/text/qdistancefieldglyphcache.cpp b/src/extras/text/qdistancefieldglyphcache.cpp index 30db3ff4e..a3819b8fe 100644 --- a/src/extras/text/qdistancefieldglyphcache.cpp +++ b/src/extras/text/qdistancefieldglyphcache.cpp @@ -284,7 +284,9 @@ DistanceFieldFont* QDistanceFieldGlyphCache::getOrCreateDistanceFieldFont(const actualFont.setPixelSize(QT_DISTANCEFIELD_BASEFONTSIZE(useDoubleRes) * QT_DISTANCEFIELD_SCALE(useDoubleRes)); // create new font cache - DistanceFieldFont *dff = new DistanceFieldFont(actualFont, useDoubleRes, m_rootNode); + // we set the parent node to nullptr, since the parent node of QTextureAtlasses + // will be set when we pass them to QText2DMaterial later + DistanceFieldFont *dff = new DistanceFieldFont(actualFont, useDoubleRes, nullptr); m_fonts.insert(key, dff); return dff; } diff --git a/src/render/picking/qobjectpicker.cpp b/src/render/picking/qobjectpicker.cpp index 6dde055f6..9537d4982 100644 --- a/src/render/picking/qobjectpicker.cpp +++ b/src/render/picking/qobjectpicker.cpp @@ -111,6 +111,8 @@ namespace Qt3DRender { \sa PickingSettings, Geometry, Attribute, PickEvent, PickTriangleEvent, NoPicking + \note To receive hover events in QtQuick, the hoverEnabled property of Scene3D must also be set. + \note Instances of this component shouldn't be shared, not respecting that condition will most likely result in undefined behavior. */ diff --git a/src/render/renderers/opengl/graphicshelpers/submissioncontext.cpp b/src/render/renderers/opengl/graphicshelpers/submissioncontext.cpp index 028fd607a..bedc7f61c 100644 --- a/src/render/renderers/opengl/graphicshelpers/submissioncontext.cpp +++ b/src/render/renderers/opengl/graphicshelpers/submissioncontext.cpp @@ -589,7 +589,7 @@ QSize SubmissionContext::renderTargetSize(const QSize &surfaceSize) const } else { renderTargetSize = m_surface->size(); if (m_surface->surfaceClass() == QSurface::Window) { - int dpr = static_cast<QWindow *>(m_surface)->devicePixelRatio(); + const float dpr = static_cast<QWindow *>(m_surface)->devicePixelRatio(); renderTargetSize *= dpr; } } diff --git a/src/render/renderers/opengl/textures/gltexture.cpp b/src/render/renderers/opengl/textures/gltexture.cpp index 95f63480c..11d2c2be9 100644 --- a/src/render/renderers/opengl/textures/gltexture.cpp +++ b/src/render/renderers/opengl/textures/gltexture.cpp @@ -623,6 +623,13 @@ void GLTexture::uploadGLTextureData() void GLTexture::updateGLTextureParameters() { const QAbstractTexture::Target actualTarget = m_properties.target; + const bool isMultisampledTexture = (actualTarget == QAbstractTexture::Target2DMultisample || + actualTarget == QAbstractTexture::Target2DMultisampleArray); + // Multisampled textures can only be accessed by texelFetch in shaders + // and don't support wrap modes and mig/mag filtes + if (isMultisampledTexture) + return; + m_gl->setWrapMode(QOpenGLTexture::DirectionS, static_cast<QOpenGLTexture::WrapMode>(m_parameters.wrapModeX)); if (actualTarget != QAbstractTexture::Target1D && actualTarget != QAbstractTexture::Target1DArray && diff --git a/tests/auto/extras/extras.pro b/tests/auto/extras/extras.pro index bbdf118b3..e5ae28e97 100644 --- a/tests/auto/extras/extras.pro +++ b/tests/auto/extras/extras.pro @@ -6,5 +6,6 @@ contains(QT_CONFIG, private_tests) { qtorusgeometry \ qforwardrenderer \ qfirstpersoncameracontroller \ - qorbitcameracontroller + qorbitcameracontroller \ + qtext2dentity } diff --git a/tests/auto/extras/qtext2dentity/qtext2dentity.pro b/tests/auto/extras/qtext2dentity/qtext2dentity.pro new file mode 100644 index 000000000..1b8bd0c6f --- /dev/null +++ b/tests/auto/extras/qtext2dentity/qtext2dentity.pro @@ -0,0 +1,13 @@ +TEMPLATE = app + +TARGET = tst_qtext2dentity + +QT += 3dcore 3dcore-private 3dextras 3dquickextras qml testlib + +CONFIG += testcase + +SOURCES += \ + tst_qtext2dentity.cpp + +RESOURCES += qtext2dentity.qrc + diff --git a/tests/auto/extras/qtext2dentity/qtext2dentity.qml b/tests/auto/extras/qtext2dentity/qtext2dentity.qml new file mode 100644 index 000000000..bc699acf6 --- /dev/null +++ b/tests/auto/extras/qtext2dentity/qtext2dentity.qml @@ -0,0 +1,93 @@ +/**************************************************************************** +** +** Copyright (C) 2019 The Qt Company Ltd. +** Contact: https://www.qt.io/licensing/ +** +** This file is part of the Qt3D module of the Qt Toolkit. +** +** $QT_BEGIN_LICENSE:BSD$ +** Commercial License Usage +** Licensees holding valid commercial Qt licenses may use this file in +** accordance with the commercial license agreement provided with the +** Software or, alternatively, in accordance with the terms contained in +** a written agreement between you and The Qt Company. For licensing terms +** and conditions see https://www.qt.io/terms-conditions. For further +** information use the contact form at https://www.qt.io/contact-us. +** +** BSD License Usage +** Alternatively, you may use this file under the terms of the BSD license +** as follows: +** +** "Redistribution and use in source and binary forms, with or without +** modification, are permitted provided that the following conditions are +** met: +** * Redistributions of source code must retain the above copyright +** notice, this list of conditions and the following disclaimer. +** * Redistributions in binary form must reproduce the above copyright +** notice, this list of conditions and the following disclaimer in +** the documentation and/or other materials provided with the +** distribution. +** * Neither the name of The Qt Company Ltd nor the names of its +** contributors may be used to endorse or promote products derived +** from this software without specific prior written permission. +** +** +** THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS +** "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT +** LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR +** A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT +** OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, +** SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT +** LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, +** DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY +** THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT +** (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE +** OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE." +** +** $QT_END_LICENSE$ +** +****************************************************************************/ + +import Qt3D.Core 2.0 +import Qt3D.Render 2.0 +import Qt3D.Input 2.0 +import Qt3D.Extras 2.9 + +Entity { + id: sceneRoot + property alias text2DEntity: text + Camera { + id: camera + projectionType: CameraLens.PerspectiveProjection + fieldOfView: 45 + aspectRatio: 16/9 + nearPlane : 0.1 + farPlane : 1000.0 + position: Qt.vector3d( 0.0, 0.0, 140.0 ) + upVector: Qt.vector3d( 0.0, 1.0, 0.0 ) + viewCenter: Qt.vector3d( 0.0, 0.0, 0.0 ) + } + + OrbitCameraController { + camera: camera + } + + components: [ + RenderSettings { + activeFrameGraph: ForwardRenderer { + clearColor: Qt.rgba(1, 1, 1, 1) + camera: camera + } + }, + // Event Source will be set by the Qt3DQuickWindow + InputSettings { } + ] + + Text2DEntity { + id: text + text: "foo" + width: 200 + height: 40 + color: "darkgrey" + } +} diff --git a/tests/auto/extras/qtext2dentity/qtext2dentity.qrc b/tests/auto/extras/qtext2dentity/qtext2dentity.qrc new file mode 100644 index 000000000..424d200ae --- /dev/null +++ b/tests/auto/extras/qtext2dentity/qtext2dentity.qrc @@ -0,0 +1,5 @@ +<RCC> + <qresource> + <file>qtext2dentity.qml</file> + </qresource> +</RCC> diff --git a/tests/auto/extras/qtext2dentity/tst_qtext2dentity.cpp b/tests/auto/extras/qtext2dentity/tst_qtext2dentity.cpp new file mode 100644 index 000000000..6fcc2e637 --- /dev/null +++ b/tests/auto/extras/qtext2dentity/tst_qtext2dentity.cpp @@ -0,0 +1,98 @@ +/**************************************************************************** +** +** Copyright (C) 2019 The Qt Company Ltd. +** Contact: https://www.qt.io/licensing/ +** +** This file is part of the Qt3D module of the Qt Toolkit. +** +** $QT_BEGIN_LICENSE:BSD$ +** Commercial License Usage +** Licensees holding valid commercial Qt licenses may use this file in +** accordance with the commercial license agreement provided with the +** Software or, alternatively, in accordance with the terms contained in +** a written agreement between you and The Qt Company. For licensing terms +** and conditions see https://www.qt.io/terms-conditions. For further +** information use the contact form at https://www.qt.io/contact-us. +** +** BSD License Usage +** Alternatively, you may use this file under the terms of the BSD license +** as follows: +** +** "Redistribution and use in source and binary forms, with or without +** modification, are permitted provided that the following conditions are +** met: +** * Redistributions of source code must retain the above copyright +** notice, this list of conditions and the following disclaimer. +** * Redistributions in binary form must reproduce the above copyright +** notice, this list of conditions and the following disclaimer in +** the documentation and/or other materials provided with the +** distribution. +** * Neither the name of The Qt Company Ltd nor the names of its +** contributors may be used to endorse or promote products derived +** from this software without specific prior written permission. +** +** +** THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS +** "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT +** LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR +** A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT +** OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, +** SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT +** LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, +** DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY +** THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT +** (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE +** OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE." +** +** $QT_END_LICENSE$ +** +****************************************************************************/ + +#include <QtTest/QTest> +#include <Qt3DExtras/QText2DEntity> +#include <Qt3DQuickExtras/qt3dquickwindow.h> +#include <Qt3DQuick/QQmlAspectEngine> +#include <Qt3DCore/private/qnode_p.h> + +QVector<Qt3DCore::QNode *>lookupNodeByClassName(Qt3DCore::QNode *root, const QString &className) +{ + auto children = root->childNodes(); + QVector<Qt3DCore::QNode *> childrenList; + for (auto i : children) { + if (i->metaObject()->className() == className) + childrenList << i; + else + childrenList << lookupNodeByClassName(i, className); + } + return childrenList; +} + +class tst_qtext2dentity : public QObject +{ + Q_OBJECT + +private slots: + void checkChangeArbiter(); +}; + +void tst_qtext2dentity::checkChangeArbiter() +{ +#if !defined(QT_OPENGL_ES_2_ANGLE) + // GIVEN + Qt3DExtras::Quick::Qt3DQuickWindow view; + view.setSource(QUrl("qrc:/qtext2dentity.qml")); + view.show(); + QCoreApplication::processEvents(); + + // THEN + auto rootEntity = view.engine()->aspectEngine()->rootEntity(); + QVERIFY(rootEntity != nullptr); + auto atlases = lookupNodeByClassName(rootEntity.data(), "Qt3DExtras::QTextureAtlas"); + QVERIFY(atlases.length() == 1); + auto atlas = atlases[0]; + QTRY_VERIFY(Qt3DCore::QNodePrivate::get(atlas)->m_changeArbiter); +#endif +} + +QTEST_MAIN(tst_qtext2dentity) +#include "tst_qtext2dentity.moc" diff --git a/tests/manual/downloading/main.qml b/tests/manual/downloading/main.qml index a78cf605e..f8631e9b1 100644 --- a/tests/manual/downloading/main.qml +++ b/tests/manual/downloading/main.qml @@ -81,7 +81,7 @@ Entity { Mesh { id: mesh - source: "https://codereview.qt-project.org/gitweb?p=qt/qt3d.git;a=blob_plain;hb=refs/heads/dev;f=examples/qt3d/exampleresources/assets/chest/Chest.obj" + source: "https://codereview.qt-project.org/gitweb?p=qt/qt3d.git;a=blob_plain;f=examples/qt3d/exampleresources/assets/obj/plane-10x10.obj" onStatusChanged: console.log("Mesh status " + status) } |