Use Utils::StrDup in place of strdup

This avoids warnings about strdup being deprecated in favour of _strdup
on Windows.

TEST=manual build, observe no warnings are produced

Bug: flutter/flutter#59199
Change-Id: Ide390f3a719b9598bea0fc9976be3fb337f4de37
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/242386
Reviewed-by: Alexander Aprelev <[email protected]>
Reviewed-by: Ryan Macnak <[email protected]>
Commit-Queue: Alexander Aprelev <[email protected]>

Author: cbracken

runtime/vm/log_test.cc

@@ -5,6 +5,7 @@
 #include "platform/globals.h"
 
 #include "include/dart_tools_api.h"
+#include "platform/utils.h"
 #include "vm/dart_api_impl.h"
 #include "vm/dart_entry.h"
 #include "vm/debugger.h"
@@ -23,7 +24,7 @@ static void TestPrinter(const char* buffer) {
     free(const_cast<char*>(test_output_));
     test_output_ = NULL;
   }
-  test_output_ = strdup(buffer);
+  test_output_ = Utils::StrDup(buffer);
 
   // Also print to stdout to see the overall result.
   OS::PrintErr("%s", test_output_);

runtime/vm/tags.cc

@@ -4,6 +4,7 @@
 
 #include "vm/tags.h"
 
+#include "platform/utils.h"
 #include "vm/isolate.h"
 #include "vm/json_stream.h"
 #include "vm/native_entry.h"
@@ -159,7 +160,7 @@ void UserTags::AddStreamableTagName(const char* tag) {
       return;
     }
   }
-  subscribed_tags_.Add(strdup(tag));
+  subscribed_tags_.Add(Utils::StrDup(tag));
 }
 
 void UserTags::RemoveStreamableTagName(const char* tag) {