Bug 43448 – [Qt] [QtTestBrowser] Clean up static and global menu state controls

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Antonio Gomes]

The main idea is to have a LauncherData class (as a POD) to suply LauncherWindow class with the data it needs.

All static and global related classes would go away in favor of that.

Patch coming ...

[Antonio Gomes]

Created attachment 63457 [details]
(committed with r64673, r=hausmann|kenneth) patch v1

After LauncherWindow class was refactored out of from main.cpp, all global variables that were anging in main.cpp became temporarily public static class members of newly added LauncherWindow class. This design was not properly handling the initial purpose of the global variables: newly created launcher windows should inherit the settings of the originating one.
    
In order to properly fix the problem, this patch introduces a LauncherData class, as a POD. It comprises all data needed to handling the goal described above.

[Kenneth Rohde Christiansen]

Comment on attachment 63457 [details]
(committed with r64673, r=hausmann|kenneth) patch v1

Why not just make a settings object?

[Simon Hausmann]

Comment on attachment 63457 [details]
(committed with r64673, r=hausmann|kenneth) patch v1

Looks like a nice cleanup :)

You could also call it LauncherConfiguration or *Config instead of the generic "Data" :)

[Antonio Gomes]

(In reply to comment #3)
> (From update of attachment 63457 [details])
> Looks like a nice cleanup :)
> 
> You could also call it LauncherConfiguration or *Config instead of the generic "Data" :)

Kenneth also suggested WindowOptions. What do you think?

[Antonio Gomes]

Comment on attachment 63457 [details]
(committed with r64673, r=hausmann|kenneth) patch v1

Clearing flags on attachment: 63457

Committed r64673: <http://trac.webkit.org/changeset/r64673>