[webkit-reviews] review granted: [Bug 86540] MHTML files should be loadable from all schemes considered local, not just file: : [Attachment 142540] Patch

[bugzilla-daemon at webkit.org]

Adam Barth <abarth at webkit.org> has granted Greg Spencer
<gspencer at chromium.org>'s request for review:
Bug 86540: MHTML files should be loadable from all schemes considered local,
not just file:
https://bugs.webkit.org/show_bug.cgi?id=86540

Attachment 142540: Patch
https://bugs.webkit.org/attachment.cgi?id=142540&action=review

------- Additional Comments from Adam Barth <abarth at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=142540&action=review


> Source/WebCore/loader/archive/mhtml/MHTMLArchive.cpp:106
>      // For security reasons we only load MHTML pages from the local file
system.

I'd change this comment slightly to say "from local URLs" rather than "from the
local file system".