Changes condition for early exit of requestRectangleData

Author: mzschwartz5

packages/engine/Source/Scene/ClippingPolygonCollection.js

@@ -33,8 +33,6 @@ import VectorPipeline from "../Core/VectorPipeline.js";
 // Reused flyweight for reading/writing individual BufferPolygons.
 const bufferPolygonScratch = new BufferPolygon();
 
-const scratchRectangle = new Rectangle();
-
 /**
  * A ClippingPolygon paired with the index of its mirrored primitive in a collection's BufferPolygonCollection.
  *
@@ -1177,12 +1175,7 @@ ClippingPolygonCollection.prototype.requestRectangleData = function (
 
   const vectorTileData = {};
 
-  // Early out if there are no polygons or the rectangle does not intersect the collection's bounding rectangle
-  const collectionRectangle = this._vectorCollectionData.rectangle;
-  if (
-    this.length === 0 ||
-    !Rectangle.intersection(rectangle, collectionRectangle, scratchRectangle)
-  ) {
+  if (this.length === 0) {
     return vectorTileData;
   }
 
@@ -1193,6 +1186,11 @@ ClippingPolygonCollection.prototype.requestRectangleData = function (
     vectorTileData,
   );
 
+  // No overlapping polygons means no textures to pack (and the caller can skip the clipping rendering step)
+  if (vectorTileData.polygonRings.length === 0) {
+    return vectorTileData;
+  }
+
   VectorPipeline.packPolygonGrid(vectorTileData);
 
   VectorPipeline.packPolygonTextures(context, vectorTileData);