diff options
author | Michal Klocek <michal.klocek@qt.io> | 2021-10-14 10:27:53 +0200 |
---|---|---|
committer | Michal Klocek <michal.klocek@qt.io> | 2022-02-17 09:36:44 +0100 |
commit | 82357352b49fdcba80727feec441053f676b5af2 (patch) | |
tree | b61ec1a88445c5d8f5edb428cf573dfddd0986a7 /tests/auto/quick | |
parent | 35a1ffb234ef5dd32c00a30c7ab6236c926da8f9 (diff) |
Introduce "--webEngineArgs" to prevent unexpected sub-process args
If there is no QTWEBENGINE_CHROMIUM_FLAGS env export we pass all args to
Chromium as "command line". However passing "-type=1" to qt application
with webenigne ends will crash since this arg is used for
sub-process type handling.
Prevent any accidental user args which can affect chromium switches
and from now on pass only args explicitly stated after "--webEngineArgs" option.
[ChangeLog] Command line arguments meant for webengine has to be now
stated after "--webEngineArgs" option.
Fixes: QTBUG-68820
Change-Id: I57b3921d2250f75d445ee7173566dadc41e4ca93
Reviewed-by: Michael BrĂ¼ning <michael.bruning@qt.io>
Diffstat (limited to 'tests/auto/quick')
-rw-r--r-- | tests/auto/quick/qmltests/tst_qmltests.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/auto/quick/qmltests/tst_qmltests.cpp b/tests/auto/quick/qmltests/tst_qmltests.cpp index 967cabfec..36899f78a 100644 --- a/tests/auto/quick/qmltests/tst_qmltests.cpp +++ b/tests/auto/quick/qmltests/tst_qmltests.cpp @@ -255,7 +255,7 @@ int main(int argc, char **argv) // Force to use English language for testing due to error message checks QLocale::setDefault(QLocale("en")); - static QByteArrayList params = {QByteArrayLiteral("--use-fake-device-for-media-stream")}; + static QByteArrayList params = {QByteArrayLiteral("--webEngineArgs"),QByteArrayLiteral("--use-fake-device-for-media-stream")}; QList<const char *> w_argv(argc); for (int i = 0; i < argc; ++i) w_argv[i] = argv[i]; for (int i = 0; i < params.size(); ++i) w_argv.append(params[i].data()); |