Google Suggest

Darin Adler darin at apple.com
Thu Apr 28 01:15:52 CEST 2005


On Apr 27, 2005, at 2:54 AM, Allan Sandfeld Jensen wrote:

> Oops. I gave you the wrong version of the patch, but it can still  
> be done a
> lot simpler than you do. Here is the current version:
>
>     case FrameElement:
>       if (m_frame->m_frame)
>         return getDOMNode(exec,m_frame->m_frame->element());
>       else
>         return Undefined();

That looks good, although we can't use it in our tree because we  
haven't made the change to how frames work that you have.

But it's missing a security check. I believe it needs an isSafeScript  
call.

     -- Darin



More information about the Khtml-devel mailing list