[webkit-reviews] review granted: [Bug 37198] [Qt] Interrupting JavaScript is cumbersome when you use QtLaucher for testing or profiling. : [Attachment 52867] GUI option instead of command line argument
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Apr 19 17:40:14 PDT 2010
Simon Hausmann <hausmann at webkit.org> has granted Balazs Kelemen
<kb at inf.u-szeged.hu>'s request for review:
Bug 37198: [Qt] Interrupting JavaScript is cumbersome when you use QtLaucher
for testing or profiling.
https://bugs.webkit.org/show_bug.cgi?id=37198
Attachment 52867: GUI option instead of command line argument
https://bugs.webkit.org/attachment.cgi?id=52867&action=review
------- Additional Comments from Simon Hausmann <hausmann at webkit.org>
r+, but I think you can remove
+ WebPage(QObject* parent = 0, bool interruptingJavaScriptEnabled = false);
as you don't seem to call the constructor with that argument.
I'm generally not a fan of boolean arguments that make the caller code harder
to read,
More information about the webkit-reviews
mailing list