fixup

src/style/pauseable_placement.js

@@ -18,8 +18,7 @@ class LayerPlacement {
     _bucketParts: Array<BucketPart>;
 
     constructor(styleLayer: SymbolStyleLayer) {
-        this._sortAcrossTiles = styleLayer.type === 'symbol' &&
-            styleLayer.layout.get('symbol-z-order') !== 'viewport-y' &&
+        this._sortAcrossTiles = styleLayer.layout.get('symbol-z-order') !== 'viewport-y' &&
             styleLayer.layout.get('symbol-sort-key').constantOr(1) !== undefined;
 
         this._currentTileIndex = 0;
@@ -49,13 +48,14 @@ class LayerPlacement {
 
         while (this._currentPartIndex < bucketParts.length) {
             const bucketPart = bucketParts[this._currentPartIndex];
-            placement.placeLayerBucket(bucketPart, this._seenCrossTileIDs, showCollisionBoxes);
+            placement.placeLayerBucketPart(bucketPart, this._seenCrossTileIDs, showCollisionBoxes);
 
             this._currentPartIndex++;
             if (shouldPausePlacement()) {
                 return true;
             }
         }
+        return false;
     }
 }
 

src/symbol/placement.js

@@ -327,7 +327,7 @@ export class Placement {
         }
     }
 
-    placeLayerBucket(bucketPart: Object, seenCrossTileIDs: { [string | number]: boolean }, showCollisionBoxes: boolean) {
+    placeLayerBucketPart(bucketPart: Object, seenCrossTileIDs: { [string | number]: boolean }, showCollisionBoxes: boolean) {
 
         const {
             bucket,

test/unit/data/symbol_bucket.test.js

@@ -61,7 +61,7 @@ test('SymbolBucket', (t) => {
         const parts = [];
         placement.getBucketParts(parts, layer, tile, false);
         for (const part of parts) {
-            placement.placeLayerBucket(part, {}, false);
+            placement.placeLayerBucketPart(part, {}, false);
         }
     };
     const a = placement.collisionIndex.grid.keysLength();