[webkit-reviews] review requested: [Bug 15938] Inspector/Qt
requires patch to SecurityOrigin : [Attachment 17188] Use
FrameLoader::shouldTreatURLAsLocal from SecurityOrigin::canAccess
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sun Nov 11 13:26:03 PST 2007
Holger Freyther <freyther at handhelds.org> has asked for review:
Bug 15938: Inspector/Qt requires patch to SecurityOrigin
http://bugs.webkit.org/show_bug.cgi?id=15938
Attachment 17188: Use FrameLoader::shouldTreatURLAsLocal from
SecurityOrigin::canAccess
http://bugs.webkit.org/attachment.cgi?id=17188&action=edit
------- Additional Comments from Holger Freyther <freyther at handhelds.org>
Remove the unused m_portSet. Remove m_host, m_protocol and m_port in favor of
storing a KURL directly. Use KURL::isLocalFile instead of comparing to "file"
directly. Avoid calling lower() on (Deprecated)String but use the case
insensitive compare besides in ::setFrame.
Like the last patch no regressions were introduced and please take a look at it
in regard to:
-performance
-security
-correctness
More information about the webkit-reviews
mailing list