Fix EventSource to stop ignoring EventCommand.SendManifest (#27979)

sywhang · Anipik · commit 942f3ac191bf · 2020-01-14T10:35:31.000-08:00
diff --git a/src/System.Private.CoreLib/shared/System/Diagnostics/Tracing/EventSource.cs b/src/System.Private.CoreLib/shared/System/Diagnostics/Tracing/EventSource.cs
@@ -2820,14 +2820,12 @@ internal void DoCommand(EventCommandEventArgs commandArgs)
                 }
                 else
                 {
-#if !FEATURE_PERFTRACING
                     if (commandArgs.Command == EventCommand.SendManifest)
                     {
                         // TODO: should we generate the manifest here if we hadn't already?
                         if (m_rawManifest != null)
                             SendManifest(m_rawManifest);
                     }
-#endif
 
                     // These are not used for non-update commands and thus should always be 'default' values
                     // Debug.Assert(enable == true);

Original file line number	Diff line number	Diff line change
`@@ -2820,14 +2820,12 @@ internal void DoCommand(EventCommandEventArgs commandArgs)`
`2820`	`2820`	`}`
`2821`	`2821`	`else`
`2822`	`2822`	`{`
`2823`		`-#if !FEATURE_PERFTRACING`
`2824`	`2823`	`if (commandArgs.Command == EventCommand.SendManifest)`
`2825`	`2824`	`{`
`2826`	`2825`	`// TODO: should we generate the manifest here if we hadn't already?`
`2827`	`2826`	`if (m_rawManifest != null)`
`2828`	`2827`	`SendManifest(m_rawManifest);`
`2829`	`2828`	`}`
`2830`		`-#endif`
`2831`	`2829`
`2832`	`2830`	`// These are not used for non-update commands and thus should always be 'default' values`
`2833`	`2831`	`// Debug.Assert(enable == true);`