MacKeyboard: Don't generate keypress for non-printable char

base/mac/mac_util.h

 // Copyright (c) 2012 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.
 
 #ifndef BASE_MAC_MAC_UTIL_H_
 #define BASE_MAC_MAC_UTIL_H_
 
 #include <AvailabilityMacros.h>
 #include <Carbon/Carbon.h>
 #include <string>
 
 #include "base/base_export.h"
 #include "base/logging.h"
 
 #if defined(__OBJC__)
 #import <Foundation/Foundation.h>
 #else  // __OBJC__
 class NSImage;
+class NSString;
 #endif  // __OBJC__
 
 namespace base {
 
 class FilePath;
 
 namespace mac {
 
 // Full screen modes, in increasing order of priority.  More permissive modes
 // take predecence.
@@ skipping 84 matching lines @@
 BASE_EXPORT bool WasLaunchedAsLoginItemRestoreState();
 
 // Returns true if the current process was automatically launched as a
 // 'Login Item' with 'hide on startup' flag. Used to suppress opening windows.
 BASE_EXPORT bool WasLaunchedAsHiddenLoginItem();
 
 // Remove the quarantine xattr from the given file. Returns false if there was
 // an error, or true otherwise.
 BASE_EXPORT bool RemoveQuarantineAttribute(const FilePath& file_path);
 
+// Return true if 's' is empty or contains at least one printable character
+BASE_EXPORT bool StringIsPrintable(NSString* s);

[dtapuska, 2015/11/23 14:42:08]

This is too generic in the base..

Since you are only testing it inside the scope of the render_host_view_mac I'd
move it to an anonymous function

+
 // Run-time OS version checks. Use these instead of
 // base::SysInfo::OperatingSystemVersionNumbers. Prefer the "OrEarlier" and
 // "OrLater" variants to those that check for a specific version, unless you
 // know for sure that you need to check for a specific version.
 
 // Snow Leopard is Mac OS X 10.6, Darwin 10.
 BASE_EXPORT bool IsOSSnowLeopard();
 
 // Lion is Mac OS X 10.7, Darwin 11.
 BASE_EXPORT bool IsOSLion();
@@ skipping 107 matching lines @@
 // If any error occurs, none of the input pointers are touched.
 BASE_EXPORT bool ParseModelIdentifier(const std::string& ident,
                                       std::string* type,
                                       int32* major,
                                       int32* minor);
 
 }  // namespace mac
 }  // namespace base
 
 #endif  // BASE_MAC_MAC_UTIL_H_