addEventListener beforeunload not working

[pasviegas]

Hey, I am using a external library that uses setInterval and adds a beforeunload listener to clean it before leaving the page. But thats not happening, so it is causing the user to be prompted a message before leaving.

It seems that polymer is overriding the addEventListener and not firing the beforeunload properly.

Is it a known issue? Is there any fix?

Thanks a lot.

[arv]

Are you setting eventObject.returnValue or are you doing it differently?

[pasviegas]

This is the function which is called:

function(a) {
        try {
            if (!this.id) {
                return
            }
            var b = this.opaqueElement;
            if (this.isIgnored(b)) {
                return
            }
            if (b) {
                this.click(b);
                this.opaqueElement = null
            }
        } catch (c) {}
    }

Thanks! =)

[arv]

I don't see where you are setting returnValue? Can you provide a test case (jsbin)?

[pasviegas]

Actually, it's a third party library that is adding the listener and it only works in the right domain. So I don't think I can properly make a jsbin.

But if I remove polymer it works.. =T

[arv]

Does it work in Firefox without polymer? (Chrome used to have an extension to addEventListener)

[arv]

I can reproduce weirdness... not the same issue as you are seeing though. I get "false" instead of the string that I want.

[pasviegas]

Yep, I am getting false too =)

Seems that it happens only in chrome and safari..

[arv]

I can explain the false one. That is Chrome doing the wrong thing with the return value. I noticed that BeforeUnloadEvent was never registered.

I'm debugging now. I expect to have this fixed today.

[pasviegas]

I am trying to test it and see if it works for me, how can I use your pr?

[dfreedm]

@pasviegas Just merged this in, sorry for the delay.

[pasviegas]

Working, thanks a lot!