Add reload test result feature

Author: penspanic

Diff for: Editor/PerformanceTestingReportViewerWindow.cs

@@ -27,7 +27,12 @@ public void CreateGUI()
             var asset = AssetDatabase.LoadAssetAtPath<VisualTreeAsset>(path);
             asset.CloneTree(rootVisualElement);
             viewer = rootVisualElement.Q<Editor.UI.PerformanceTestReportViewer>();
+            viewer.RequestReload += () => InitViewer(viewer);
+            InitViewer(viewer);
+        }
 
+        private void InitViewer(Editor.UI.PerformanceTestReportViewer viewer)
+        {
             if (File.Exists(EditorConfigPath))
                 viewerOptions = ViewerOptions.DeserializeFromString(File.ReadAllText(EditorConfigPath));
             else
@@ -38,6 +43,7 @@ public void CreateGUI()
             ViewerModule.Instance.ViewerOptions = viewerOptions;
 
             viewer.Init(viewerOptions);
+            
         }
 
         private void Update()

Diff for: Editor/UI/ChartUtility.cs

@@ -155,9 +155,10 @@ public static void GenerateBars(int index, int totalCount, double min, double ma
             double gap = max - min;
             float yPosStart = ChartUtility.YPosByIndex(chartRect, index, totalCount, out float ySize);
             float ySpacing = 1;
-            float eachYSize = (ySize - (values.Length + 1) * ySpacing) / values.Length;
+            float yMargin = 6;
+            float eachYSize = (ySize - (2 * yMargin) - (values.Length * ySpacing)) / values.Length;
             float realYSize = Math.Min(10, eachYSize);
-            float yMargin = (ySize - realYSize * values.Length - ySpacing * (values.Length - 1)) / 2f;
+            yMargin = (ySize - (realYSize + ySpacing) * values.Length) / 2f;
             for (int i = 0; i < values.Length; i++)
             {
                 float yPos = yPosStart + yMargin + i * ySpacing + i * realYSize; 

Diff for: Editor/UI/Layouts/BarsVisualizer.uxml

@@ -1,11 +1,9 @@
 <ui:UXML xmlns:ui="UnityEngine.UIElements" xmlns:uie="UnityEditor.UIElements" xsi="http://www.w3.org/2001/XMLSchema-instance" engine="UnityEngine.UIElements" editor="UnityEditor.UIElements" noNamespaceSchemaLocation="../../../../../UIElementsSchema/UIElements.xsd" editor-extension-mode="False">
     <ui:VisualElement name="VisualElement" style="flex-grow: 1; min-width: 500px; min-height: 400px; border-left-color: rgb(29, 29, 29); border-right-color: rgb(29, 29, 29); border-top-color: rgb(29, 29, 29); border-bottom-color: rgb(29, 29, 29); border-left-width: 3px; border-right-width: 3px; border-top-width: 3px; border-bottom-width: 3px; margin-left: 1px; margin-right: 1px; margin-top: 1px; margin-bottom: 1px;">
-        <ui:Label tabindex="-1" display-tooltip-when-elided="true" name="headerArea" style="background-color: rgb(55, 55, 55);">
-            <ui:Label tabindex="-1" text="Label" display-tooltip-when-elided="true" name="header" style="margin-left: 0; margin-right: 0; margin-top: 0; margin-bottom: 0; padding-left: 0; padding-right: 0; padding-top: 0; padding-bottom: 0; font-size: 16px; -unity-text-align: middle-center; -unity-font-style: bold;" />
-        </ui:Label>
+        <ui:Label tabindex="-1" text="Label" display-tooltip-when-elided="true" name="header" style="margin-left: 20px; margin-right: 20px; margin-top: 2px; margin-bottom: 2px; padding-left: 0; padding-right: 0; padding-top: 0; padding-bottom: 0; font-size: 16px; -unity-text-align: middle-center; -unity-font-style: bold; text-overflow: ellipsis; white-space: nowrap;" />
         <ui:VisualElement name="chartArea" style="flex-grow: 1; flex-direction: row; padding-top: 20px; padding-left: 20px;">
-            <ui:VisualElement name="renderArea" style="flex-grow: 1; border-left-color: rgba(0, 0, 0, 0.61); border-right-color: rgba(0, 0, 0, 0.61); border-top-color: rgba(0, 0, 0, 0.61); border-bottom-color: rgba(0, 0, 0, 0.61); border-left-width: 2px; border-right-width: 2px; border-top-width: 2px; border-bottom-width: 2px;" />
-            <ui:VisualElement name="yAxis" style="flex-grow: 1; width: 80px; max-width: 50%; min-width: 50%;" />
+            <ui:VisualElement name="renderArea" style="flex-grow: 1; border-left-color: rgba(0, 0, 0, 0.61); border-right-color: rgba(0, 0, 0, 0.61); border-top-color: rgba(0, 0, 0, 0.61); border-bottom-color: rgba(0, 0, 0, 0.61); border-left-width: 2px; border-right-width: 2px; border-top-width: 2px; border-bottom-width: 2px; min-width: 35%;" />
+            <ui:VisualElement name="yAxis" style="flex-grow: 1; width: 80px; max-width: 50%; min-width: 10%;" />
         </ui:VisualElement>
         <ui:VisualElement name="descriptionArea" style="margin-left: 20px;">
             <ui:VisualElement name="xAxis" style="flex-grow: 1; background-color: rgba(0, 0, 0, 0); padding-top: 0; padding-right: 0; padding-bottom: 0; padding-left: 0; margin-top: 5px; max-height: 25px; min-height: 25px; height: 25px;" />

Diff for: Editor/UI/PerformanceTestReportViewer.cs

@@ -18,8 +18,10 @@ public class PerformanceTestReportViewer : VisualElement
 
         private static readonly string layoutPath = $"{Constants.LayoutPath}/{nameof(PerformanceTestReportViewer)}.uxml";
 
+        public event Action RequestReload;
         public event Action RequestRefresh;
-        
+
+        private Button reloadButton;
         private ScrollView sampleTypesScrollView;
         private ScrollView contextsScrollView;
         private TestResultListView testResultListView;
@@ -37,6 +39,7 @@ public PerformanceTestReportViewer()
         {
             AssetDatabase.LoadAssetAtPath<VisualTreeAsset>(layoutPath).CloneTree(this);
 
+            reloadButton = this.Q<Button>(nameof(reloadButton));
             sampleTypesScrollView = this.Q<ScrollView>(nameof(sampleTypesScrollView));
             contextsScrollView = this.Q<ScrollView>(nameof(contextsScrollView));
             testResultListView = this.Q<TestResultListView>(nameof(testResultListView));
@@ -46,6 +49,8 @@ public PerformanceTestReportViewer()
             viewerTypeDropdown = this.Q<DropdownField>(nameof(viewerTypeDropdown));
             tagsScrollView = this.Q<ScrollView>(nameof(tagsScrollView));
 
+            testResultListView.OnTreeViewItemSelected += OnTreeViewItemSelected;
+            reloadButton.clicked += () => RequestReload?.Invoke();
             RequestRefresh += Refresh;
         }
 
@@ -58,8 +63,6 @@ public void Init(ViewerOptions viewerOptions)
                 viewerOptions.IsSampleTargetOn, viewerOptions.SetIsSampleTargetOn,
                 () => RequestRefresh?.Invoke());
             testResultListView.Init(viewerOptions);
-
-            testResultListView.OnTreeViewItemSelected += OnTreeViewItemSelected;
         }
 
         private void Refresh()

