Force U.S. English keyboard layout for TextfieldTest.KeysWithModifiersTest

ui/events/test/keyboard_layout_mac.cc

+// Copyright 2016 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 "ui/events/test/keyboard_layout.h"
+
+#include "base/logging.h"
+
+namespace ui {
+
+PlatformKeyboardLayout GetPlatformKeyboardLayout(KeyboardLayout layout) {
+  // Right now tests only need US English.  If other layouts need to be
+  // supported in the future this code should be extended.
+  DCHECK_EQ(KEYBOARD_LAYOUT_ENGLISH_US, layout);

[Wez, 2016/09/13 20:03:16]

nit: I generally recommend leaving a blank-line between logical blocks in code -
in this case the DCHECK is not related to the input_source_id definition.

[Tomasz Moniuszko, 2016/09/14 13:56:06]

Done.

+  const char input_source_id[] = "com.apple.keylayout.US";

[Wez, 2016/09/13 20:03:16]

kInputSourceId? Or kUsInputSourceId?

[Tomasz Moniuszko, 2016/09/14 13:56:06]

Done.

+
+  base::ScopedCFTypeRef<CFMutableDictionaryRef> filter_dict(

[Wez, 2016/09/13 20:03:16]

Can we give the variables in this section more helpful names, e.g. this is the
input-source-list filter, right? Similarly the id_ref below is really the
input-source-id-ref, for example.

[Tomasz Moniuszko, 2016/09/14 13:56:06]

I moved this code from chrome/test/chromedriver/test_util.cc. Sure, I can rename
variables.

[Wez, 2016/09/16 00:57:18]

Thanks!

+      CFDictionaryCreateMutable(kCFAllocatorDefault, 1,
+                                &kCFTypeDictionaryKeyCallBacks,
+                                &kCFTypeDictionaryValueCallBacks));
+  base::ScopedCFTypeRef<CFStringRef> id_ref(CFStringCreateWithCString(
+      kCFAllocatorDefault, input_source_id, kCFStringEncodingUTF8));
+  CFDictionaryAddValue(filter_dict, kTISPropertyInputSourceID, id_ref);
+  base::ScopedCFTypeRef<CFArrayRef> sources(
+      TISCreateInputSourceList(filter_dict, true));
+  if (CFArrayGetCount(sources) != 1)
+    return PlatformKeyboardLayout();
+
+  return PlatformKeyboardLayout(
+      (TISInputSourceRef)CFArrayGetValueAtIndex(sources, 0));
+}
+
+PlatformKeyboardLayout ScopedKeyboardLayout::GetActiveLayout() {
+  return PlatformKeyboardLayout(TISCopyCurrentKeyboardInputSource());
+}
+
+void ScopedKeyboardLayout::ActivateLayout(PlatformKeyboardLayout layout) {
+  DCHECK(layout);
+  OSStatus result = TISSelectInputSource(layout);
+  DCHECK_EQ(noErr, result);
+}
+
+}  // namespace ui