[webkit-reviews] review granted: [Bug 58593] WebKit2: Implement layoutTestController.setPluginsEnabled() in WebKitTestRunner. : [Attachment 144144] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat May 26 17:02:58 PDT 2012


Darin Adler <darin at apple.com> has granted Marcelo Lira
<marcelo.lira at openbossa.org>'s request for review:
Bug 58593: WebKit2: Implement layoutTestController.setPluginsEnabled() in
WebKitTestRunner.
https://bugs.webkit.org/show_bug.cgi?id=58593

Attachment 144144: Patch
https://bugs.webkit.org/attachment.cgi?id=144144&action=review

------- Additional Comments from Darin Adler <darin at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=144144&action=review


> Tools/ChangeLog:11
> +	   (WTR):

Stray line that should be deleted. Better to write per-function comments, also.


> Tools/ChangeLog:13
> +	   (LayoutTestController):

Another stray line that needs to be deleted. The script to prepare change logs
doesn’t get them completely right, so the programmers has to do a bit of work.


More information about the webkit-reviews mailing list