[OpenLayers-Commits] r6496 - in trunk/openlayers/lib/OpenLayers: Layer Tile

commits at openlayers.org commits at openlayers.org
Wed Mar 12 10:21:27 EDT 2008


Author: tschaub
Date: 2008-03-12 10:21:27 -0400 (Wed, 12 Mar 2008)
New Revision: 6496

Modified:
   trunk/openlayers/lib/OpenLayers/Layer/GML.js
   trunk/openlayers/lib/OpenLayers/Layer/GeoRSS.js
   trunk/openlayers/lib/OpenLayers/Tile/WFS.js
Log:
No longer do we force re-parsing of request.responseText for WFS, GML, and GeoRSS layers.  Thanks avlee for spotting this performance hit we've been taking since at least r174.  r=crschmidt (closes #1428)

Modified: trunk/openlayers/lib/OpenLayers/Layer/GML.js
===================================================================
--- trunk/openlayers/lib/OpenLayers/Layer/GML.js	2008-03-12 12:53:34 UTC (rev 6495)
+++ trunk/openlayers/lib/OpenLayers/Layer/GML.js	2008-03-12 14:21:27 UTC (rev 6496)
@@ -131,7 +131,7 @@
     requestSuccess:function(request) {
         var doc = request.responseXML;
         
-        if (!doc || request.fileType!="XML") {
+        if (!doc) {
             doc = request.responseText;
         }
         

Modified: trunk/openlayers/lib/OpenLayers/Layer/GeoRSS.js
===================================================================
--- trunk/openlayers/lib/OpenLayers/Layer/GeoRSS.js	2008-03-12 12:53:34 UTC (rev 6495)
+++ trunk/openlayers/lib/OpenLayers/Layer/GeoRSS.js	2008-03-12 14:21:27 UTC (rev 6496)
@@ -131,7 +131,7 @@
      */
     parseData: function(ajaxRequest) {
         var doc = ajaxRequest.responseXML;
-        if (!doc || ajaxRequest.fileType!="XML") {
+        if (!doc) {
             doc = OpenLayers.parseXMLString(ajaxRequest.responseText);
         }
         

Modified: trunk/openlayers/lib/OpenLayers/Tile/WFS.js
===================================================================
--- trunk/openlayers/lib/OpenLayers/Tile/WFS.js	2008-03-12 12:53:34 UTC (rev 6495)
+++ trunk/openlayers/lib/OpenLayers/Tile/WFS.js	2008-03-12 14:21:27 UTC (rev 6496)
@@ -128,7 +128,7 @@
         if (this.features) {
             var doc = request.responseXML;
             
-            if (!doc || request.fileType!="XML") {
+            if (!doc) {
                 doc = OpenLayers.parseXMLString(request.responseText);
             }
             if (this.layer.vectorMode) {



More information about the Commits mailing list