[webkit-reviews] review granted: [Bug 100388] [Qt] Flaky security tests : [Attachment 171194] Patch

[bugzilla-daemon at webkit.org]

Jocelyn Turcotte <jocelyn.turcotte at digia.com> has granted Allan Sandfeld Jensen
<allan.jensen at digia.com>'s request for review:
Bug 100388: [Qt] Flaky security tests
https://bugs.webkit.org/show_bug.cgi?id=100388

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

------- Additional Comments from Jocelyn Turcotte <jocelyn.turcotte at digia.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=171194&action=review


> Tools/DumpRenderTree/qt/DumpRenderTreeQt.cpp:521
> +

r=me if you remove this extra empty line and if you can add a short comment on
why we do this setUrl (and maybe adapt the comment above that says we stop.)