Fix cache mixing with high-dpi and low-dpi osm providers

This cache fixes the cache mixing problem that has been introduced
with the osm high dpi tiles support.

high dpi providers fall back to low dpi ones and ultimately
to hardcoded providers (also low dpi), and can also be enabled/disabled
via plugin parameter, thus leaving the cache for a given map id
dirty for the next run.

With this patch high dpi tiles are named differently from low dpi ones.
If high-dpi providers are selected, but become not available, the
cache can also change the tileset to load at runtime

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

1 files changed, 3 insertions, 3 deletions

diff --git a/src/location/maps/qgeotilefetcher_p.h b/src/location/maps/qgeotilefetcher_p.h
index cbd8b995..e3621d3d 100644
--- a/src/location/maps/qgeotilefetcher_p.h
+++ b/src/location/maps/qgeotilefetcher_p.h
@@ -65,6 +65,7 @@ class QGeoTileSpec;
 class Q_LOCATION_EXPORT QGeoTileFetcher : public QObject
 {
     Q_OBJECT
+    Q_DECLARE_PRIVATE(QGeoTileFetcher)
 
 public:
     QGeoTileFetcher(QObject *parent = 0);
@@ -83,19 +84,18 @@ Q_SIGNALS:
     void tileError(const QGeoTileSpec &spec, const QString &errorString);
 
 protected:
+    QGeoTileFetcher(QGeoTileFetcherPrivate &dd, QObject *parent = 0);
+
     void timerEvent(QTimerEvent *event);
     QGeoTiledMappingManagerEngine::CacheAreas cacheHint() const;
     virtual bool initialized() const;
 
 private:
-    QGeoTileFetcherPrivate *d_ptr;
 
     virtual QGeoTiledMapReply *getTileImage(const QGeoTileSpec &spec) = 0;
     void handleReply(QGeoTiledMapReply *reply, const QGeoTileSpec &spec);
 
-    Q_DECLARE_PRIVATE(QGeoTileFetcher)
     Q_DISABLE_COPY(QGeoTileFetcher)
-
     friend class QGeoTiledMappingManagerEngine;
 };