Bug 1701451 Comment 22 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
I looked into more. nsWindow::Resize()  was not root cause of the hang(nsWindow::OnPaint() flood).nsWindow::Resize() caused the problem because, argument aHeight became negative value. It happened by size mismatch between GetClientBounds() and mBounds in [nsBaseWidget::ResizeClient()](https://searchfox.org/mozilla-central/rev/3151f97de27730793c2e298716df760999423f26/widget/nsBaseWidget.cpp#1499).

The size mismatch was caused by UpdateLayeredWindow() call in [PresentableSharedImage::PresentToWindow()](https://searchfox.org/mozilla-central/rev/3151f97de27730793c2e298716df760999423f26/widget/windows/RemoteBackbuffer.cpp#263). When window size was changed, old window size could be applied via size of mSharedImage.
Revision 1 by
on 
I looked into more. nsWindow::Resize()  was not root cause of the hang(nsWindow::OnPaint() flood).nsWindow::Resize() caused the problem because, argument aHeight became negative value. It happened by size mismatch between GetClientBounds() and mBounds in [nsBaseWidget::ResizeClient()](https://searchfox.org/mozilla-central/rev/3151f97de27730793c2e298716df760999423f26/widget/nsBaseWidget.cpp#1499).

The size mismatch was caused by UpdateLayeredWindow() call in [PresentableSharedImage::PresentToWindow()](https://searchfox.org/mozilla-central/rev/3151f97de27730793c2e298716df760999423f26/widget/windows/RemoteBackbuffer.cpp#263). When window size was changed, old window size could be applied via size of mSharedImage. The old size was applied as window size.

Back to Bug 1701451 Comment 22