Change-Id: I2d04e910c7a08e1c630bc490d1f8b1be3d27a6f1
Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>

Error page should be disabled while testing loadFail signal since
testing signals of the internal error page is not possible yet.

Change-Id: I07f0bb6378fcbf5d18634197808801c94606803c
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

Remove QQuickWebEngineSettingsPrivate object's pointer from the global
setting list before the object is destructed.

Change-Id: I1cca6bf8aa2438903692d9e2ef863c8fc543adbd
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

There is currently no convenient way to deploy QtWebEngine into an application
bundle on OSX. macdeployqt copies frameworks into a .app bundle's Frameworks
directory but this makes no sense unless all the needed files are also
distributed with the bundle.

This patch moves:
- The ffmpegsumo.so library into Libraries/
- Locale .pak files, qtwebengine_resources.pak and icudtl.dat into Resources/
- QtWebEngineProcess into its own .app bundle, itself into Helpers/

QMAKE_BUNDLE_DATA is used to copy files into the bundle while INSTALLS is
used when installing normally. A LOCALE_LIST is explicitly listed since
QMAKE_BUNDLE_DATA can't handle the * glob to match all source files.

Change-Id: I5c0df57b4b9e93f9cce34a74a6e024bf90d37b5c
Task-number: QTBUG-41611
Reviewed-by: Zeno Albisser <zeno.albisser@digia.com>
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

core_module.pro is the one responsible for deployment of the binary so
it makes sense to also let it handle the other files to install.

This leaves core.pro to only handle sub-projects dependencies.

Change-Id: Ibe433eceec7fb0a9f806dfa89dccb0bedeaccc59
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

This avoids using a General matrix for tile transformations which are
usually only translations in our case.

Change-Id: Ic947d7ef9b78de6d5d16ea718cc03255e38a8097
Reviewed-by: Michael Bruning <michael.bruning@digia.com>
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

This should help users understand which prerequisites are
missing from their system and may also help find invalid CI
configuration quicker.

Change-Id: If564825c11782b6e760f016893010008d9a551fa
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

Adds a test to check the return types of runJavaScript.

Change-Id: Ifd0cb4adf57c972e8a8ad240d51f11f0cc17aa3d
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

Change-Id: I935ee10d10248ab525aca31c1b1c04d2927915c1
Reviewed-by: Michael Bruning <michael.bruning@digia.com>

The Windows command shell try to interpret the single quotation mark
as a command.

Change-Id: I4dc4382bdbe0c90565f6ac56c33a80971669344c
Reviewed-by: Andras Becsi <andras.becsi@digia.com>
Reviewed-by: Michael Bruning <michael.bruning@digia.com>

QtWebEngine::initialize happens after QGuiApplication instantiation,
so it is too late to simply set the application attribute.
In that scenario, we don't want to override the shared context and risk
a double free when both QGuiApplication and our cleanup handler try
to delete it.

Change-Id: Id52884abbb0c2380208876d9c00e8ddbfbc21eda
Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

Use explicit url in quick API too and use GetLastCommittedURL instead of
GetVisibleURL for getting the expected urls in tests.

Change-Id: If3251323645979643f3dc0e16491fe35b584f51c
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

This fixes 7639513f

.

Change-Id: I512ca97d8b4fc8b7c48f03ab32d476bb3a791157
Reviewed-by: Pierre Rossi <pierre.rossi@gmail.com>
Reviewed-by: Zeno Albisser <zeno.albisser@digia.com>

Adapt test results to the chromium behavior and remove expected
failures.

Change-Id: I277462125822e50f84c0d0ae795846a71c9a2b10
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

The GCC version check is only run on Linux.
Change-Id: I524508d763005190764526ce337dc348e5ba4fcb
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>

Chromium is only supported on ARMv6 and ARMv7 for ARM architecture.

Change-Id: I0493c9ba2baf4a537f759286622056093c0d4a25
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

The build did not work when configuring to linux-g++-64,
although the platform is supported.

Task-number: QTBUG-41918
Change-Id: If2c8a5722d7d6b0ca28a99763531d8ffd9534de7
Reviewed-by: Andras Becsi <andras.becsi@digia.com>
Reviewed-by: Zeno Albisser <zeno.albisser@digia.com>

This prevents the crash when a webpage requests geolocation permission.

Change-Id: I7331b869ec1439d678cee7941691289220718f3d
Reviewed-by: Zeno Albisser <zeno.albisser@digia.com>

The snapshot ninja may cause issues in some environments, where a different
ninja version may work. If the user has NINJA_PATH set in their environment,
prefer that over bootstrapping our own.

Change-Id: I4c6d2f479aeae1f5137705ed2deb995b59ea287a
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

Change-Id: I91ce455ac5e3992e50d13c14f4035f91baad85bf
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

Task-number: QTBUG-41952
Change-Id: Ic21089a34364696c8ff7e68e397e39f8f0b7f5cc
Reviewed-by: Zeno Albisser <zeno.albisser@digia.com>

