Problems with pollDoScroll and IE family (1.6.1 rc)

[Florian Gnägi]

Hi

I get errors with prototype 1.6.1rc in IE browsers. In IE debugger I
get the "Out of stack space" message.

I could track it down to the lines 635 and 636 in event.js. When I
remove the lines, everything runs smoothly.

http://github.com/sstephenson/prototype/blob/483c7886de5c0c6e721dee06b3f9d1359614700f/src/dom/event.js#L635-636

Could this be a sideeffect of using prototype with ExtJS?

Thanks for any hints
Florian

[david]

Hi Florian,

i found this comment on the dean edwards blog.
go and see it: http://dean.edwards.name/weblog/2006/06/again/#comment121098

I think it explain your problem.

--
david

On 3 juin, 19:08, Florian Gnägi <florian.gna...@gmail.com> wrote:
> Hi
>
> I get errors with prototype 1.6.1rc in IE browsers. In IE debugger I
> get the "Out of stack space" message.
>
> I could track it down to the lines 635 and 636 in event.js. When I
> remove the lines, everything runs smoothly.
>

> http://github.com/sstephenson/prototype/blob/483c7886de5c0c6e721dee06...

[gnaegi]

Hi David

Thanks for the link, but this does not help me. I know what
pollDoScroll() does. My problem is that when I use prototype in my
webapp, I get Out of stack space error messages in IE because the
pollDoScroll() method is executed over 600 times in a recursion and
the expression document.documentElement.doScroll('left'); will always
fail.

I traced it down to the fact that together with ExtJS
pollDoScroll.defer(); does not seem to wait long enough. I don't know
if this is an ExtJS-Prototype combination problem or if this is
because my app does some other things while loading.

When I replace pollDoScroll.defer(); with pollDoScroll.delay(0);
everything works.

How is the process to get in tough with the Prototype devels? Is this
something we can change in the release or is there a reason why it
uses pollDoScroll.defer(); instead of pollDoScroll.delay(0);?

Thanks for any hints
Florian

[T.J. Crowder]

Hi,

> When I replace pollDoScroll.defer(); with pollDoScroll.delay(0);
> everything works.

That's weird, since xyz.defer() is functionally equivalent (with a bit
more overhead) to xyz.delay(0.01).

> How is the process to get in tough with the Prototype devels?

http://prototypejs.org/contribute

HTH,
--
T.J. Crowder
tj / crowder software / com
Independent Software Engineer, consulting services available

[gnaegi]

Hi HTH

Yes, weird but as it seems a known problem between ExtJS and Prototype
for some time. This article explains that ExtJS does override the
Prototype defer() method, thus calling delay(0.01) is actually at
runtime not the same as calling defer()... This might also have other
side effects, but it's the first I have so far: http://extjs.com/forum/showthread.php?p=198159

Still, when I modify the Prototype source code to use delay instead of
defer in the pollDoScroll I don't get any errors in IE.

Am I really the only one with this issue?

thanks anyway & cheers
Florian