Removed ownjsonSerializer from WindowManager

[NimbusFox]

Removed the unneeded serializer and applied the settings to the global this._jsonSerializer as when the X and Y values are -1 in the options it would lead to some settings not being applied. E.g. ContextIsolation with it now applying the default values when serializing, settings mentioned before are correctly assigned on Electron's end

In regards to #906

[FlorianRappl]

LGTM!

Any idea why we might have needed / used the previous behavior on the _jsonSerializer @GregorBiswanger ?

[FlorianRappl]

(Wrong base branch used - I changed the target to develop, but that has no impact on the actions, i.e., it does not re-trigger the actions to run)

[softworkz]

Any idea why we might have needed / used the previous behavior on the _jsonSerializer @GregorBiswanger ?

The earlier SocketIO libraries didn't have built-in serialization.

[FlorianRappl]

The earlier SocketIO libraries didn't have built-in serialization.

Yes, but why was the behavior set to DefaultValueHandling.Ignore?

[softworkz]

The earlier SocketIO libraries didn't have built-in serialization.

Yes, but why was the behavior set to DefaultValueHandling.Ignore?

Typically it would be about reducing payloads, but

• that's not really a concern on local loopback networking
• it's not valid to do so across language boundaries, because the JS side will unlikely have the same default values (usually none at all)