DevTools: remove section editing state, manual height in SSP
A) SSP used to manually set height, 'contain: strict' during edits
for performance. This CL removes them, since there is no
noticeable gain. See crbug.com/684341
B) Removes unnecessary 'editing state tracking', once used to
manually focus sections before ShadowDOM-delegatesFocus.
See also crbug.com/791494
Pre-keyboard navigation) SSP has tabIndex 0. On editingCommitted,
we ensure focus goes back to the pane.
After-keynav) Individual sections have tabIndex -1. On stopEditing,
parent pane has focus, but we manually move focus to the section.
After ShadowDOM V1) parent pane no longer has tabIndex, but uses
'delegatesFocus'. This makes moving focus around unnecessary.
However, it also introduces 'blue flickering' when users 'Tab'
navigate through properties, since focus is first delegated to
the section. This bug is not addressed in this CL.
Commit-Queue: Erik Luo <firstname.lastname@example.org>
Reviewed-by: Joel Einbinder <email@example.com>
2 files changed