Merge pull request #943 from OpenGeoscience/skip-geom-compute-bounds

Skip having vgl compute geometry bounds.
OpenGeoscience · Oct 30, 2018 · 4cec837 · 4cec837
2 parents 1a84741 + 4acf0c5
commit 4cec837
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 0 deletions.
diff --git a/src/gl/contourFeature.js b/src/gl/contourFeature.js
@@ -240,6 +240,11 @@ var gl_contourFeature = function (arg) {
     geom.addSource(sourceValues);
     geom.addSource(sourceOpacity);
     geom.addPrimitive(primitive);
+    /* We don't need vgl to comptue bounds, so make the geo.computeBounds just
+     * set them to 0. */
+    geom.computeBounds = function () {
+      geom.setBounds(0, 0, 0, 0, 0, 0);
+    };
     m_mapper.setGeometryData(geom);
   };
 

diff --git a/src/gl/lineFeature.js b/src/gl/lineFeature.js
@@ -724,6 +724,11 @@ var gl_lineFeature = function (arg) {
     geom.addSource(strkOpacityData);
     geom.addSource(flagsData);
     geom.addPrimitive(triangles);
+    /* We don't need vgl to comptue bounds, so make the geo.computeBounds just
+     * set them to 0. */
+    geom.computeBounds = function () {
+      geom.setBounds(0, 0, 0, 0, 0, 0);
+    };
     m_mapper.setGeometryData(geom);
     return m_this;
   };

diff --git a/src/gl/pointFeature.js b/src/gl/pointFeature.js
@@ -588,6 +588,11 @@ var gl_pointFeature = function (arg) {
     geom.addSource(sourceAlpha);
     geom.addSource(sourceStrokeOpacity);
     geom.addPrimitive(primitive);
+    /* We don't need vgl to comptue bounds, so make the geo.computeBounds just
+     * set them to 0. */
+    geom.computeBounds = function () {
+      geom.setBounds(0, 0, 0, 0, 0, 0);
+    };
     m_mapper.setGeometryData(geom);
   };
 

diff --git a/src/gl/polygonFeature.js b/src/gl/polygonFeature.js
@@ -311,6 +311,11 @@ var gl_polygonFeature = function (arg) {
     geom.addSource(sourceFillColor);
     geom.addSource(sourceFillOpacity);
     geom.addPrimitive(trianglePrimitive);
+    /* We don't need vgl to comptue bounds, so make the geo.computeBounds just
+     * set them to 0. */
+    geom.computeBounds = function () {
+      geom.setBounds(0, 0, 0, 0, 0, 0);
+    };
     m_mapper.setGeometryData(geom);
 
     s_init.call(m_this, arg);