[core] Push Source::url down to subclasses

7 files changed, 12 insertions, 13 deletions

diff --git a/src/mbgl/annotation/annotation_source.cpp b/src/mbgl/annotation/annotation_source.cpp
index 55e6fc82a3..637109ff8d 100644
--- a/src/mbgl/annotation/annotation_source.cpp
+++ b/src/mbgl/annotation/annotation_source.cpp
@@ -5,7 +5,7 @@
 namespace mbgl {
 
 AnnotationSource::AnnotationSource()
-    : Source(SourceType::Annotations, AnnotationManager::SourceID, "", util::tileSize) {
+    : Source(SourceType::Annotations, AnnotationManager::SourceID, util::tileSize) {
 }
 
 Range<uint8_t> AnnotationSource::getZoomRange() {
diff --git a/src/mbgl/style/source.cpp b/src/mbgl/style/source.cpp
index 3c5690e980..21fb7fee36 100644
--- a/src/mbgl/style/source.cpp
+++ b/src/mbgl/style/source.cpp
@@ -32,13 +32,9 @@ namespace style {
 
 static SourceObserver nullObserver;
 
-Source::Source(SourceType type_,
-               std::string id_,
-               std::string url_,
-               uint16_t tileSize_)
+Source::Source(SourceType type_, std::string id_, uint16_t tileSize_)
     : type(type_),
       id(std::move(id_)),
-      url(std::move(url_)),
       tileSize(tileSize_),
       observer(&nullObserver) {
 }
diff --git a/src/mbgl/style/source.hpp b/src/mbgl/style/source.hpp
index 36e6570fb5..99d634a887 100644
--- a/src/mbgl/style/source.hpp
+++ b/src/mbgl/style/source.hpp
@@ -32,10 +32,7 @@ class SourceObserver;
 
 class Source : public TileObserver, private util::noncopyable {
 public:
-    Source(SourceType,
-           std::string id,
-           std::string url,
-           uint16_t tileSize);
+    Source(SourceType, std::string id, uint16_t tileSize);
     ~Source() override;
 
     bool loaded = false;
@@ -71,7 +68,6 @@ public:
 
     const SourceType type;
     const std::string id;
-    const std::string url;
     uint16_t tileSize = util::tileSize;
     bool enabled = false;
 
diff --git a/src/mbgl/style/sources/geojson_source.cpp b/src/mbgl/style/sources/geojson_source.cpp
index 6e1068dadb..7daf148291 100644
--- a/src/mbgl/style/sources/geojson_source.cpp
+++ b/src/mbgl/style/sources/geojson_source.cpp
@@ -63,7 +63,8 @@ std::unique_ptr<GeoJSONSource> GeoJSONSource::parse(const std::string& id,
 GeoJSONSource::GeoJSONSource(std::string id_,
                              std::string url_,
                              std::unique_ptr<mapbox::geojsonvt::GeoJSONVT>&& geojsonvt_)
-    : Source(SourceType::GeoJSON, std::move(id_), std::move(url_), util::tileSize),
+    : Source(SourceType::GeoJSON, std::move(id_), util::tileSize),
+      url(url_),
       geojsonvt(std::move(geojsonvt_)) {
 }
 
diff --git a/src/mbgl/style/sources/geojson_source.hpp b/src/mbgl/style/sources/geojson_source.hpp
index 7022d387e9..996dac3aca 100644
--- a/src/mbgl/style/sources/geojson_source.hpp
+++ b/src/mbgl/style/sources/geojson_source.hpp
@@ -28,10 +28,13 @@ public:
 
     void load(FileSource&) final;
 
+    const std::string& getURL() const { return url; }
+
 private:
     Range<uint8_t> getZoomRange() final;
     std::unique_ptr<Tile> createTile(const OverscaledTileID&, const UpdateParameters&) final;
 
+    const std::string url;
     std::unique_ptr<mapbox::geojsonvt::GeoJSONVT> geojsonvt;
     std::unique_ptr<AsyncRequest> req;
 };
diff --git a/src/mbgl/style/tile_source.cpp b/src/mbgl/style/tile_source.cpp
index 12c62f07a5..29cbcc7226 100644
--- a/src/mbgl/style/tile_source.cpp
+++ b/src/mbgl/style/tile_source.cpp
@@ -12,7 +12,8 @@ TileSource::TileSource(SourceType type_,
                        std::string url_,
                        uint16_t tileSize_,
                        std::unique_ptr<Tileset>&& tileset_)
-    : Source(type_, std::move(id_), std::move(url_), tileSize_),
+    : Source(type_, std::move(id_), tileSize_),
+      url(std::move(url_)),
       tileset(std::move(tileset_)) {
 }
 
diff --git a/src/mbgl/style/tile_source.hpp b/src/mbgl/style/tile_source.hpp
index c0dc0da8c7..b001d174f6 100644
--- a/src/mbgl/style/tile_source.hpp
+++ b/src/mbgl/style/tile_source.hpp
@@ -24,11 +24,13 @@ public:
 
     void load(FileSource&) final;
 
+    const std::string& getURL() const { return url; }
     const Tileset* getTileset() const { return tileset.get(); }
 
 protected:
     Range<uint8_t> getZoomRange() final;
 
+    const std::string url;
     std::unique_ptr<const Tileset> tileset;
     std::unique_ptr<AsyncRequest> req;
 };