Ref #96672.

* Revert "Distinguish a user removing a row from 'clear' in search"

This reverts commit 4cf9c461.

* Revert "Fix searchResult tests"

This reverts commit 7646b358.

* Make search dirty on any removal, but ony if it does not leave the view empty.

Removes the workaround for maximizing the panel. Instead track if the editor is visible and only lay the view out if the editor is visible

Part of #97016

Fixes #95556

Part of https://github.com/microsoft/vscode/issues/96999

fyi @dbaeumer

fixes microsoft/vscode-js-debug#455

fyi @bpasero

Fix for #96968

This is a scoped fix for #96968.

The cause of the issue is the following:

1. Webview must be rendered outside of the main editor DOM. We do this by absolutely positioning them over some element in the DOM.
1. In split views, we try to lay the webview out over an element that has 0 height.
1. Due to my workaround in ea07e9bb, this causes the webview to either not show at all (because it also will have zero height) or partially show

This fix forces the webivew's parent in the split view to have 100%. That actually seems like a reasonable default but I've scoped my fix to just webviews