Diff for: Editor/UI/TestResultListView.cs

@@ -65,18 +65,7 @@ public TestResultListView()
         {
             AssetDatabase.LoadAssetAtPath<VisualTreeAsset>(layoutPath).CloneTree(this);
             resultsTreeView = this.Q<TreeView>(nameof(resultsTreeView));
-        }
-
-        public void Init(ViewerOptions viewerOptions)
-        {
-            this.viewerOptions = viewerOptions;
 
-            resultsTreeView.Clear();
-
-            if (ViewerModule.Instance.PerformanceTestResults == null)
-                return;
-
-            resultsTreeView.SetRootItems(BuildTreeDataList());
             resultsTreeView.makeItem = MakeItem;
             resultsTreeView.bindItem = BindItem;
 
@@ -88,6 +77,17 @@ public void Init(ViewerOptions viewerOptions)
                 var item = selectedList.Single() as ITreeViewItem;
                 OnTreeViewItemSelected?.Invoke(item);
             };
+        }
+
+        public void Init(ViewerOptions viewerOptions)
+        {
+            this.viewerOptions = viewerOptions;
+            resultsTreeView.Clear();
+
+            if (ViewerModule.Instance.PerformanceTestResults == null)
+                return;
+
+            resultsTreeView.SetRootItems(BuildTreeDataList());
             resultsTreeView.ExpandRootItems();
         }
 

Diff for: Editor/UI/Visualizers/BarsVisualizer.cs

@@ -174,8 +174,8 @@ private void OnMouseMove(MouseMoveEvent evt)
 
                 if (_groupNames != null)
                 {
-                    int groupIndex = index % _groupNames.Length;
-                    int valueIndex = index / _groupNames.Length;
+                    int groupIndex = index / showingElements.Length;
+                    int valueIndex = index % showingElements.Length;
                     valueTooltip.text = showingElements[valueIndex].Values[groupIndex]!.Value.ToString("F2");
                 }
                 else