Bug 43518

Summary: [Qt][QtTestBrowser] Remove unneeded QAction class members
Product: WebKit Reporter: Antonio Gomes <tonikitoo>
Component: Tools / TestsAssignee: Antonio Gomes <tonikitoo>
Severity: Normal Keywords: Qt, QtTriaged
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Description Flags
(committed with r64694, r=kenneth) patch none

Description Antonio Gomes 2010-08-04 16:15:52 PDT
LauncherWindow class has two totally unneeded class members: m_flopAnimated and m_flipYAnimated. Also, in initializeView method there are some dead code block trying to connect them at the wrong time, when they have not been instanciated.
Comment 1 Antonio Gomes 2010-08-04 16:38:53 PDT
Created attachment 63511 [details]
(committed with r64694, r=kenneth) patch

This patch:
- removes the two cited class members in favor of local variables in createChrome method;
- removes the dead code block.
Comment 2 Antonio Gomes 2010-08-04 17:26:48 PDT
Comment on attachment 63511 [details]
(committed with r64694, r=kenneth) patch

Clearing flags on attachment: 63511

Committed r64694: <http://trac.webkit.org/changeset/64694>