Change-Id: If6f8fd446c562d6ccaf368e1d5afb5eee113fece
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Change-Id: I1fff4d22c11c540b955abbfeb0707c8622291bef
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

This patch moves the lower/upper bound check on setZoom from
QDeclarativeGeoMap to the gesture area, allowing to set higher zoom
levels than the maximumZoomLevel when setting Map.zoomLevel directly,
for the map types that support overzoom.
This is now safe as the bound check is introduced in the tile fetcher,
so no invalid tiles will be requested, and is beneficial when
combining layers supporting different maximum zoom levels.

Change-Id: I08ee9c282ee2ebc1dafa3c68a238b93ffbc1ba02
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Task-number: QTBUG-58801
Change-Id: I2a980c6d218d0d7e44277f7b31fcdd7355e31698
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Until now we have not honored the min/max zoom levels specified
in QGeoCameraCapabilities when requesting tiles.
As a result we have often fired requests for non-existing tile layers.

This change complements the added support for individual camera caps
for each map type, and makes the tile fetcher honor the bounds
defined in the camera capabilities.

This also allows to set larger zoom levels in the renderer, overzooming
existing tiles, without the fetcher firing requests for invalid
resources

Change-Id: Ic8a523a114147109f7ef8af3510a3ab78d06d714
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Since the only part of QGeoMapType used in the method is mapId, there's
no need to pass the whole map type as argument.
The benefit is that the camera caps can be requested solely based on
the info in QGeoTileSpec.

Change-Id: Iafd0e2a1d4d45fbf02b862efe56841001cbebd75
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

This will turn -Werror on for our tested compilers, to avoid introducing
new warnings.

Change-Id: I27b55fdf514247549455fffd14b17ae6dce9976f
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

This is not just a regular test. This is a plugin that does
load(qt_plugin).

 qgeotiledmappingmanagerengine_test.h:89:14: error: 'createMap' overrides a member function but is not marked 'override' [-Werror,-Winconsistent-missing-override]

Change-Id: I523b0abacd5148b2bf08fffd14b488ffa441e68b
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

It is reasonable to add mapReady property to the condition
for the start of test cases where it is required that the
map is initialized. This covers the case when one of the
initialization steps runs asynchronously.

Change-Id: I2eff52dbe3a7bec0b2f3fb26d607398829b5c9f2
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>

Somehow MSVC still sees it as unused, despite our best efforts in
Q_ASSERT. So add Q_UNUSED.

 qdeclarativepinchgenerator.cpp(218) : warning C4100: 'event' : unreferenced formal parameter

Change-Id: I523b0abacd5148b2bf08fffd14b46aa8f8409f42
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>

This fixes the -no-gui build.

Task-number: QTBUG-60062
Change-Id: I71400dfddac8ddf6a74906be8ba1272bf4a86781
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>

This patch makes the structure of the pro file of the itemsoverlay
plugin the same as the other plugins (like the mapbox plugin).

Change-Id: Iada6296562282b238c6f793b9f0b80ade8ec6ddf
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Found by MSVC 2015

 qclipperutils.cpp(65): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data

Change-Id: I523b0abacd5148b2bf08fffd14b41a36046adafa
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>

In this way there's a way to easily get to a zoom level where the map
is sharp

Change-Id: I176cf3870d9c3fe970191d46c804e13daab674fa
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Change-Id: Ic169824201ed6aeaeb2a482b28da989f9312181d
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Change-Id: I7f81f3dab3ef0243c45ec22bafa8687f756134f4
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Change-Id: I7a82c0d63f35d1cfd264801e58089408254757c8
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Using qAsConst where possible to prevent QList from detaching.

Change-Id: Ief03fb17761e350335f945b7134cf4a9266971d7
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

m_map inside QDeclarativeGeoMap is created, but also destroyed, by the
engine.
In some cases (dynamic QML object creation/destruction) the engine may
disappear before the Map.
This patch introduces a QPointer to prevent a dangling pointer

Change-Id: Ice3c57b4dbeb96ee8d7e4d401654b9085c105dec
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

It's marked as exported from QtGui, yet the implementation is in
qdoublematrix.cpp. That cannot be.

Found by MSVC 2015:

 qdoublematrix4x4.cpp(1097): warning C4273: 'operator >>': inconsistent dll linkage
 qdoublematrix4x4_p.h(939): note: see previous definition of '>>'

Change-Id: I523b0abacd5148b2bf08fffd14b419faf3d1e584
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Mapbox GL depends on ICU, so it should be after -lqmapboxgl in
the linker command line.

