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

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has denied Holger Freyther
<freyther at handhelds.org>'s request 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 Darin Adler <darin at apple.com>
The additional string allocation from calling left() on the URL string will
almost certainly cause an unacceptable slowdown; there's a reason we added
those special cases for "http" and "file" to avoid memory allocation.

The concept of the patch is fine, but we need a more efficient version.