Freeze while selecting item in the results tree #407

jairbubbles · 2020-10-28T14:13:38Z

Randomly the UI freezes when selecting an item after doing a search.

When breaking in the code it seems to be stuck while measuring items in the VirtualizingStackPanel:

 	PresentationCore.dll!System.Windows.Media.Visual.GetDpi()	Inconnu
 	PresentationFramework.dll!System.Windows.FrameworkElement.MeasureCore(System.Windows.Size availableSize)	Inconnu
 	PresentationCore.dll!System.Windows.UIElement.Measure(System.Windows.Size availableSize)	Inconnu
 	PresentationFramework.dll!System.Windows.Controls.Grid.MeasureCell(int cell, bool forceInfinityV)	Inconnu
 	PresentationFramework.dll!System.Windows.Controls.Grid.MeasureCellsGroup(int cellsHead, System.Windows.Size referenceSize, bool ignoreDesiredSizeU, bool forceInfinityV, out bool hasDesiredSizeUChanged)	Inconnu
 	PresentationFramework.dll!System.Windows.Controls.Grid.MeasureOverride(System.Windows.Size constraint)	Inconnu
 	PresentationFramework.dll!System.Windows.FrameworkElement.MeasureCore(System.Windows.Size availableSize)	Inconnu
 	PresentationCore.dll!System.Windows.UIElement.Measure(System.Windows.Size availableSize)	Inconnu
 	PresentationFramework.dll!System.Windows.Controls.Control.MeasureOverride(System.Windows.Size constraint)	Inconnu
 	PresentationFramework.dll!System.Windows.FrameworkElement.MeasureCore(System.Windows.Size availableSize)	Inconnu
 	PresentationCore.dll!System.Windows.UIElement.Measure(System.Windows.Size availableSize)	Inconnu
 	PresentationFramework.dll!System.Windows.Controls.VirtualizingStackPanel.MeasureChild(ref System.Windows.Controls.Primitives.IItemContainerGenerator generator, ref System.Windows.Controls.Primitives.IContainItemStorage itemStorageProvider, ref System.Windows.Controls.Primitives.IContainItemStorage parentItemStorageProvider, ref object parentItem, ref bool hasUniformOrAverageContainerSizeBeenSet, ref double computedUniformOrAverageContainerSize, ref double computedUniformOrAverageContainerPixelSize, ref bool computedAreContainersUniformlySized, ref bool hasAnyContainerSpanChanged, ref System.Collections.IList items, ref object item, ref System.Collections.IList children, ref int childIndex, ref bool visualOrderChanged, ref bool isHorizontal, ref System.Windows.Size childConstraint, ref System.Windows.Rect viewport, ref System.Windows.Controls.VirtualizationCacheLength cacheSize, ref System.Windows.Controls.VirtualizationCacheLengthUnit cacheUnit, ref long scrollGeneration, ref bool foundFirstItemInViewport, ref double firstItemInViewportOffset, ref System.Windows.Size stackPixelSize, ref System.Windows.Size stackPixelSizeInViewport, ref System.Windows.Size stackPixelSizeInCacheBeforeViewport, ref System.Windows.Size stackPixelSizeInCacheAfterViewport, ref System.Windows.Size stackLogicalSize, ref System.Windows.Size stackLogicalSizeInViewport, ref System.Windows.Size stackLogicalSizeInCacheBeforeViewport, ref System.Windows.Size stackLogicalSizeInCacheAfterViewport, ref bool mustDisableVirtualization, bool isBeforeFirstItem, bool isAfterFirstItem, bool isAfterLastItem, bool skipActualMeasure, bool skipGeneration, ref bool hasBringIntoViewContainerBeenMeasured, ref bool hasVirtualizingChildren)	Inconnu
 	PresentationFramework.dll!System.Windows.Controls.VirtualizingStackPanel.MeasureOverrideImpl(System.Windows.Size constraint, ref double? lastPageSafeOffset, ref System.Collections.Generic.List<double> previouslyMeasuredOffsets, ref double? lastPagePixelSize, bool remeasure)	Inconnu
 	PresentationFramework.dll!System.Windows.Controls.VirtualizingStackPanel.MeasureOverride(System.Windows.Size constraint)	Inconnu
 	PresentationFramework.dll!System.Windows.FrameworkElement.MeasureCore(System.Windows.Size availableSize)	Inconnu

I have a fairly good repro:

