Bug 108271

Summary: Do not register autostart for plugins from file:// (or nowhere)
Product: WebKit Reporter: Dean Jackson <dino>
Component: Plug-insAssignee: Dean Jackson <dino>
Status: RESOLVED FIXED    
Severity: Normal CC: jonlee, simon.fraser, thorton, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch thorton: review+

Dean Jackson
Reported 2013-01-29 17:40:04 PST
PlugIns that come from a file:// origin should never be added to the autostart list.
Attachments
Patch (1.72 KB, patch)
2013-01-29 17:45 PST, Dean Jackson 		thorton: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Dean Jackson
Comment 1 2013-01-29 17:41:17 PST
<rdar://problem/13037458>
Dean Jackson
Comment 2 2013-01-29 17:45:55 PST
Created attachment 185364 [details] Patch
Tim Horton
Comment 3 2013-01-29 18:32:43 PST
Comment on attachment 185364 [details] Patch Do we really need the log?
Tim Horton
Comment 4 2013-01-29 18:33:13 PST
Comment on attachment 185364 [details] Patch Also somewhat surprised we don't have something somewhere to match file://
Tim Horton
Comment 5 2013-01-29 18:39:32 PST
Maybe you should use SchemeRegistry::shouldTreatURLSchemeAsLocal, if you know how to split the scheme off.
Dean Jackson
Comment 6 2013-02-08 12:27:04 PST
Committed r142310: <http://trac.webkit.org/changeset/142310>
Note You need to log in before you can comment on or make changes to this bug.