Add chrome_crash_reporter_client_win.cc to the source file list for chrome_elf

base/files/file_path.cc

Index: base/files/file_path.cc
diff --git a/base/files/file_path.cc b/base/files/file_path.cc
index 2c199e131a56ee1976dded1dd25af158ac1c6252..cb00b446d1dd8799e8e7fdcf965892ee09940c0e 100644
--- a/base/files/file_path.cc
+++ b/base/files/file_path.cc
@@ -693,6 +693,10 @@ bool FilePath::ReadFromPickle(PickleIterator* iter) {
 
 int FilePath::CompareIgnoreCase(StringPieceType string1,
                                 StringPieceType string2) {
+  using CharUpperFunction = wchar_t*(WINAPI*)(wchar_t * str);
+  CharUpperFunction char_upper_api = reinterpret_cast<CharUpperFunction>(
+      ::GetProcAddress(::GetModuleHandle(L"user32.dll"), "CharUpperW"));

[ananta, 2016/06/10 05:54:57]

Will change this to use a static function pointer.

+  CHECK(char_upper_api);
   // Perform character-wise upper case comparison rather than using the
   // fully Unicode-aware CompareString(). For details see:
   // http://blogs.msdn.com/michkap/archive/2005/10/17/481600.aspx

[scottmg, 2016/06/10 19:36:19]

The linked blog post

http://web.archive.org/web/20100223192005/http://blogs.msdn.com/michkap/archi...

mentions "LCMapString with the LCMAP_UPPERCASE flag (and without the
LCMAP_LINGUISTIC_CASING flag!)" as being an option also, and that's in kernel32.

@@ -702,9 +706,9 @@ int FilePath::CompareIgnoreCase(StringPieceType string1,
   StringPieceType::const_iterator string2end = string2.end();
   for ( ; i1 != string1end && i2 != string2end; ++i1, ++i2) {
     wchar_t c1 =
-        (wchar_t)LOWORD(::CharUpperW((LPWSTR)(DWORD_PTR)MAKELONG(*i1, 0)));
+        (wchar_t)LOWORD(char_upper_api((LPWSTR)(DWORD_PTR)MAKELONG(*i1, 0)));
     wchar_t c2 =
-        (wchar_t)LOWORD(::CharUpperW((LPWSTR)(DWORD_PTR)MAKELONG(*i2, 0)));
+        (wchar_t)LOWORD(char_upper_api((LPWSTR)(DWORD_PTR)MAKELONG(*i2, 0)));
     if (c1 < c2)
       return -1;
     if (c1 > c2)