feat: WPERF-954 wpa plugin config

See merge request Linaro/WindowsPerf/vs-extension!110

=== ChangeLog ===

* chore: fix build

* chore: remove extra $ in path settings

* WPERF-997 Spe hardcode filters

See merge request Linaro/WindowsPerf/vs-extension!109

=== ChangeLog ===

* add default SPE filters in case wperf list doesn't support it

* chore: remove slopy extra empty line

* feat: update SPE filters groupbox header and placeholder texts

* feat: update spe sample displayed name to use the Feature name

* fix: update typo when unchecking SPE checkbox

* chore: make config check happen on each setting dialog call

* feat: disable SPE button on if SPE is unavailable

* feat: fix overhead percent in SPE output

* fix: line highlighter errors

* fix: lower case all filepaths

* feat: added memory to the form

* fix: add handleSPESamplingFinished call to the end of the sampling run

* fix: trailing character on spe filter list

* fix: remove default frequency on SPE

* fix: update highlighter file paths to be case insensitive

* feat: add SPE output handling

* feat: start handling SPE sampling display

* feat: add spe sampling settings

feat: add default slelected directory in picker

feat: add wpa plugins path default when checkbox is checked

Author: Alaaeddine-chakroun

WindowsPerfGUI/Options/WPAOptions.xaml.cs

@@ -30,6 +30,7 @@
 
 using System.Windows.Controls;
 using System.Windows.Forms;
+using WindowsPerfGUI.Utils;
 using UserControl = System.Windows.Controls.UserControl;
 
 namespace WindowsPerfGUI.Options
@@ -48,23 +49,19 @@ public WPAOptions()
 
         public void Initialize()
         {
-            string defaultSearchDir = Environment.GetEnvironmentVariable(
-                "WPA_ADDITIONAL_SEARCH_DIRECTORIES"
-            );
-
             string checkboxLabel = "Use default search directory";
-            if (!string.IsNullOrEmpty(defaultSearchDir))
+            if (!string.IsNullOrEmpty(WperfDefaults.DefaultWPASearchDir))
             {
-                checkboxLabel += $": {defaultSearchDir}";
+                checkboxLabel += $": {WperfDefaults.DefaultWPASearchDir}";
             }
 
             UseDefaultSearchLocation.Content = checkboxLabel;
 
-            if (!string.IsNullOrEmpty(defaultSearchDir))
+            if (!string.IsNullOrEmpty(WperfDefaults.DefaultWPASearchDir))
             {
                 EnvironmentVariableNotice.Visibility = System.Windows.Visibility.Visible;
                 EnvironmentVariableNotice.Text =
-                    $"WPA_ADDITIONAL_SEARCH_DIRECTORIES=\"{defaultSearchDir}\"";
+                    $"WPA_ADDITIONAL_SEARCH_DIRECTORIES=\"{WperfDefaults.DefaultWPASearchDir}\"";
             }
             UseDefaultSearchLocation.IsChecked = WPerfOptions.Instance.UseDefaultSearchDirectory;
             CustomSearchDir.IsEnabled = !WPerfOptions.Instance.UseDefaultSearchDirectory;
@@ -78,6 +75,10 @@ private void UseDefaultSearchLocation_Click(object sender, System.Windows.Routed
             CustomSearchDir.IsEnabled = !newValue;
             SelectDirectoryButton.IsEnabled = !newValue;
             UseDefaultSearchLocation.IsChecked = newValue;
+            if (newValue)
+            {
+                CustomSearchDir.Text = WperfDefaults.DefaultWPASearchDir;
+            }
             WPerfOptions.Instance.UseDefaultSearchDirectory = newValue;
             WPerfOptions.Instance.Save();
         }

WindowsPerfGUI/Options/WPerfPath.xaml.cs

@@ -131,7 +131,7 @@ private void SetWperfVersion(WperfVersion wperfVersion, bool shouldForce = false
 
                 TextBlock componentLabel = TextBlockFactory(component, offset);
                 TextBlock componentVersionLabel = TextBlockFactory(
-                    $"{componentVersion} ({gitVersion}) -- ${featureString}",
+                    $"{componentVersion} ({gitVersion}) -- {featureString}",
                     offset,
                     true
                 );
@@ -144,7 +144,6 @@ private void SetWperfVersion(WperfVersion wperfVersion, bool shouldForce = false
             {
                 return;
             }
-
         }
 
         private void SetPredefinedEventsAndMetrics(WperfList wperfList, bool shouldForce = false)
@@ -155,8 +154,6 @@ private void SetPredefinedEventsAndMetrics(WperfList wperfList, bool shouldForce
             {
                 return;
             }
-
-
         }
 
         public static TextBlock TextBlockFactory(string text, int offset, bool isRight = false)

WindowsPerfGUI/Utils/WperfDefaults.cs

@@ -38,6 +38,10 @@ static class WperfDefaults
 #nullable enable
         public static string? Frequency;
         public static int? TotalGPCNum;
+
+        public static string DefaultWPASearchDir = Environment.GetEnvironmentVariable(
+            "WPA_ADDITIONAL_SEARCH_DIRECTORIES"
+        );
 #nullable disable
     }
 }