MacViews: Fix crash caused due to nil TextInputClient.

ui/views/cocoa/bridged_native_widget_unittest.mm

 // Copyright 2014 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.
 
 #import "ui/views/cocoa/bridged_native_widget.h"
 
 #import <Cocoa/Cocoa.h>
 
 #include <memory>
 
@@ skipping 39 matching lines @@
 #define EXPECT_EQ_RANGE_3(expected_literal, expected_cocoa, actual_views) \
   EXPECT_EQ_RANGE(expected_literal, actual_views);                        \
   EXPECT_EQ_RANGE(expected_cocoa, actual_views);
 
 #define EXPECT_EQ_3(expected_literal, expected_cocoa, actual_views) \
   EXPECT_EQ(expected_literal, actual_views);                        \
   EXPECT_EQ(expected_cocoa, actual_views);
 
 namespace {
 
+// Implemented NSResponder action messages for use in tests.
+NSArray* const kMoveActions = @[
+  @"moveForward:",
+  @"moveRight:",
+  @"moveBackward:",
+  @"moveLeft:",
+  @"moveUp:",
+  @"moveDown:",
+  @"moveWordForward:",
+  @"moveWordBackward:",
+  @"moveToBeginningOfLine:",
+  @"moveToEndOfLine:",
+  @"moveToBeginningOfParagraph:",
+  @"moveToEndOfParagraph:",
+  @"moveToEndOfDocument:",
+  @"moveToBeginningOfDocument:",
+  @"pageDown:",
+  @"pageUp:",
+  @"moveWordRight:",
+  @"moveWordLeft:",
+  @"moveToLeftEndOfLine:",
+  @"moveToRightEndOfLine:"
+];
+
+NSArray* const kSelectActions = @[
+  @"moveBackwardAndModifySelection:",
+  @"moveForwardAndModifySelection:",
+  @"moveWordForwardAndModifySelection:",
+  @"moveWordBackwardAndModifySelection:",
+  @"moveUpAndModifySelection:",
+  @"moveDownAndModifySelection:",
+  @"moveToBeginningOfLineAndModifySelection:",
+  @"moveToEndOfLineAndModifySelection:",
+  @"moveToBeginningOfParagraphAndModifySelection:",
+  @"moveToEndOfParagraphAndModifySelection:",
+  @"moveToEndOfDocumentAndModifySelection:",
+  @"moveToBeginningOfDocumentAndModifySelection:",
+  @"pageDownAndModifySelection:",
+  @"pageUpAndModifySelection:",
+  @"moveParagraphForwardAndModifySelection:",
+  @"moveParagraphBackwardAndModifySelection:",
+  @"moveRightAndModifySelection:",
+  @"moveLeftAndModifySelection:",
+  @"moveWordRightAndModifySelection:",
+  @"moveWordLeftAndModifySelection:",
+  @"moveToLeftEndOfLineAndModifySelection:",
+  @"moveToRightEndOfLineAndModifySelection:"
+];
+
+NSArray* const kDeleteActions = @[
+  @"deleteForward:", @"deleteBackward:", @"deleteWordForward:",
+  @"deleteWordBackward:", @"deleteToBeginningOfLine:", @"deleteToEndOfLine:",
+  @"deleteToBeginningOfParagraph:", @"deleteToEndOfParagraph:"
+];
+
+NSArray* const kMiscActions = @[ @"insertText:", @"cancelOperation:" ];
+
 // Empty range shortcut for readibility.
 NSRange EmptyRange() {
   return NSMakeRange(NSNotFound, 0);
 }
 
 // Sets |composition_text| as the composition text with caret placed at
 // |caret_pos| and updates |caret_range|.
 void SetCompositionText(ui::TextInputClient* client,
                         const base::string16& composition_text,
                         const int caret_pos,
@@ skipping 867 matching lines @@
 TEST_F(BridgedNativeWidgetTest, TextInput_DeleteToBeginningOfParagraph) {
   TestDeleteBeginning(@selector(deleteToBeginningOfParagraph:));
 }
 
 TEST_F(BridgedNativeWidgetTest, TextInput_DeleteToEndOfParagraph) {
   TestDeleteEnd(@selector(deleteToEndOfParagraph:));
 }
 
 // Test move commands against expectations set by |dummy_text_view_|.
 TEST_F(BridgedNativeWidgetTest, TextInput_MoveEditingCommands) {
-  NSArray* selectors = @[
-    @"moveForward:",
-    @"moveRight:",
-    @"moveBackward:",
-    @"moveLeft:",
-    @"moveUp:",
-    @"moveDown:",
-    @"moveWordForward:",
-    @"moveWordBackward:",
-    @"moveToBeginningOfLine:",
-    @"moveToEndOfLine:",
-    @"moveToBeginningOfParagraph:",
-    @"moveToEndOfParagraph:",
-    @"moveToEndOfDocument:",
-    @"moveToBeginningOfDocument:",
-    @"pageDown:",
-    @"pageUp:",
-    @"moveWordRight:",
-    @"moveWordLeft:",
-    @"moveToLeftEndOfLine:",
-    @"moveToRightEndOfLine:"
-  ];
-  TestEditingCommands(selectors);
+  TestEditingCommands(kMoveActions);
 }
 
 // Todo(karandeepb): Enable this test once the behavior of all move and select
 // commands are fixed.
 // Test move and select commands against expectations set by |dummy_text_view_|.
 TEST_F(BridgedNativeWidgetTest,
        TextInput_MoveAndSelectEditingCommands_DISABLED) {
-  NSArray* selectors = @[
-    @"moveBackwardAndModifySelection:",
-    @"moveForwardAndModifySelection:",
-    @"moveWordForwardAndModifySelection:",
-    @"moveWordBackwardAndModifySelection:",
-    @"moveUpAndModifySelection:",
-    @"moveDownAndModifySelection:",
-    @"moveToBeginningOfLineAndModifySelection:",
-    @"moveToEndOfLineAndModifySelection:",
-    @"moveToBeginningOfParagraphAndModifySelection:",
-    @"moveToEndOfParagraphAndModifySelection:",
-    @"moveToEndOfDocumentAndModifySelection:",
-    @"moveToBeginningOfDocumentAndModifySelection:",
-    @"pageDownAndModifySelection:",
-    @"pageUpAndModifySelection:",
-    @"moveParagraphForwardAndModifySelection:",
-    @"moveParagraphBackwardAndModifySelection:",
-    @"moveRightAndModifySelection:",
-    @"moveLeftAndModifySelection:",
-    @"moveWordRightAndModifySelection:",
-    @"moveWordLeftAndModifySelection:",
-    @"moveToLeftEndOfLineAndModifySelection:",
-    @"moveToRightEndOfLineAndModifySelection:"
-  ];
-  TestEditingCommands(selectors);
+  TestEditingCommands(kSelectActions);
 }
 
 // Test delete commands against expectations set by |dummy_text_view_|.
 TEST_F(BridgedNativeWidgetTest, TextInput_DeleteCommands) {
-  NSArray* selectors = @[
-    @"deleteForward:", @"deleteBackward:", @"deleteWordForward:",
-    @"deleteWordBackward:", @"deleteToBeginningOfLine:", @"deleteToEndOfLine:",
-    @"deleteToBeginningOfParagraph:", @"deleteToEndOfParagraph:"
-  ];
-  TestEditingCommands(selectors);
+  TestEditingCommands(kDeleteActions);
+}
+
+// Test that we don't crash during an action message even if the TextInputClient
+// is nil. Regression test for crbug.com/615745.
+TEST_F(BridgedNativeWidgetTest, NilTextInputClient) {
+  [ns_view_ setTextInputClient:nil];
+  NSMutableArray* selectors = [NSMutableArray array];
+  [selectors addObjectsFromArray:kMoveActions];
+  [selectors addObjectsFromArray:kSelectActions];
+  [selectors addObjectsFromArray:kDeleteActions];
+  [selectors addObjectsFromArray:kMiscActions];
+
+  for (NSString* selector in selectors)
+    [ns_view_ doCommandBySelector:NSSelectorFromString(selector)];
 }
 
 // Test firstRectForCharacterRange:actualRange for cases where query range is
 // empty or outside composition range.
 TEST_F(BridgedNativeWidgetTest, TextInput_FirstRectForCharacterRange_Caret) {
   InstallTextField("");
   ui::TextInputClient* client = [ns_view_ textInputClient];
 
   // No composition. Ensure bounds and range corresponding to the current caret
   // position are returned.
@@ skipping 146 matching lines @@
   [center postNotificationName:NSWindowDidExitFullScreenNotification
                         object:window];
   EXPECT_EQ(1, [window ignoredToggleFullScreenCount]);  // No change.
   EXPECT_FALSE(bridge()->target_fullscreen_state());
 
   widget_->CloseNow();
 }
 
 }  // namespace test
 }  // namespace views