[core] Initial state of Source::Impl::enabled must be true

Until Style::recalculate() is called to check that there are no visible layers using the source, we have to assume there are. Otherwise, Style::isLoaded() can return a false positive.

2 files changed, 8 insertions, 2 deletions

diff --git a/src/mbgl/style/source_impl.hpp b/src/mbgl/style/source_impl.hpp
index 52b2e46a38..fa278b3623 100644
--- a/src/mbgl/style/source_impl.hpp
+++ b/src/mbgl/style/source_impl.hpp
@@ -74,7 +74,11 @@ public:
     const std::string id;
 
     bool loaded = false;
-    bool enabled = false;
+
+    // Tracks whether the source is used by any layers visible at the current zoom level. Must
+    // be initialized to true so that Style::isLoaded() does not produce false positives if
+    // called before Style::recalculate().
+    bool enabled = true;
 
 protected:
     void invalidateTiles();
diff --git a/src/mbgl/style/style.cpp b/src/mbgl/style/style.cpp
index 1009654fc4..7317100bec 100644
--- a/src/mbgl/style/style.cpp
+++ b/src/mbgl/style/style.cpp
@@ -305,7 +305,9 @@ bool Style::isLoaded() const {
     }
 
     for (const auto& source: sources) {
-        if (source->baseImpl->enabled && !source->baseImpl->isLoaded()) return false;
+        if (source->baseImpl->enabled && !source->baseImpl->isLoaded()) {
+            return false;
+        }
     }
 
     if (!spriteStore->isLoaded()) {