open this binlog.
search for "ReferenceCopyLocalPaths"
select the last item in the results tree
hit the "Down Key" (it's the last item so it should have no effect)

NB: the text scaling is at 125%

The text was updated successfully, but these errors were encountered:

jairbubbles · 2020-10-28T14:16:08Z

Looks like it's already logged #21

jairbubbles · 2020-10-28T14:21:40Z

And I have a pretty recent (the latest?) .NET Framework version installed (528372).

KirillOsenkov · 2020-10-28T18:01:52Z

Yes, this is #21, the workaround is to uncheck this box:
#21 (comment)

KirillOsenkov · 2020-10-28T18:09:16Z

BTW thanks for the repro steps, I tried to reproduce and couldn't on my machine. This is highly sensitive to screen DPI, window size, timing of input events, so really hard to find a reliable repro that works across machines. So the bug may still be there or it may be fixed.

jairbubbles · 2020-10-28T20:23:54Z

Hummm... I still do reproduce it even if I disable the tree virtualisation.

If I attach the debugger I still see the VirtualizingStackPanel in the stack trace:

>	PresentationCore.dll!System.Windows.UIElement.Arrange(System.Windows.Rect finalRect)	Inconnu
 	PresentationFramework.dll!System.Windows.Controls.Grid.ArrangeOverride(System.Windows.Size arrangeSize)	Inconnu
 	PresentationFramework.dll!System.Windows.FrameworkElement.ArrangeCore(System.Windows.Rect finalRect)	Inconnu
 	PresentationCore.dll!System.Windows.UIElement.Arrange(System.Windows.Rect finalRect)	Inconnu
 	PresentationFramework.dll!System.Windows.Controls.Control.ArrangeOverride(System.Windows.Size arrangeBounds)	Inconnu
 	PresentationFramework.dll!System.Windows.Controls.TreeViewItem.ArrangeOverride(System.Windows.Size arrangeSize)	Inconnu
 	PresentationFramework.dll!System.Windows.FrameworkElement.ArrangeCore(System.Windows.Rect finalRect)	Inconnu
 	PresentationCore.dll!System.Windows.UIElement.Arrange(System.Windows.Rect finalRect)	Inconnu
 	PresentationFramework.dll!System.Windows.Controls.VirtualizingStackPanel.ArrangeOtherItemsInExtendedViewport(bool isHorizontal, System.Windows.UIElement child, System.Windows.Size childDesiredSize, double arrangeLength, int index, ref System.Windows.Rect rcChild, ref System.Windows.Size previousChildSize, ref System.Windows.Point previousChildOffset, ref int previousChildItemIndex)	Inconnu
 	PresentationFramework.dll!System.Windows.Controls.VirtualizingStackPanel.ArrangeOverride(System.Windows.Size arrangeSize)	Inconnu

drewnoakes · 2020-10-28T21:15:19Z

I noticed a long delay yesterday while clicking around on search results. I think it occurred when I clicked an item which was one of many siblings in the tree. If I see it again I'll get a trace.

KirillOsenkov · 2020-10-28T21:30:19Z

yes, if you have a long list and not using virtualization, the long delay is expected and by design (just lots of layout work that needs to happen)

jairbubbles · 2020-10-29T07:19:58Z

@KirillOsenkov It is possible that the left tree control (filter) still have virtualization even if the checkbox is unticked and that's the one which is faulty in my case?

I'll look at the code source when I have free time

KirillOsenkov · 2020-10-29T07:24:40Z

Oh interesting, it might be possible indeed, need to check

jairbubbles · 2020-10-30T18:58:54Z

I can confirm that this is the other tree which is responsible for the freeze. If I disable the virtualization on "resultsList" by removing:

VirtualizingPanel.IsVirtualizing="True"
VirtualizingPanel.VirtualizationMode="Recycling"

I no longer have the issue.

SamBent · 2020-11-09T18:23:58Z

Although they're both hangs, this is not really the same as #21 - they have different root causes. To avoid polluting #21, I recommend moving the discussion back here.

The new hang is occurring with the latest .NET updates (that fixed the hangs in #21). Neither @KirillOsenkov nor I can repro it, but I think it's very sensitive to the exact height of the "resultsList" TreeView and the data it displays, and to the exact sequence of scrolling operations; we just haven't stumbled on the right combination. The .stf trace @jairbubbles attached to #21 should help, but I'm still tinkering. @anton-zaicev - can you attach an .stf trace for your case, to confirm that it's the same issue?

Note that this is really an issue in .NET/WPF; there's nothing wrong with Kirill's app. I've asked @KirillOsenkov to open an issue in the internal .NET database, which will justify spending time and effort to diagnose and fix the problem in a .NET servicing update.

KirillOsenkov · 2020-11-09T18:32:27Z

I've just made a change for the resultsList to respect the virtualization setting:
4b307e4

At least as a workaround you can now turn off the virtualization checkbox on the startup screen to get unblocked:
#21 (comment)

anton-zaicev · 2020-11-09T18:42:59Z

@SamBent, here is my .stf trace: scrollTrace.zip

jairbubbles · 2020-11-09T19:46:59Z

@SamBent You might have missed it in the other thread but when compiling locally and run with .NET Core 3.1.9 I can't reproduce it. Maybe it doesn't contain the fix that created that new issue?

SamBent · 2020-11-10T23:27:16Z

The two STF traces both show hangs when using MouseWheelDown to scroll forward from a position that is very near, but not quite at, the end of the data. Which means they're probably hitting the same root-cause. But they're different in that one (from @jairbubbles) has "tidy" heights for the viewport and the ListViewItems - 936, 17.6, +2.4 for non-leaves - while the other (from @anton-zaicev) has "messy" heights - 546.857142857143, 17.1428571428571. It probably doesn't matter, but I'm curious if anyone can explain this?

My guess is that the "resultsList" TreeView has UseLayoutRounding=true, and that both users have high-DPI monitors (or enabled DPI scaling), one at 125% yielding an effective pixel size of 100/125 = 0.8, the other at 175% yielding an effective pixel size of 100/175 = .571428571428571. The trace doesn't record either UseLayoutRounding or effective pixel size so I can't confirm my guess. Can you?

anton-zaicev · 2020-11-11T17:39:41Z

@SamBent, yes, I have 175% scale.
MainWindow has UseLayoutRounding=true.

KirillOsenkov · 2020-11-13T04:08:59Z

@jairbubbles @anton-zaicev if you email me at my first name dot last name at microsoft, I'll e-mail you instructions on how to test the fix. @SamBent was able to reproduce and produced a fix that needs to be tested before they'll allow us to check it in.

KirillOsenkov · 2020-11-20T03:21:37Z

OK great news! I was able to reproduce the bug and validate the fix.

Here's the video:

reproduced the hang with old binary
did not reproduce with old binary, kinda hung but not reliably
reproduced with old binary
=== installed the new binary ===
did not reproduce with new binary
did not reproduce with new binary
=== revert to old binary ===
reproduced with old binary

Repro guidance:

Windows Sandbox works great
Set your host OS DPI to 125%
In the Sandbox, install MSBuild Structured Log Viewer from https://msbuildlog.com
Open freeze.binlog and search for ReferenceCopyLocalPaths
Resize the viewer window such that the third scroll wheel scrolls almost to the end
Try to repro by using scroll wheel only - repeat steps 5 and 6 until you can reliably reproduce
Resize the Windows Sandbox window to exactly envelope the viewer window at that size
Close and reopen the viewer and maximize, presumably it will fill the size and the repro will be easy
Close the viewer, open C:\Windows\Microsoft.NET\assembly\GAC_MSIL\PresentationFramework\v4.0_4.0.0.0__31bf3856ad364e35 and replace PresentationFramework.dll with the one I sent
Repeat steps 4-6 - hopefully the bug won't reproduce

anton-zaicev · 2020-11-30T16:25:36Z

I had also validated the fix and can confirm that the bug is fixed.
@SamBent, what is the planned date of release for this fix?

SamBent · 2020-11-30T17:25:28Z

The fix is in the pipeline of review/approval/packaging/testing for inclusion in a .NET servicing update. If no problems arise, it should appear early in 2021. I'll post back to this thread when that happens.

anton-zaicev · 2021-02-12T10:20:12Z

@SamBent, are there any news about planned release date?

SamBent · 2021-02-12T17:21:06Z

Thanks for the reminder.

The fix was released in the .NET Update Preview on Jan 26, and in the .NET Update on Feb 9. The former is available for download, the latter is part of the monthly Windows Update payload. (The KB number depends on which OS version you're running.)

KirillOsenkov · 2021-03-26T18:03:04Z

I haven't seen any new reports of hangs for a couple of months now, so I'll consider this issue resolved. Thanks again everyone.

anton-zaicev · 2021-07-06T11:14:45Z

I think we had found regress - same bug after recent update, can you re-check that?
PresentationFramework.dll - version 4.8.4360.0

KirillOsenkov · 2021-07-06T15:01:41Z

Could you please upload PresentationFramework.dll here?

anton-zaicev · 2021-07-06T15:07:14Z

Here it is.
PresentationFramework.zip

KirillOsenkov · 2021-07-06T19:23:18Z

Hmm, I have the exact same version of the .dll, but I haven't noticed any hangs. I'll keep a close eye on it.

Any more details about the hang that you can share?

@anton-zaicev any chance that you could record another scrollTrace.stf like you did the last time?
#407 (comment)

#21 (comment)

anton-zaicev · 2021-07-06T19:28:55Z

Have you tried it with DPI value different from 100%?
I'll be back with scroll trace soon.

KirillOsenkov · 2021-07-06T19:47:37Z

I usually run with 200% DPI, maybe I should try 125% again (last time we had a repro with 125%)

anton-zaicev · 2021-07-08T19:25:47Z

It is not so easy to repro this time, but behaviour is all the same (DPI > 100%, same scroll actions), this time UseLayoutRounding property is involved. I end up with the same situation (scroll freezing) but only with the UseLayoutRounding="True" for virtualized container, with "False" value there is no any undesired behaviour and it is the temporary fix for now. I had collected scroll trace but not from the MSBuildStructuredLog app, there is same infinite scroll calculations loop as in the previous case. So, how I see, I should move to WPF repo and create issue there?
ScrollTrace.zip

KirillOsenkov · 2021-07-08T20:21:50Z

yes please open an issue on dotnet/wpf and tag @SamBent

jairbubbles mentioned this issue Oct 30, 2020

Infinite Measure/Arrange/Layout/Scroll loop possible with the treeview #21

Closed

KirillOsenkov pinned this issue Nov 9, 2020

jairbubbles changed the title ~~Freeze while selecting item in the tree~~ Freeze while selecting item in the results tree Nov 9, 2020

KirillOsenkov closed this as completed Mar 26, 2021

KirillOsenkov unpinned this issue Mar 26, 2021

KirillOsenkov reopened this Jul 6, 2021

anton-zaicev mentioned this issue Jul 13, 2021

Scroll freezing dotnet/wpf#4834

Open

KirillOsenkov mentioned this issue Mar 15, 2022

WPF: ArgumentNullException in VisualTreeUtils.AsNonNullVisual #570

Open

KirillOsenkov added the WPF label Nov 9, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Freeze while selecting item in the results tree #407

Freeze while selecting item in the results tree #407

jairbubbles commented Oct 28, 2020 •

edited

jairbubbles commented Oct 28, 2020

jairbubbles commented Oct 28, 2020

KirillOsenkov commented Oct 28, 2020

KirillOsenkov commented Oct 28, 2020

jairbubbles commented Oct 28, 2020

drewnoakes commented Oct 28, 2020

KirillOsenkov commented Oct 28, 2020

jairbubbles commented Oct 29, 2020

KirillOsenkov commented Oct 29, 2020

jairbubbles commented Oct 30, 2020

SamBent commented Nov 9, 2020

KirillOsenkov commented Nov 9, 2020

anton-zaicev commented Nov 9, 2020

jairbubbles commented Nov 9, 2020 •

edited

SamBent commented Nov 10, 2020

anton-zaicev commented Nov 11, 2020

KirillOsenkov commented Nov 13, 2020

KirillOsenkov commented Nov 20, 2020 •

edited

anton-zaicev commented Nov 30, 2020

SamBent commented Nov 30, 2020

anton-zaicev commented Feb 12, 2021

SamBent commented Feb 12, 2021

KirillOsenkov commented Mar 26, 2021

anton-zaicev commented Jul 6, 2021

KirillOsenkov commented Jul 6, 2021

anton-zaicev commented Jul 6, 2021

KirillOsenkov commented Jul 6, 2021

anton-zaicev commented Jul 6, 2021

KirillOsenkov commented Jul 6, 2021

anton-zaicev commented Jul 8, 2021

KirillOsenkov commented Jul 8, 2021

Freeze while selecting item in the results tree #407

Freeze while selecting item in the results tree #407

Comments

jairbubbles commented Oct 28, 2020 • edited

jairbubbles commented Oct 28, 2020

jairbubbles commented Oct 28, 2020

KirillOsenkov commented Oct 28, 2020

KirillOsenkov commented Oct 28, 2020

jairbubbles commented Oct 28, 2020

drewnoakes commented Oct 28, 2020

KirillOsenkov commented Oct 28, 2020

jairbubbles commented Oct 29, 2020

KirillOsenkov commented Oct 29, 2020

jairbubbles commented Oct 30, 2020

SamBent commented Nov 9, 2020

KirillOsenkov commented Nov 9, 2020

anton-zaicev commented Nov 9, 2020

jairbubbles commented Nov 9, 2020 • edited

SamBent commented Nov 10, 2020

anton-zaicev commented Nov 11, 2020

KirillOsenkov commented Nov 13, 2020

KirillOsenkov commented Nov 20, 2020 • edited

Repro guidance:

anton-zaicev commented Nov 30, 2020

SamBent commented Nov 30, 2020

anton-zaicev commented Feb 12, 2021

SamBent commented Feb 12, 2021

KirillOsenkov commented Mar 26, 2021

anton-zaicev commented Jul 6, 2021

KirillOsenkov commented Jul 6, 2021

anton-zaicev commented Jul 6, 2021

KirillOsenkov commented Jul 6, 2021

anton-zaicev commented Jul 6, 2021

KirillOsenkov commented Jul 6, 2021

anton-zaicev commented Jul 8, 2021

KirillOsenkov commented Jul 8, 2021

jairbubbles commented Oct 28, 2020 •

edited

jairbubbles commented Nov 9, 2020 •

edited

KirillOsenkov commented Nov 20, 2020 •

edited