Bug 47335

Summary: Move user stylesheet test into platform independent directory
Product: WebKit Reporter: Patrick R. Gansterer <paroga>
Component: Tools / TestsAssignee: Patrick R. Gansterer <paroga>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, commit-queue, ddkilzer, eric, mitz, ossy, pfeldman, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Patrick R. Gansterer
Reported 2010-10-07 02:47:40 PDT
see patch
Attachments
Patch (6.75 KB, patch)
2010-10-07 02:53 PDT, Patrick R. Gansterer 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Patrick R. Gansterer
Comment 1 2010-10-07 02:53:54 PDT
Created attachment 70062 [details] Patch
Eric Seidel (no email)
Comment 2 2010-12-13 00:38:39 PST
Comment on attachment 70062 [details] Patch LGTM!
WebKit Review Bot
Comment 3 2010-12-13 02:00:46 PST
Comment on attachment 70062 [details] Patch Clearing flags on attachment: 70062 Committed r73893: <http://trac.webkit.org/changeset/73893>
WebKit Review Bot
Comment 4 2010-12-13 02:00:51 PST
All reviewed patches have been landed. Closing bug.
WebKit Review Bot
Comment 5 2010-12-13 03:42:52 PST
http://trac.webkit.org/changeset/73893 might have broken Qt Linux Release The following tests are not passing: fast/loader/user-stylesheet-fast-path.html
Csaba Osztrogonác
Comment 6 2010-12-13 03:53:20 PST
(In reply to comment #5) > http://trac.webkit.org/changeset/73893 might have broken Qt Linux Release > The following tests are not passing: > fast/loader/user-stylesheet-fast-path.html New bug filed on it: https://bugs.webkit.org/show_bug.cgi?id=50911
Eric Seidel (no email)
Comment 7 2010-12-13 11:11:26 PST
pfeldman believes this broken on Chromium: http://code.google.com/p/chromium/issues/detail?id=66686 At least I think. i can't tell from that bug.
Eric Seidel (no email)
Comment 8 2010-12-13 11:18:36 PST
Given the number of platforms this appears to fail on, we may wish to roll this out. Patrick?
Patrick R. Gansterer
Comment 9 2010-12-13 11:19:57 PST
(In reply to comment #8) > Given the number of platforms this appears to fail on, we may wish to roll this out. Patrick? I think we should add it to the different skip lists istead of rolling out.
Note You need to log in before you can comment on or make changes to this bug.