[webkit-reviews] review requested: [Bug 15938] Inspector/Qt requires patch to SecurityOrigin : [Attachment 17243] Add FrameLoader::shouldTreatSchemeAsLocal

[bugzilla-daemon at webkit.org]

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 17243: Add FrameLoader::shouldTreatSchemeAsLocal
http://bugs.webkit.org/attachment.cgi?id=17243&action=edit

------- Additional Comments from Holger Freyther <freyther at handhelds.org>
Altnernative implementation. Add FrameLoader::shouldTreatSchemeAsLocal and call
it with KURL::protocol from within FrameLoader and SecurityOrigin when we have
a KURL. In CachedResource and Document we continue to use the
shouldTreatURLAsLocal function as we have (Deprecated)Strings there.

This function is doing the splitting of the URL first before it is comparing
file and http as fast path. And due not using .find in shouldTreatSchemeAsLocal
we need to check for the null string otherwise we end up with a crash in the
HashTable.

This introduces no regressions and I would welcome feedback and specially in
regard to Performance and Correctness.