[webkit-reviews] review granted: [Bug 35794] [Qt] QtLauncher need a menu option to show/hide FPS : [Attachment 50101] Patch

[bugzilla-daemon at webkit.org]

Simon Hausmann <hausmann at webkit.org> has granted Jesus Sanchez-Palencia
<jesus.palencia at openbossa.org>'s request for review:
Bug 35794: [Qt] QtLauncher need a menu option to show/hide FPS
https://bugs.webkit.org/show_bug.cgi?id=35794

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

------- Additional Comments from Simon Hausmann <hausmann at webkit.org>
Apart from the commit queue problem the patch looks good to me (I've been
looking for this option, too :)

I think in general it would be good to replace all the commandline options of
QtLauncher with menus actions.