Adding a unit test to RenderWidgetHostViewAura

content/browser/renderer_host/render_widget_host_view_aura_unittest.cc

 // 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.
 
 #include "content/browser/renderer_host/render_widget_host_view_aura.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <tuple>
@@ skipping 4732 matching lines @@
     views_[index]->SelectionChanged(expected_text, 0U, gfx::Range(0, 5));
 
     // Retrieve the selected text from clipboard and verify it is as expected.
     base::string16 result_text;
     clipboard->ReadText(ui::CLIPBOARD_TYPE_SELECTION, &result_text);
     EXPECT_EQ(expected_text, result_text);
   }
 }
 #endif
 
+// This test verifies that when any view on the page cancels an ongoing
+// composition, the RenderWidgetHostViewAura will receive the notification and
+// the current composition is canceled.
+TEST_F(InputMethodStateAuraTest, ImeCancelCompositionForAllViews) {
+  for (auto* view : views_) {
+    ActivateViewForTextInputManager(view, ui::TEXT_INPUT_TYPE_TEXT);
+    // There is no composition in the beginning.
+    EXPECT_FALSE(has_composition_text());
+    SetHasCompositionTextToTrue();
+    view->ImeCancelComposition();

[Charlie Reis, 2017/01/17 22:31:38]

nit: Put EXPECT_TRUE(has_composition_text()) before this line?

[EhsanK, 2017/01/17 22:45:47]

It will definitely help with readability of the test. But we already have the
same expect inside SetHasCompositionTextToTrue(). Is it still recommended to put
the same one here as well?

[Charlie Reis, 2017/01/17 22:49:20]

Oh, I didn't see that before.  It's ok as is then.

+    // The composition must have been canceled.
+    EXPECT_FALSE(has_composition_text());
+  }
+}
+
 }  // namespace content