Bug 98891

Summary: [WK2] should enable tests that use setAlwaysAcceptCookies()
Product: WebKit Reporter: Jussi Kukkonen (jku) <jussi.kukkonen>
Component: Tools / TestsAssignee: Jussi Kukkonen (jku) <jussi.kukkonen>
Status: RESOLVED FIXED    
Severity: Normal CC: webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch for cq none

Jussi Kukkonen (jku)
Reported 2012-10-10 05:51:51 PDT
There's a bunch of tests in wk2/TestExpectations with comment "WebKit2 needs layoutTestController.setAlwaysAcceptCookies". TestRunner has had that functionality for some months now, so the tests should be enabled (barring other problems).
Attachments
patch for cq (2.11 KB, patch)
2012-10-11 00:19 PDT, Jussi Kukkonen (jku) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Jussi Kukkonen (jku)
Comment 1 2012-10-11 00:19:52 PDT
Created attachment 168152 [details] patch for cq
Jussi Kukkonen (jku)
Comment 2 2012-10-11 02:39:07 PDT
Comment on attachment 168152 [details] patch for cq Nice surprise, nothing had broken during the time these were skipped (although many tests are already skipped in port specific TestExpectations, but that's fine). commit-queue, please?
WebKit Review Bot
Comment 3 2012-10-11 04:54:42 PDT
Comment on attachment 168152 [details] patch for cq Clearing flags on attachment: 168152 Committed r131044: <http://trac.webkit.org/changeset/131044>
WebKit Review Bot
Comment 4 2012-10-11 04:54:45 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.