This fixes cross-compile settings detection for cross-compilation
environments where the C compiler cross-compilation flags are set in
CC/QMAKE_CC instead of CFLAGS/QMAKE_CFLAGS (e.g. OpenEmbedded).

Change-Id: I0a7b0f6c47a00775ec18073a16359c8351363285
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

Change-Id: I972d3560d6aa2640a2702f219cf802d2476b1c6e
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

That code was borrowed straight from Chromium and used the Chromium
coding style.

Change-Id: I212aea23f19056b901cbc770640faed4998f02fd
Reviewed-by: Zeno Albisser <zeno.albisser@digia.com>

This is required so that embedders can implement custom error pages,
and is also needed for tests, where error-page loading has side-effects
on subsequent page loads.

Change-Id: I5ae276a58864a2fa9d7b608bea3340b482a92f4e
Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>

This should be removed once desktop notifications
have been implemented.

Task-number: QTBUG-41893
Change-Id: Ia004ee8946ae4441027161f5dddf05d304df8b1b
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

Change-Id: Ie27e4af5f4517bb1e1eea1262ab32140b3dff816
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

Add crash tests for the Quick API and mark the unimplemented features
in the test logs.

Change-Id: I8e6b322fb1bb5ed75fed8b9556e2e975c9227f0a
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

We did not load any localization data on OS X until now,
so the error page did not have any strings, and some
tests asserted because of missing localization.

Change-Id: Icb6de1f7e3aaa6815483d379efaf00e8ed6a1dbf
Reviewed-by: Michael Bruning <michael.bruning@digia.com>

Doing so causes a check to fail in URLRequest::BeforeRequestComplete.

Change-Id: I487042b922b3f357bd8d9579167d9b372963ff29
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

As it turns out failing to do so can mess up the page state.

Change-Id: I8cc785b01c73bde5443bdb78554d7b3809d96afe
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

We need a better solution to find out if we are building on a 64bit system.

This reverts commit 58806890

.

Change-Id: I43c48b5cbc5e96f93b65165b0445670a832b0206
Reviewed-by: Jani Heikkinen <jani.heikkinen@digia.com>

In order to support Fragment (#) based routing and history navigation
together, canGoBack/canGoForward properties should be notified when the
url is changed instead of when the page is loading.

Task-number: QTBUG-41849
Change-Id: Ic995fffa5c683dade88fbd98ffea90527a77261b
Reviewed-by: Michael Bruning <michael.bruning@digia.com>
Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>
Reviewed-by: Jarred Nicholls <jarred.nicholls@gmail.com>

Those were using wrong action names and also the wrong trigger
methods in the wrong context.

Change-Id: If124bf7836a553dc4a632db208cee41fb998c622
Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>

The current delegate just downloads files to the default download
location. By not setting the delegate for downloads, we effectively
disable them.

Change-Id: I1b416f76ac9b4395ee0bcf9986134f4bf056d8ed
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

This is used to represent tables, lists and trees to screen readers.
There are some properties still not implemented, notably selection which
is also not available in blink.

Change-Id: I57053881445e39a4967a1aabd85a594271402f63
Reviewed-by: Zeno Albisser <zeno.albisser@digia.com>
Reviewed-by: Andras Becsi <andras.becsi@digia.com>
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>

Embedded platforms are not always called exactly
linux-g++ only. A platform may also be called
linux-arm-gnueabi-g++ or similar.

Fail gracefully if the host architecture is not x86_64.
We require 64bit for linking QtWebEngineCore.

Change-Id: I62c32606517bed6ed4307720d3c95e8019ec134b
Reviewed-by: Andras Becsi <andras.becsi@digia.com>
Reviewed-by: Michael Bruning <michael.bruning@digia.com>

It's faster to use QLatin1String in places where we don't actually need
a QString object (like for comparison, or when creating a string with
QStringBuilder).

Change-Id: I9b8e28040dc2b79a767e15015b85822d06141bbe
Reviewed-by: Zeno Albisser <zeno.albisser@digia.com>

The requestedUrl function didn't return empty URL even if
the empty URL was really requested. It was assumed if
GetOriginalRequestURL returned empty string that means the requested url was
not set in the navigation entry.

This fix handles that case when empty url is set in the navigation
entry as requested url. If the navigation entry is in pending state that means the
request url has not been set yet thus the actual URL should be returned.

Change-Id: Ic2eff5c487686f7c0e349a7a34a86b80551a002f
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

If the error page loading is interrupted by loading of another page the
signals of the second page should not be suppressed.

Change-Id: I537daabf80ffed005acaaf2a6019e52a3250c523
Reviewed-by: Andras Becsi <andras.becsi@digia.com>
Reviewed-by: Zeno Albisser <zeno.albisser@digia.com>
Reviewed-by: Adam Kallai <kadam@inf.u-szeged.hu>