Bug 165699 - Remove QuickLook scheme from the list of secure schemes
Summary: Remove QuickLook scheme from the list of secure schemes
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: WebKit Local Build
Hardware: iPhone / iPad iOS 10
: P2 Normal
Assignee: Daniel Bates
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-12-09 16:20 PST by Daniel Bates
Modified: 2016-12-09 18:04 PST (History)
1 user (show)

See Also:

Attachments
Patch (2.03 KB, patch)
2016-12-09 16:21 PST, Daniel Bates 		aestes: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Bates 2016-12-09 16:20:08 PST 
Following r207155 we no longer need to treat the QuickLook scheme as a secure scheme. For completeness, we added the QuickLook scheme to the list of secure schemes in the patch for bug #150145.
Comment 1 Daniel Bates 2016-12-09 16:21:16 PST 
Created attachment 296734 [details]
Patch
Comment 2 Andy Estes 2016-12-09 16:22:51 PST 
Comment on attachment 296734 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=296734&action=review

> Source/WebCore/ChangeLog:10
> +        origin from which they were served. We added the QuickLooks scheme to the list of secure

QuickLook, not QuickLooks.

> Source/WebCore/ChangeLog:12
> +        QuickLooks document served over HTTPS. Now that we use a unique origin for QuickLook

Ditto.
Comment 3 Daniel Bates 2016-12-09 18:04:19 PST 
Committed r209641: <http://trac.webkit.org/changeset/209641>