Fix: Cursor missing in omnibox after entering a alphabet in NTP 'Search box'

ui/views/controls/textfield/textfield_unittest.cc

 // 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.
 
 #include "ui/views/controls/textfield/textfield.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <set>
@@ skipping 235 matching lines @@
 bool MockInputMethod::HasComposition() {
   return composition_.text.length() || result_text_.length();
 }
 
 void MockInputMethod::ClearComposition() {
   composition_.Clear();
   result_text_.clear();
 }
 
 // A Textfield wrapper to intercept OnKey[Pressed|Released]() ressults.
+// After user event, the textfield sets cursor_enabled in RenderText to true.
 class TestTextfield : public views::Textfield {
  public:
   TestTextfield()
      : Textfield(),
        key_handled_(false),
        key_received_(false),
        weak_ptr_factory_(this) {}
 
   // ui::TextInputClient overrides:
   void InsertChar(const ui::KeyEvent& e) override {
@@ skipping 123 matching lines @@
     // paste action did not change the content. So |new_contents| may match
     // |last_contents_|. For more info, see http://crbug.com/79002
     last_contents_ = new_contents;
   }
 
   void OnBeforeUserAction(Textfield* sender) override {
     ++on_before_user_action_;
   }
 
   void OnAfterUserAction(Textfield* sender) override {
+    textfield_->SetCursorEnabled(true);
     ++on_after_user_action_;
   }
 
   void OnAfterCutOrCopy(ui::ClipboardType clipboard_type) override {
     copied_to_clipboard_ = clipboard_type;
   }
 
   void InitTextfield() {
     InitTextfields(1);
   }
@@ skipping 2696 matching lines @@
   textfield_->SetTextInputType(ui::TEXT_INPUT_TYPE_PASSWORD);
   ui::AXNodeData node_data_protected;
   node_data_protected.state = 0;
   textfield_->GetAccessibleNodeData(&node_data_protected);
   EXPECT_EQ(ui::AX_ROLE_TEXT_FIELD, node_data_protected.role);
   EXPECT_EQ(ASCIIToUTF16("********"),
             node_data_protected.GetString16Attribute(ui::AX_ATTR_VALUE));
   EXPECT_TRUE(node_data_protected.HasStateFlag(ui::AX_STATE_PROTECTED));
 }
 
-// Test if the cursor visibility is controlled by |cursor_enabled_| in
+// Verifies if the cursor visibility is controlled by |cursor_enabled_| in
 // RenderText.
 TEST_F(TextfieldTest, CursorVisibility) {
   InitTextfield();
   gfx::RenderText* render_text = test_api_->GetRenderText();
 
   render_text->SetCursorEnabled(false);
   textfield_->OnBlur();
   textfield_->OnFocus();
   EXPECT_FALSE(test_api_->IsCursorVisible());
 
   render_text->SetCursorEnabled(true);
   textfield_->OnBlur();
   textfield_->OnFocus();
   EXPECT_TRUE(test_api_->IsCursorVisible());
 }
 
+// Verifies if the cursor visibility is changed to true from false after insert
+// charactor in textfield.
+TEST_F(TextfieldTest, CursorVisibilityChangeAfterInserting) {
+  InitTextfield();
+
+  test_api_->GetRenderText()->SetCursorEnabled(false);
+  textfield_->OnBlur();
+  textfield_->OnFocus();
+  EXPECT_FALSE(test_api_->IsCursorVisible());
+  SendKeyEvent('a');
+  EXPECT_TRUE(test_api_->IsCursorVisible());
+
+  SetClipboardText(ui::CLIPBOARD_TYPE_COPY_PASTE, "abc");
+  textfield_->SetText(base::string16());
+  SendKeyEvent(ui::VKEY_V, false, true);
+  EXPECT_STR_EQ("abc", textfield_->text());
+  EXPECT_TRUE(test_api_->IsCursorVisible());
+}
+
+// Verifies if the cursor visibility is changed to true from false after paste
+// some text to the textfield.
+TEST_F(TextfieldTest, CursorVisibilityChangeAfterPaste) {
+  InitTextfield();
+
+  test_api_->GetRenderText()->SetCursorEnabled(false);
+  textfield_->OnBlur();
+  textfield_->OnFocus();
+  SetClipboardText(ui::CLIPBOARD_TYPE_COPY_PASTE, "abc");
+  textfield_->SetText(base::string16());
+  SendKeyEvent(ui::VKEY_V, false, true);
+  EXPECT_STR_EQ("abc", textfield_->text());
+  EXPECT_TRUE(test_api_->IsCursorVisible());
+}
+
 }  // namespace views