Opened 15 years ago

Closed 14 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)

ChromeSafariPatch.patch (9.0 KB ) - added by chrisclaydon 14 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 by jbirch, 14 years ago

Wow, that sucks. Is this still an issue in 2.1 Bruce?

comment:2 by brucedechant, 14 years ago

I just tested this against 2.1 and the problem still exists.

comment:3 by brucedechant, 14 years ago

External ID: 1300521

comment:4 by tomfukushima, 14 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 chrisclaydon, 14 years ago

Attachment: ChromeSafariPatch.patch added

comment:5 by chrisclaydon, 14 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 chrisclaydon, 14 years ago

I forgot to mention that the patch was created against the sandbox/adsk/2.2gp branch

comment:7 by chrisclaydon, 14 years ago

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.