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>

Add the missing default_encoding to the web preferences.

Use a 0ms timer instead of the 100ms in order to apply the settings
as soon as possible and avoid waiting mechanisms in the test.

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

The build configuration requires the following changes:
- toolkit_views=1: for native_theme_aura, as otherwise
  nine_image_painter_factory.cc would be excluded.
- use_system_harfbuzz=0: we do not want to rely on this
  being provided on the system, therefore being explicit.
- want_separate_host_toolset=1: assures that icuuc gyp
  target will be available.

The file gl_surface_ozone.cc must be added to the exclusion
list to avoid duplicate symbols with gl_surface_at.cpp.

The defines EGL_API_FB and LINUX must be set to 1,
to explicitly define that we are using EGL and are building
for a linux target.

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

Adding a local LoadLibrary function to wrap the new
call to base::LoadNativeLibrary and do the error handling.

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

The Ozone platform base class has received several
new pure virtual functions that needed to be implemented.
Most of the functions are implemented according to
how it is done in OzonePlatformGbm.

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

Fixing unused parameter warnings.
Change-Id: Ic4b60b85f2434aa745e1aeb412984dc3f8a62820
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>

Change-Id: I2d9f23a9aa8871df7b3054628a2db49ea858c522
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@digia.com>

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

    - Remove url variable (it's included from global doc config)
    - Fix qhp selector for QML types
    - Sort the examples in qhp

Change-Id: I7f72aebe4f5482ac922a294ad156f8a73e4ccc6e
Reviewed-by: Martin Smith <martin.smith@digia.com>

Fix the name used for the \module command to include the
module name to properly generate the C++ class reference
page for Qt WebEngine Widgets, and link to the new page
from the module landing page.

Change-Id: If0814650fa16ecf1b5c66e8e46e598d576a8c21d
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>

This makes deployment easier and was also what was done in QtWebKit
for the WebKit2 WebProcess executable

Change-Id: I2245c1f3337f19509c08c08c0baf40dc0a964890
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@digia.com>

Change-Id: Ide283188e04ed4f6619040225c40c2e6850514c8
Reviewed-by: Simon Hausmann <simon.hausmann@digia.com>

A QCoreApplication is needed in order to correctly load the
configuration from qt.conf and setup the Qt environment variables
accordingly. This is important to be able to deploy applications.

This patch creates a dummy QCoreApplication in process main to
have the environment setup correctly.

Change-Id: If536b9edf357d5925fee02f2528872101139f87e
Reviewed-by: Simon Hausmann <simon.hausmann@digia.com>

-added \since and added commands such as \readonly
-converted enum values to use \value
-edited language and other minor issues.
-only the main Qt WebEngine, the widget docs will be edited
 later.

Change-Id: Ifbb76a6472b5efede15daebc78d605b8b449d957
Task-number: QTBUG-40757
Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>
Reviewed-by: Topi Reiniö <topi.reinio@digia.com>

Also added a link to it in a new section briefly explaining the
difference between Qt WebKit and Qt WebEngine.

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

- Fix the example pages titles to match the qdocconf files
- Remove information not as relevant for QtWebEngine as it was for
  QtWebKit
- Fix the QtWebEngineWidgets snippet not to refer to webengine
- Remove the dead link and the reference to Arora from QtWebEngine

Change-Id: Ib3c57c8dd534330b3323c94fb2217374b3e52cf9
Reviewed-by: Andras Becsi <andras.becsi@digia.com>
Reviewed-by: Jerome Pasion <jerome.pasion@digia.com>

This pulls in the changes to the .gitattributes and
for disabling the gpu blacklisting for llvmpipe.

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

PbufferGLSurfaceWGL is a reference counted class, hence deleting
it manually triggers an assert in debug builds if Release is not
called or the reference count is non-zero.

Using a scoped_refptr prevents this and deletes the object
correctly.

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

Window.showModalDialog() is not supported since Chromium 37, so we can
remove the showModalDialog test case.

If the view is not visible, the width and the height of the requested
geometry is always 0 in tst_QWebEnginePage::openWindowDefaultSize.

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

Until now we relied on the PATH to pick the right clang version on
OS X which turned out to be a source of issues on build nodes that
have a lot of old cruft lying around which was picked up by gyp.
Set make_clang_dir so that gyp uses the configured clang version
from Qt and remove the env manipulation from our gyp script.

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

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

Enabling rendering into a WGL backbuffer, in addition to the EGL/angle
one.

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

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

When the PATH is properly set up, this should not be necessary
and it overrode the explicitly set QTWEBENGINEPROCESS_PATH.

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

This patch fixes widgets rendering on HiDPI displays and notifies
about the a potential device pixel ratio change when the window is
moved. This addresses the blurriness when a web view is moved
between an attached LoDPI screen and HiDPI screen resulting from
the device pixel ratio not being updated.

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

If we get a replacement content, we can see a DidFinishLoad event for a frame.
This error page should be ignored based on the frame.

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

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

Since the default arm_tune option applied by chromium is not
recognized by our gcc toolchain set arm_tune to empty string
to prevent chromium from setting a default. Our toolchains
set the arm version which already implies the corresponding
mtune option anyway.

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

This logic was removed in 3a30ed4e


but was still needed by the velocity tracker used for fling gestures.

Also disable swipe gestures in the gesture config since they have
unwanted side-effects on the gesture detection and we don't use them.

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

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

In the same fashion as what is done in QObject::connect's
documentation, we can keep it simple here and simply document
what is expected.

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

Configuring Qt with -force-debug-info -no-strip should carry the
needed cflags to ninja through gyp.

Also prevent stripping the library in release with force_debug_info.
qmake will already call strip --strip-unneeded by default when
running make install for release builds so remove that condition
completely from core_module.pro.

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

Also silence some warnings by tagging functions as reimplemented.

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

That is the role that is returned from the WebView.

Change-Id: I9ab817e259c8fe69d756ffa576082066cf6b4ea8
Reviewed-by: Andras Becsi <andras.becsi@digia.com>
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>

Use the WebFrame's more reliable contentAsMarkup() and
contentAsText() methods instead of going down to the Document
element.

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

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

This method will be called if JavaScript calls window.close() and it
currently crashes because of Q_UNREACHABLE.

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

The unified toolbar doesn't work yet with QOpenGLWidget, which we use
to render RenderWidgetHostViewQtDelegateWidget.

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

Both QWebEnginePage::createWindow and QQuickWebEngineNewViewRequest
behave as if JavascriptCanOpenWindows was disabled by default, when
the application isn't handling new window signals.

LocalStorageEnabled should have no negative impact to be enabled
beside the used disk space, but is still less inconvenient than
cookies or disk cache, which are also enabled by default.

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