[webkit-reviews] review requested: [Bug 41165] FrameLoader cleanup : Remove m_URL member : [Attachment 65597] split patch - remove m_URL and have FrameLoader::url() call Document::url()

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Aug 26 12:28:30 PDT 2010


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 65597: split patch - remove m_URL and have FrameLoader::url() call
Document::url()
https://bugs.webkit.org/attachment.cgi?id=65597&action=review

------- Additional Comments from Nate Chapin <japhet at chromium.org>
This splits the patch into 2 as Darin suggested.

The main reason I'm doing this is to check my thinking on the rules of
initializing the url in the Document constructor and that my layout test
rewriting seems sane (see my previous comment for my pontification on the
topic).


More information about the webkit-reviews mailing list