RESOLVED FIXED 166356
WebKitTestRunner: Add comment to TestRunner::setCanOpenWindows() 
https://bugs.webkit.org/show_bug.cgi?id=166356
Summary WebKitTestRunner: Add comment to TestRunner::setCanOpenWindows()
Daniel Bates
Reported 2016-12-21 11:28:41 PST
The remark in the patch for bug #43389 makes it unclear whether we should remove the test plugins/get-url-with-blank-target.html. Should we feel this test has value and want to keep it then we will need to implement testRunner.setCanOpenWindows() in WebKitTestRunner such that WebKitTestRunner forbids opening windows by default (this matches the behavior in DumpRenderTree). We should update the comment in TestRunner::setCanOpenWindows() to explain its purpose should we decide to keep the test plugins/get-url-with-blank-target.html.
Attachments
Patch - add comment (5.40 KB, patch)
2016-12-21 11:33 PST, Daniel Bates 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Daniel Bates
Comment 1 2016-12-21 11:33:45 PST
Created attachment 297610 [details] Patch - add comment
Daniel Bates
Comment 2 2016-12-21 11:36:42 PST
Committed r210070: <http://trac.webkit.org/changeset/210070>
Note You need to log in before you can comment on or make changes to this bug.