Hi

I think this might be a typo yes. But actually, the problem is way
worse than I thought. This error manifests itself whenever Chrome
sends an iframe. This does not only happens with cross-domain iframes.

I am investigating currently. At least it helps knowing that nobody at
Qx knows what this.__form.src means, so I definitely guess it is a
total error / typo.

On Fri, Oct 1, 2010 at 4:42 PM, Alexander Steitz
<[email protected]> wrote:
> Hi Jean-Noël,
>
>> This works except on Chrome, where it fails on l. 271 of Iframe.js
>> because this.__form is undefined. I guess this is because of security
>> concerns (the code is trying to access something it does not have
>> access to). But what I dont understand at all is what the code is
>> supposed to do (so that I can patch it properly). The main thing I am
>> stuck on is:
>>
>>       if (this.__form.src) {
>>         return;
>>       }
>>
>>
>> Why is this code there???? I have searched and there is no other
>> references at all to this.__form.src so what is this code checking?
> I don't know it exactly, but after a first short look at the code I would 
> tend to say that this is a typo. I guess what was meant here was
>
> --snip--
> if (this.__frame.src) {
>  return;
> }
> --snip--
>
> because this code is the handler function for the "onload" event of the 
> iframe.
>
>> I have noticed that when you add the iframe in Chrome, an onLoad event
>> is immediately fired so I guess this code prevents the callback from
>> actually marking the request as already completed... but I really need
>> more explanations to properly patch this.
> This might be the problem, yes.
>
>> Thanks for any help (if someone can reply quickly... I would be very
>> grateful as I need to patch this quite urgently in production ;)
> My advice: patch this code locally and re-test. If the issue is really a typo 
> create a bug report with a patch attached.
>
> cheers,
>  Alex
> ------------------------------------------------------------------------------
> Start uncovering the many advantages of virtual appliances
> and start using them to simplify application deployment and
> accelerate your shift to cloud computing.
> http://p.sf.net/sfu/novell-sfdev2dev
> _______________________________________________
> qooxdoo-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
>



-- 
Jean-Noël Rivasseau
Directeur
(1) 778 786 3460 / (33) 01 82 88 05 26
Kameleoon - morphing the web
http://www.kameleoon.com/

------------------------------------------------------------------------------
Start uncovering the many advantages of virtual appliances
and start using them to simplify application deployment and
accelerate your shift to cloud computing.
http://p.sf.net/sfu/novell-sfdev2dev
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to