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

chrome/test/chromedriver/chrome_launcher.cc

Index: chrome/test/chromedriver/chrome_launcher.cc
diff --git a/chrome/test/chromedriver/chrome_launcher.cc b/chrome/test/chromedriver/chrome_launcher.cc
index 03a5e1f6593010b06c143a4dc736727c7f71f6a9..cc66330811bd65b1cb96b3745dbce43177d2f3cc 100644
--- a/chrome/test/chromedriver/chrome_launcher.cc
+++ b/chrome/test/chromedriver/chrome_launcher.cc
@@ -53,6 +53,7 @@
 #include <sys/types.h>
 #elif defined(OS_WIN)
 #include "base/win/scoped_handle.h"
+#include "chrome/test/chromedriver/keycode_text_conversion.h"
 #endif
 
 namespace {
@@ -344,6 +345,10 @@ Status LaunchDesktopChrome(
   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)