Bug 96069

Summary: [WK2][WTR] Unskip detached-stylesheet-without-wrapper.html test case.
Product: WebKit Reporter: Kangil Han <kangil.han>
Component: WebKit EFLAssignee: Kangil Han <kangil.han>
Status: RESOLVED FIXED    
Severity: Normal CC: gyuyoung.kim, kenneth, lucas.de.marchi, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: Linux   
Attachments:
Description Flags
patch
kenneth: commit-queue-
patch
none
patch none

Kangil Han
Reported 2012-09-06 22:36:14 PDT
fast/dom/StyleSheet/detached-stylesheet-without-wrapper.html test case has been passed on EFL layout test. WebKit EFL doesn't have to skip this.
Attachments
patch (3.85 KB, patch)
2012-09-06 22:38 PDT, Kangil Han 		kenneth: commit-queue-
DetailsFormatted DiffDiff
patch (3.85 KB, patch)
2012-09-07 01:16 PDT, Kangil Han 		no flags
DetailsFormatted DiffDiff
patch (3.84 KB, patch)
2012-09-07 01:25 PDT, Kangil Han 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Kangil Han
Comment 1 2012-09-06 22:38:02 PDT
Created attachment 162673 [details] patch
Kenneth Rohde Christiansen
Comment 2 2012-09-07 00:52:17 PDT
Comment on attachment 162673 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=162673&action=review > LayoutTests/ChangeLog:4 > + [WK2][WTR] Unskip detached-stylesheet-without-wrapper.html test case in EFL. > + https://bugs.webkit.org/show_bug.cgi?id=96069 The EFL here is confusing as ou are skipping this on multiple platforms. Apart from that it is fine
Kangil Han
Comment 3 2012-09-07 01:16:17 PDT
Created attachment 162700 [details] patch How about this? :-)
Kenneth Rohde Christiansen
Comment 4 2012-09-07 01:18:20 PDT
Comment on attachment 162700 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=162700&action=review > LayoutTests/ChangeLog:4 > + [EFL][WK2][WTR] Unskip detached-stylesheet-without-wrapper.html test case. > + https://bugs.webkit.org/show_bug.cgi?id=96069 It should not have EFL prefix it is it not EFL only and it is not.
Kangil Han
Comment 5 2012-09-07 01:25:28 PDT
Created attachment 162706 [details] patch I misunderstood your comment. :/ Try it again!
WebKit Review Bot
Comment 6 2012-09-07 12:15:07 PDT
Comment on attachment 162706 [details] patch Clearing flags on attachment: 162706 Committed r127904: <http://trac.webkit.org/changeset/127904>
WebKit Review Bot
Comment 7 2012-09-07 12:15:11 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.