[Webkit-unassigned] [Bug 39427] Increase limit on number of (i)frames from 200 to 1000.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=39427





--- Comment #12 from Jeremy Orlow <jorlow at chromium.org>  2010-05-21 09:39:38 PST ---
(In reply to comment #11)
> (From update of attachment 56596 [details])
> > RenderEmbeddedObject::isURLAllowed() looks to see if 
> > Page::frameCount() is < 200. It seems to me that it should use the
> > same limit as HTMLFrameElementBase.
> 
> I don't know what this check is about, but it definitely seems so! Now I wonder what was broken due to not having it fixed, too. Ideally, we would have a test case that fails with the current version of your patch, but not with a final one.

Agreed.  Hans, you should probably first look at the ChangeLog associated with this limit being added in.  Hopefully that'll make it clear why.  If not, maybe you can do some investigating including contacting the author and/or ccing him/her.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.