[chromedriver] Fixed window SendKeys (single quote missing) problem

chrome/test/chromedriver/chrome_launcher.cc

 // Copyright (c) 2013 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 "chrome/test/chromedriver/chrome_launcher.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/base64.h"
@@ skipping 35 matching lines @@
 #include "crypto/rsa_private_key.h"
 #include "crypto/sha2.h"
 #include "third_party/zlib/google/zip.h"
 
 #if defined(OS_POSIX)
 #include <fcntl.h>
 #include <sys/stat.h>
 #include <sys/types.h>
 #elif defined(OS_WIN)
 #include "base/win/scoped_handle.h"
+#include "chrome/test/chromedriver/keycode_text_conversion.h"
 #endif
 
 namespace {
 
 const char* const kCommonSwitches[] = {
     "ignore-certificate-errors", "metrics-recording-only"};
 
 #if defined(OS_LINUX)
 const char kEnableCrashReport[] = "enable-crash-reporter-for-testing";
 #endif
@@ skipping 271 matching lines @@
   if (!CreatePipe(&out_read, &out_write, &sa_attr, 0))
       return Status(kUnknownError, "CreatePipe() - Pipe creation failed");
   // Prevent handle leak.
   base::win::ScopedHandle scoped_out_read(out_read);
   base::win::ScopedHandle scoped_out_write(out_write);
 
   options.stdout_handle = out_write;
   options.stderr_handle = out_write;
   options.stdin_handle = GetStdHandle(STD_INPUT_HANDLE);
   options.inherit_handles = true;
+  // use US keyboard layout for key conversion
+  if (!SwitchToUSKeyboardLayout()) {
+    VLOG(0) << "Can not set to US keyboard layout\n";

[samuong, 2015/01/23 19:36:58]

We should also mention that some keycodes may be interpreted incorrectly.
Also you don't need to put a '\n' at the end of this.

[andrewcheng1, 2015/01/24 01:40:00]

Done.

[andrewcheng1, 2015/02/19 00:56:59]

Done.

+  }

[samuong, 2015/01/23 19:36:58]

You mentioned earlier that this is a per-process setting, so I think it would
make more sense to do this once when ChromeDriver starts, rather than once every
time we launch a new Chrome session. On Linux, we set the locale for keycode
conversion near the top of main() - is there any reason not to do the same on
Windows?

https://code.google.com/p/chromium/codesearch#chromium/src/chrome/test/chrome...

[andrewcheng1, 2015/01/24 01:40:00]

sent you email -  main()  will not work  (reason not clear)

[andrewcheng1, 2015/02/19 00:56:59]

 sent you email -  main()  will not work  (which it should)
 work only per thread

----------- following come from M.S document ------------
Prior to Windows 8: If the specified input locale identifier is not already
loaded, the function loads and activates the input locale identifier for the
current thread.

also tried setting
KLF_SETFORPROCESS
Prior to Windows 8: This flag is valid only with KLF_ACTIVATE. Activates the
specified input locale identifier for the entire process and sends the
WM_INPUTLANGCHANGE message to the current thread's Focus or Active window.
Typically, LoadKeyboardLayout activates an input locale identifier only for the
current thread.

 #endif
 
 #if defined(OS_WIN)
   std::string command_string = base::WideToUTF8(command.GetCommandLineString());
 #else
   std::string command_string = command.GetCommandLineString();
 #endif
   VLOG(0) << "Launching chrome: " << command_string;
   base::Process process = base::LaunchProcess(command, options);
   if (!process.IsValid())
@@ skipping 478 matching lines @@
   // Write empty "First Run" file, otherwise Chrome will wipe the default
   // profile that was written.
   if (base::WriteFile(
           user_data_dir.Append(chrome::kFirstRunSentinel), "", 0) != 0) {
     return Status(kUnknownError, "failed to write first run file");
   }
   return Status(kOk);
 }
 
 }  // namespace internal