Change-Id: I08db215e954aea317c2a7dd635b171fbde99ef00
Reviewed-on: http://gerrit.dmdirc.com/3096
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I81c0f466fc01929d60b6d52a2fdf1c02ef64a063
Reviewed-on: http://gerrit.dmdirc.com/3095
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: Ia4b0279b0db183c9267c39e9ecc2387d21f5fa11
Reviewed-on: http://gerrit.dmdirc.com/3094
Reviewed-by: Chris Smith <chris@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Previously it only released the instance if the window was closed by the user
interacting with it through the window manager, rather than if it was closed
programatically (through an OK/Cancel button, for example).

Change-Id: I07ef95ebf4e47279c027747b03c5adcd1b37eea4
Reviewed-on: http://gerrit.dmdirc.com/3091
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Removes some deprecated method calls.

Change-Id: I71c273fa52b63101de5bf70db455568c84aca60a
Reviewed-on: http://gerrit.dmdirc.com/3090
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Also minor tidying up.

Change-Id: Icc9c5aab819344867c1772c7f98f156d341619c5
Reviewed-on: http://gerrit.dmdirc.com/3088
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

This removes the need for cinch, so also removes the related code,
this also lays the foundation for adding a binding framework in
the future.

Change-Id: I62f7c66b25361c56fbb95fa28c74544d894ea476
Fixes-Issue: CLIENT-422
Reviewed-on: http://gerrit.dmdirc.com/3086
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Chris Smith <chris@dmdirc.com>

Change-Id: I594c92d21eec07e171cebc90eab05f5a03cbcf25
Reviewed-on: http://gerrit.dmdirc.com/3082
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Use a factory where available, and horrible singleton where not.

