Bug 156556

Summary: CSP: Remove duplicate test meta-outside-head.html
Product: WebKit Reporter: Daniel Bates <dbates>
Component: Tools / TestsAssignee: Daniel Bates <dbates>
Status: RESOLVED FIXED    
Severity: Normal CC: aestes, bfulgham, commit-queue, lforschler, mkwst
Priority: P2    
Version: WebKit Local Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Daniel Bates
Reported 2016-04-13 14:36:24 PDT
It is unnecessary to keep the test <http://trac.webkit.org/browser/trunk/LayoutTests/http/tests/security/contentSecurityPolicy/1.1/meta-outside-head.html?rev=195367> as the functionality it exercises is covered by test <http://trac.webkit.org/browser/trunk/LayoutTests/http/tests/security/contentSecurityPolicy/meta-tag-ignored-if-not-in-head.html?rev=199513> (added in bug #59858). The output of test meta-tag-ignored-if-not-in-head.html also better conveys the purpose of the test and how to interpret its result compared to the output of test meta-outside-head.html.
Attachments
Patch (2.79 KB, patch)
2016-04-13 14:38 PDT, Daniel Bates 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Daniel Bates
Comment 1 2016-04-13 14:38:26 PDT
Created attachment 276355 [details] Patch
Brent Fulgham
Comment 2 2016-04-13 14:55:46 PDT
Comment on attachment 276355 [details] Patch Great! Anything we can do to make LayoutTests smaller is a good thing!
Daniel Bates
Comment 3 2016-04-13 19:35:30 PDT
Comment on attachment 276355 [details] Patch Clearing flags on attachment: 276355 Committed r199526: <http://trac.webkit.org/changeset/199526>
Daniel Bates
Comment 4 2016-04-13 19:35:34 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.