vs2015: fix rename of _crt_va_start

sandbox/win/sandbox_poc/main_ui_window.cc

 // Copyright (c) 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <windows.h>
 #include <CommCtrl.h>
 #include <commdlg.h>
 #include <time.h>
 #include <windowsx.h>
 #include <atlbase.h>
@@ skipping 606 matching lines @@
 }
 
 void MainUIWindow::AddDebugMessage(const wchar_t* format, ...) {
   DCHECK(format);
   if (!format)
     return;
 
   const int kMaxDebugBuffSize = 1024;
 
   va_list arg_list;
+#if _MSC_VER >= 1900
+  __crt_va_start(arg_list, format);

[Reid Kleckner, 2015/06/09 00:37:59]

Why not va_start from <stdarg.h>?

[scottmg, 2015/06/09 16:27:43]

No idea, it seemed strange enough that I left it alone, but I meant to ask too.

Justin, any reason that you know of?

[scottmg, 2015/06/09 16:28:28]

Hmm, attributed to one "initial.commit" so I'm guessing just random crazy
long-ago-sandbox-hacker. I'll switch it.

[scottmg, 2015/06/09 16:33:47]

and va_start is #defined to [_]_crt_va_start (as you probably already knew) so
seems silly. https://codereview.chromium.org/1168113007/

+#else
   _crt_va_start(arg_list, format);
+#endif
 
   wchar_t text[kMaxDebugBuffSize + 1];
   vswprintf_s(text, kMaxDebugBuffSize, format, arg_list);
   text[kMaxDebugBuffSize] = L'\0';
 
   InsertLineInListView(text);
 }
 
 
 void MainUIWindow::InsertLineInListView(wchar_t* debug_message) {
@@ skipping 22 matching lines @@
   item.iItem = ListView_GetItemCount(list_view_);
   item.iSubItem = 0;
   item.mask = LVIF_TEXT | LVIF_PARAM;
   item.pszText = message_time;
   item.lParam = 0;
 
   ListView_InsertItem(list_view_, &item);
 
   delete[] message_time;
 }