diff options
author | John Firebaugh <john.firebaugh@gmail.com> | 2017-01-09 13:05:13 -0800 |
---|---|---|
committer | John Firebaugh <john.firebaugh@gmail.com> | 2017-01-10 10:15:22 -0800 |
commit | e7778509c546f4d1e0dd73e2d1deceb146c90de2 (patch) | |
tree | d8101c341d45a6a011cb37101f486814be7a6259 /test/tile | |
parent | ea4c0b77c39926c770b0003097509e36dc26621d (diff) | |
download | qtlocation-mapboxgl-e7778509c546f4d1e0dd73e2d1deceb146c90de2.tar.gz |
[core] Fix flickering caused by regression in #7586
It should be safe to invoke GeometryTileWorker::setData multiple times without invoking GeometryTileWorker::setLayers. Therefore GeometryTileWorker::redoLayout() must not consume the layers.
Diffstat (limited to 'test/tile')
-rw-r--r-- | test/tile/geojson_tile.test.cpp | 72 |
1 files changed, 72 insertions, 0 deletions
diff --git a/test/tile/geojson_tile.test.cpp b/test/tile/geojson_tile.test.cpp new file mode 100644 index 0000000000..920e946a2b --- /dev/null +++ b/test/tile/geojson_tile.test.cpp @@ -0,0 +1,72 @@ +#include <mbgl/test/util.hpp> +#include <mbgl/test/fake_file_source.hpp> +#include <mbgl/test/stub_tile_observer.hpp> +#include <mbgl/tile/geojson_tile.hpp> +#include <mbgl/tile/tile_loader_impl.hpp> + +#include <mbgl/util/default_thread_pool.hpp> +#include <mbgl/util/run_loop.hpp> +#include <mbgl/map/transform.hpp> +#include <mbgl/style/style.hpp> +#include <mbgl/style/update_parameters.hpp> +#include <mbgl/style/layers/circle_layer.hpp> +#include <mbgl/annotation/annotation_manager.hpp> + +#include <memory> + +using namespace mbgl; +using namespace mbgl::style; + +class GeoJSONTileTest { +public: + FakeFileSource fileSource; + TransformState transformState; + util::RunLoop loop; + ThreadPool threadPool { 1 }; + AnnotationManager annotationManager { 1.0 }; + style::Style style { fileSource, 1.0 }; + Tileset tileset { { "https://example.com" }, { 0, 22 }, "none" }; + + style::UpdateParameters updateParameters { + 1.0, + MapDebugOptions(), + transformState, + threadPool, + fileSource, + MapMode::Continuous, + annotationManager, + style + }; +}; + +TEST(GeoJSONTile, Issue7648) { + GeoJSONTileTest test; + GeoJSONTile tile(OverscaledTileID(0, 0, 0), "source", test.updateParameters); + + test.style.addLayer(std::make_unique<CircleLayer>("circle", "source")); + + StubTileObserver observer; + observer.tileChanged = [&] (const Tile&) { + // Once present, the bucket should never "disappear", which would cause + // flickering. + ASSERT_NE(nullptr, tile.getBucket(*test.style.getLayer("circle"))); + }; + tile.setObserver(&observer); + + tile.setPlacementConfig({}); + + mapbox::geometry::feature_collection<int16_t> features; + features.push_back(mapbox::geometry::feature<int16_t> { + mapbox::geometry::point<int16_t>(0, 0) + }); + + tile.updateData(features); + while (!tile.isComplete()) { + test.loop.runOnce(); + } + + tile.updateData(features); + while (!tile.isComplete()) { + test.loop.runOnce(); + } +} |