[webkit-reviews] review requested: [Bug 41165] FrameLoader cleanup : Remove m_URL member : [Attachment 80201] patch without any test changes

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jan 26 09:34:34 PST 2011


Nate Chapin <japhet at chromium.org> has asked  for review:
Bug 41165: FrameLoader cleanup : Remove m_URL member
https://bugs.webkit.org/show_bug.cgi?id=41165

Attachment 80201: patch without any test changes
https://bugs.webkit.org/attachment.cgi?id=80201&action=review

------- Additional Comments from Nate Chapin <japhet at chromium.org>
The only substantive code change from the previous patch is the check of
frame->ownerElement() in Document::Document() and the comment explain it.  This
patch passes all existing layout tests as-is, unlike previous iterations.


More information about the webkit-reviews mailing list