(ConfigManager was using a horrible singleton anyway, under the hood,
 so it's not that much worse)

Change-Id: I82d215bf22b68d0359fce4c33976a9bf68d2bfda
Reviewed-on: http://gerrit.dmdirc.com/3080
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: Ia5208e2dc03c352982fb6a1c49f81825c5ed773f
Reviewed-on: http://gerrit.dmdirc.com/3077
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I965365c5006bbacae9f72f7915cf55cb80c659ee
Reviewed-on: http://gerrit.dmdirc.com/3076
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

For the minute this means the prefs dialog no longer
saves state, it also means there may be some layout
changes with the height loss and parenting change for
sub dialogs launched from preferences settings.

Change-Id: I9c46e6f03920866022cfe063c500f9523a11db6e
Reviewed-on: http://gerrit.dmdirc.com/3071
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Chris Smith <chris@dmdirc.com>

Change-Id: I406ccaf85f5700c184f3528e279dde455b46ceb3
Reviewed-on: http://gerrit.dmdirc.com/3074
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Chris Smith <chris@dmdirc.com>

Change-Id: I3187d75463129f8ff7c3191dc0be6f1c16d2217c
Reviewed-on: http://gerrit.dmdirc.com/3069
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Shane Mc Cormack <shane@dmdirc.com>

Change-Id: Iebdb477a3763c4958ea34e9ccdafcbafbc5ddff8
Reviewed-on: http://gerrit.dmdirc.com/3068
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I111e31f9da6f7a4b79ab69d043de4c85c31e9b51
Reviewed-on: http://gerrit.dmdirc.com/3065
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I30ca8bc84a18cc67a9bcb3700a9b15307e389bc2
Reviewed-on: http://gerrit.dmdirc.com/3066
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Chris Smith <chris@dmdirc.com>

Bundle them up in a static class to avoid having to pass a million deps
around in each and every subclass.

Change-Id: I581e39f9da6f7a4b79ab69d043de4c81111e9b52
Reviewed-on: http://gerrit.dmdirc.com/3064
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Remove the legacy code, which was used in far fewer places than I thought.

Change-Id: I72cfa09e92110163e16a8a43f18a4127560a65c1
Reviewed-on: http://gerrit.dmdirc.com/3063
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I581e39f9da6f7a4b79ab69d043de4c85c31e9b52
Reviewed-on: http://gerrit.dmdirc.com/3062
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I9d730b109ae1757a01ee757e91ec6e833efc449d
Reviewed-on: http://gerrit.dmdirc.com/3061
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I828f2ea9980123409a7d33efc787cb3b6d64d441
Reviewed-on: http://gerrit.dmdirc.com/3060
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I82bf2ea9980123409a7d33efc787cb3b6d64d446
Reviewed-on: http://gerrit.dmdirc.com/3059
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I6396a62915d2f2243838987f77b4fd1e878db9be
Reviewed-on: http://gerrit.dmdirc.com/3057
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: Iea2ccd1e20524b40189dee9e7a4f9822f558645a
Reviewed-on: http://gerrit.dmdirc.com/3056
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Chris Smith <chris@dmdirc.com>

Change-Id: I683cd68cddf42df3c2dacf2a959fc606116ddf33
Reviewed-on: http://gerrit.dmdirc.com/3055
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I8608be6d0352227d2eca226027f3f60c0f321ab7
Reviewed-on: http://gerrit.dmdirc.com/3054
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I253ee4c7e8a8b4475048b860db244db8d82fbada
Reviewed-on: http://gerrit.dmdirc.com/3053
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I8608be6d035c2d7d2eca226027f3f60c0f0f7ac7
Reviewed-on: http://gerrit.dmdirc.com/3052
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Replace remaining legitimate usages with [Keyed]DialogProviders.

Remove the DialogManager itself, all the getters for it, all the places keeping
references around to it, all the dialogs taking it in the constructor, and all
the partridges in pear trees that had one hidden away.

Change-Id: I0a1234960a4e08c0a7c6afc08a5432102b17c125
Reviewed-on: http://gerrit.dmdirc.com/3048
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I8608be6d035c9d7deeca526047f3f60c0f0f7777
Reviewed-on: http://gerrit.dmdirc.com/3051
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: Ia428aae5a19309d6f7c19222226160f8c2d80de1
Reviewed-on: http://gerrit.dmdirc.com/3047
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: Ia428aae5a19309d622109e8baa6160f8c2d80de1
Reviewed-on: http://gerrit.dmdirc.com/3046
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I92010f61f3289579603a239ab4213a304799dedb
Reviewed-on: http://gerrit.dmdirc.com/3045
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

This fixes the issues with injection of dialogs and the behaviour of @Singleton.

Move the swing injection related classes into a new package (a new DialogModule
will follow shortly to make use of the DialogProvider).

Also introduce a SwingPreconditions class for asserting method calls are made
on/off the EDT.

Change-Id: I92010f61f2289579603a239ab46b0a304799dedc
Reviewed-on: http://gerrit.dmdirc.com/3044
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I73731643e3af5b13dd6dcaa2fdb53349a775372b
Reviewed-on: http://gerrit.dmdirc.com/3042
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I6bf2a097c7dbb54e631cd1a1a13f411df374428f
Reviewed-on: http://gerrit.dmdirc.com/3041
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I0a6ccc960a3a08d0a7c6aac08aa5b89d2b17c129
Reviewed-on: http://gerrit.dmdirc.com/3040
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

(Hit me slowly, hit me quick.)

Change-Id: I0a6ccc960a4e08d1a7c6aac08aa5b89c2b17c125
Reviewed-on: http://gerrit.dmdirc.com/3039
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I0a6ccc960a4e08d0a7c6aac08aa5b89d2b17c125
Reviewed-on: http://gerrit.dmdirc.com/3038
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I0b6ccc960a4e08d0a7c6afc08a75b89d2b17c125
Reviewed-on: http://gerrit.dmdirc.com/3037
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: Ia7e89f9d3da632451ccbec04eabf453478e256cf
Reviewed-on: http://gerrit.dmdirc.com/3036
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Also deprecate the getters in the SwingController that should be
avoided in favour of injection.

Change-Id: I0a6ccc960a4e08d0a7c6afc08a75b89d2b17c125
Reviewed-on: http://gerrit.dmdirc.com/3035
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I0a6ccc960a4e08c0a7c6afc08a75b89d2b17c125
Reviewed-on: http://gerrit.dmdirc.com/3034
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I0a6ccc960a4e08c0a7c6afc08a75b89d2b17c124
Reviewed-on: http://gerrit.dmdirc.com/3032
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I0a6ccc960a4e08c0a7c6afc08a75b89d2b17c123
Reviewed-on: http://gerrit.dmdirc.com/3031
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: Ic14614537aa544095733785a983b79556c8f2472
Reviewed-on: http://gerrit.dmdirc.com/3029
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Down with static methods! Temporarily up with deprecated methods on horrible
global state objects!

Change-Id: Ia279f9199e8ca40dc3031234569a44da042fc617
Reviewed-on: http://gerrit.dmdirc.com/3027
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Again, add a deprecated method to SwingController/Manager to escape the dependency
hell temporarily.

Change-Id: Ia79f9199e8ca40dddddd8de6f9a44da042fc617f
Depends-On: Ia79f9199e8ca40dc3031234569a44da042fc617f
Reviewed-on: http://gerrit.dmdirc.com/3025
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I420b85ed73ba6ea02ac4b5fe18411fa100994011
Reviewed-on: http://gerrit.dmdirc.com/3024
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>