Change-Id: I725d32d1132520ddd0704c7d9e998453a92e10ab
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>

Change-Id: I4f0b09cb8aac0b4d4f2d27cf350a30b7ed800f11
Reviewed-by: Nico Vertriest <nico.vertriest@qt.io>
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>

qdeclarativegeomap.cpp: 1327: warning: Can't link to 'Component.onCompleted'
qgeopath.cpp:117: warning: Undocumented parameter 'width' in QGeoPath::QGeoPath()
qgeopath.cpp:117: warning: Undocumented parameter 'path' in QGeoPath::QGeoPath()
qgeoshape.cpp:90: warning: Undocumented enum item 'PathType' in QGeoShape::ShapeType
qgeoshape.cpp:231: warning: No such parameter 'QGeoRectangle' in QGeoShape::boundingGeoRectangle()

Change-Id: Ie57dff2aabe82d61b8122362a396ac706939c3b3
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>

So it won't clash with the libclipper used by the Mapbox GL plugin
when linking Qt with -static.

With -static, all the plugins are linked together instead of each
one having its own shared library, thus, hiding private symbols.

Change-Id: I33ee4afd4b83741c5f3fae32a88da7f4a7fee75f
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>

If all of the tiles were loaded from cache and plugin for the map was set after creation of the declarative map, it can occur that item is not updated

Task-number: QTBUG-60021
Change-Id: I260b6219b65ef16726aa767d3a83c6fc756642b5
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>

Task-number: QTBUG-59281
Change-Id: I3605265ca9c330ea782bdda53ce5af004ef35a22
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>

This patch fixes the incorrect behavior of using nearest filtering
at integer zoom level, even if the texture is being overzoomed from
a lower zoom level

Change-Id: Ia3572399187a6f24829504116602685434ecfc5f
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Better rendering using Mapbox GL plugin.

Change-Id: I3ef1862988e652c1caad62920c2f3b5775f18552
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
Reviewed-by: Bruno de Oliveira Abinader <brunoabinader@gmail.com>

Document the MapParameters supported by this plugin.

Change-Id: Ib755460e2f326e9b9569da31aa4bff4015c1a90b
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>

This patch replaces the two separate flick velocities and displacements
for x and y with one vector , so that there will be no discrepancy
between the flicking direction and where the map actually flicks

Change-Id: I83164107003a423cb7332da5759d83ff689c1b31
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

 qgeotiledmappingmanagerengineosm.h:62:14: error: 'createMap' overrides a member function but is not marked 'override' [-Winconsistent-missing-override]

Change-Id: I27b55fdf514247549455fffd14b17c97ad2db003
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>

We don't care to fix them.

 clipper.cpp:3665:13: warning: unused variable 'firstLeft' [-Wunused-variable]
 sweep.cpp:706:8: warning: reference cannot be bound to dereferenced null pointer in well-defined C++ code; comparison may be assumed to always evaluate to false [-Wtautological-undefined-compare]

Change-Id: I27b55fdf514247549455fffd14b17ba6040ac332
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>

This patch forces a camera data update upon changeActiveMapType, which
may change the tile size.

Change-Id: Ib371ea7f76267bc4ed8a30eed31ec3887b4a1415
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Change-Id: I3fe2c7fecd8cf8036f84fed5c4875e39d937c9cd
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>

Change-Id: I096a14f5f1e8f83d4671990d052ecfff8095e985
Reviewed-by: Thiago Marcos P. Santos <tmpsantos@gmail.com>
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>

Change-Id: If3e920f4b4cb5bf6310914a258eb3f569de9a8ef
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Stop the refresh timer hack when all the resources are loaded
so we don't keep waking up for re-drawing when it is not needed.

Change-Id: Iada46850adf4fec04dc70c9d3f7263b71c325d8a
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>

This patch makes the rotation gesture start more smoothly instead of
abruptingly start with an initial rotation equal to starting angle
threshold, once this threshold angle has been trespassed.

Change-Id: I2be52fa53f50151b9b87244a1b53ca4105121a8b
Reviewed-by: Bruno de Oliveira Abinader <brunoabinader@gmail.com>
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Add additional sanity checks to do the geometry changing
only if the map() has been set already (that is, the
item has been added to the map)

Change-Id: Id5b5faa41c68c80c3ad322f408b7a8abe5d2f7ab
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Change-Id: I13bdd8cb9c9c92ea86144c004af06c66adb28bde
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>