fix: native View wrapper crash missing when adding child view (#43638)

fix: native View wrapper crash missing when adding child view
This commit is contained in:
Shelley Vohr 2024-09-12 10:01:05 +02:00 committed by GitHub
parent dea6a4e5fb
commit 74b54278bf
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 29 additions and 7 deletions

View file

@ -15,14 +15,10 @@ DelayedNativeViewHost::~DelayedNativeViewHost() = default;
void DelayedNativeViewHost::ViewHierarchyChanged( void DelayedNativeViewHost::ViewHierarchyChanged(
const views::ViewHierarchyChangedDetails& details) { const views::ViewHierarchyChangedDetails& details) {
// NativeViewHost doesn't expect to have children, so filter the
// ViewHierarchyChanged events before passing them on.
if (details.child == this) {
NativeViewHost::ViewHierarchyChanged(details); NativeViewHost::ViewHierarchyChanged(details);
if (details.is_add && GetWidget() && !native_view()) if (details.is_add && GetWidget() && !native_view())
Attach(native_view_); Attach(native_view_);
} }
}
bool DelayedNativeViewHost::OnMousePressed(const ui::MouseEvent& ui_event) { bool DelayedNativeViewHost::OnMousePressed(const ui::MouseEvent& ui_event) {
// NativeViewHost::OnMousePressed normally isn't called, but // NativeViewHost::OnMousePressed normally isn't called, but

View file

@ -36,6 +36,18 @@ describe('View', () => {
expect(w.contentView.children).to.have.lengthOf(1); expect(w.contentView.children).to.have.lengthOf(1);
}); });
it('can be added as a child of another View', async () => {
const w = new BaseWindow();
const v1 = new View();
const v2 = new View();
v1.addChildView(v2);
w.contentView.addChildView(v1);
expect(w.contentView.children).to.deep.equal([v1]);
expect(v1.children).to.deep.equal([v2]);
});
it('correctly reorders children', () => { it('correctly reorders children', () => {
w = new BaseWindow({ show: false }); w = new BaseWindow({ show: false });
const cv = new View(); const cv = new View();

View file

@ -135,6 +135,20 @@ describe('WebContentsView', () => {
expect(w.isFullScreen()).to.be.true('isFullScreen'); expect(w.isFullScreen()).to.be.true('isFullScreen');
}); });
it('can be added as a child of another View', async () => {
const w = new BaseWindow();
const v = new View();
const wcv = new WebContentsView();
await wcv.webContents.loadURL('data:text/html,<div id="div">This is a simple div.</div>');
v.addChildView(wcv);
w.contentView.addChildView(v);
expect(w.contentView.children).to.deep.equal([v]);
expect(v.children).to.deep.equal([wcv]);
});
describe('visibilityState', () => { describe('visibilityState', () => {
async function haveVisibilityState (view: WebContentsView, state: string) { async function haveVisibilityState (view: WebContentsView, state: string) {
const docVisState = await view.webContents.executeJavaScript('document.visibilityState'); const docVisState = await view.webContents.executeJavaScript('document.visibilityState');