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 ...
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.
Comment on attachment 63457 [details] (committed with r64673, r=hausmann|kenneth) patch v1 Why not just make a settings object?
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" :)
(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?
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>