Opened 16 years ago
Closed 15 years ago
#963 closed defect (fixed)
Ajax viewer pan/zoom executes two spatial queries on Chrome
Reported by: | brucedechant | Owned by: | |
---|---|---|---|
Priority: | medium | Milestone: | |
Component: | AJAX Viewer | Version: | 2.0.2 |
Severity: | minor | Keywords: | |
Cc: | chrisclaydon | External ID: | 1300521 |
Description
Using Chrome, the Ajax viewer executes two GETDYNAMICMAPOVERLAY requests for each pan/zoom. This is readily evident using Fiddler. Since two overlay requests are generated, all of the Fdo spatial queries are executed twice for every layer. This reduces MapGuide performance by a factor of two.
Attachments (1)
Change History (8)
comment:1 by , 15 years ago
comment:3 by , 15 years ago
External ID: | → 1300521 |
---|
comment:4 by , 15 years ago
Some info from Arthur: In ajaxmappane.templ, when the function RequestMapImage is invoked, the img tag "mapImage" will send out the http request to get the image. When the "mapImage" is loaded, it's image src will be set to "mapImage1" tag. For IE and FF, "mapImage1" will use the cached image, but for Chrome and Safari, a new request will be sent out. Further investigation is needed to make sure it's not the browser's behavior.
by , 15 years ago
Attachment: | ChromeSafariPatch.patch added |
---|
comment:5 by , 15 years ago
I've attached a patch that does 2 things:
1) It handles the "behind-the-scenes" loading of images the same way for Chrome that it does for Safari, preventing a stale image from flashing up while the new image loads when mouse-wheel zooming.
2) For Chrome and Safari, we now skip the pre-loading of map and selection images into a hidden image. We already have special handling of image loads for Safari, making use of one hidden and one visible image. The images are swapped each time a new image finishes loading. There is no need for an additional hidden image in this setup. This change fixes the problem of issuing two requests for each new map image.
comment:6 by , 15 years ago
I forgot to mention that the patch was created against the sandbox/adsk/2.2gp branch
comment:7 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Wow, that sucks. Is this still an issue in 2.1 Bruce?