ESP32: Move the BLE deinit after BLE connection is closed, post commissioning

[dhrishi]

• Problem
  The BLE deinit is done in kCommissioningComplete event. In some cases, it might happen that BLE connection close is not received from the commissioner but stil it tries to free BLE memory

• Solution
  Perform deinit when BLE connection is closed with atleast one fabric being active

• Tests
  Successful commissioning followed by reclaiming the BLE memory

[github-actions]

PR #26183: Size comparison from a14e6fb to 6630bd9

Decreases (1 build for cc32xx)

platform	target	config	section	a14e6fb	6630bd9	change	% change
cc32xx	lock	CC3235SF_LAUNCHXL	.debug_info	20330829	20330828	-1	-0.0

Full report (1 build for cc32xx)

platform	target	config	section	a14e6fb	6630bd9	change	% change
cc32xx	lock	CC3235SF_LAUNCHXL		0	0	0	0.0
			(read only)	643249	643249	0	0.0
			(read/write)	203848	203848	0	0.0
			.ARM.attributes	44	44	0	0.0
			.ARM.exidx	8	8	0	0.0
			.bss	197248	197248	0	0.0
			.comment	194	194	0	0.0
			.data	1480	1480	0	0.0
			.debug_abbrev	933224	933224	0	0.0
			.debug_aranges	87792	87792	0	0.0
			.debug_frame	302140	302140	0	0.0
			.debug_info	20330829	20330828	-1	-0.0
			.debug_line	2687904	2687904	0	0.0
			.debug_loc	2838960	2838960	0	0.0
			.debug_ranges	288072	288072	0	0.0
			.debug_str	3042335	3042335	0	0.0
			.ramVecs	780	780	0	0.0
			.resetVecs	64	64	0	0.0
			.rodata	104401	104401	0	0.0
			.shstrtab	232	232	0	0.0
			.stab	204	204	0	0.0
			.stabstr	441	441	0	0.0
			.stack	2048	2048	0	0.0
			.strtab	377963	377963	0	0.0
			.symtab	256976	256976	0	0.0
			.text	536728	536728	0	0.0

[github-actions]

PR #26183: Size comparison from a14e6fb to e7fd721

Full report (1 build for cc32xx)

platform	target	config	section	a14e6fb	e7fd721	change	% change
cc32xx	lock	CC3235SF_LAUNCHXL		0	0	0	0.0
			(read only)	643249	643249	0	0.0
			(read/write)	203848	203848	0	0.0
			.ARM.attributes	44	44	0	0.0
			.ARM.exidx	8	8	0	0.0
			.bss	197248	197248	0	0.0
			.comment	194	194	0	0.0
			.data	1480	1480	0	0.0
			.debug_abbrev	933224	933224	0	0.0
			.debug_aranges	87792	87792	0	0.0
			.debug_frame	302140	302140	0	0.0
			.debug_info	20330829	20330829	0	0.0
			.debug_line	2687904	2687904	0	0.0
			.debug_loc	2838960	2838960	0	0.0
			.debug_ranges	288072	288072	0	0.0
			.debug_str	3042335	3042335	0	0.0
			.ramVecs	780	780	0	0.0
			.resetVecs	64	64	0	0.0
			.rodata	104401	104401	0	0.0
			.shstrtab	232	232	0	0.0
			.stab	204	204	0	0.0
			.stabstr	441	441	0	0.0
			.stack	2048	2048	0	0.0
			.strtab	377963	377963	0	0.0
			.symtab	256976	256976	0	0.0
			.text	536728	536728	0	0.0

[github-actions]

PR #26183: Size comparison from f3f7ecd to 06d8373

Decreases (1 build for cc32xx)

platform	target	config	section	f3f7ecd	06d8373	change	% change
cc32xx	lock	CC3235SF_LAUNCHXL	.debug_info	20330830	20330829	-1	-0.0

Full report (1 build for cc32xx)

platform	target	config	section	f3f7ecd	06d8373	change	% change
cc32xx	lock	CC3235SF_LAUNCHXL		0	0	0	0.0
			(read only)	643249	643249	0	0.0
			(read/write)	203848	203848	0	0.0
			.ARM.attributes	44	44	0	0.0
			.ARM.exidx	8	8	0	0.0
			.bss	197248	197248	0	0.0
			.comment	194	194	0	0.0
			.data	1480	1480	0	0.0
			.debug_abbrev	933224	933224	0	0.0
			.debug_aranges	87792	87792	0	0.0
			.debug_frame	302140	302140	0	0.0
			.debug_info	20330830	20330829	-1	-0.0
			.debug_line	2687904	2687904	0	0.0
			.debug_loc	2838960	2838960	0	0.0
			.debug_ranges	288072	288072	0	0.0
			.debug_str	3042335	3042335	0	0.0
			.ramVecs	780	780	0	0.0
			.resetVecs	64	64	0	0.0
			.rodata	104401	104401	0	0.0
			.shstrtab	232	232	0	0.0
			.stab	204	204	0	0.0
			.stabstr	441	441	0	0.0
			.stack	2048	2048	0	0.0
			.strtab	377963	377963	0	0.0
			.symtab	256976	256976	0	0.0
			.text	536728	536728	0	0.0

[dhrishi]

@bzbarsky-apple @andy31415 Can we merge this?

[bzbarsky-apple]

@dhrishi There are unresolved conversations.

[dhrishi]

@dhrishi There are unresolved conversations.

I've resolved the same. It is the restyler that made the change. If undone, the CI fails as mentioned above