Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(242)

Unified Diff: content/browser/renderer_host/render_widget_host_view_aura_unittest.cc

Issue 2856093003: Update TextSelection for non-user initiated events (Closed)
Patch Set: Update TextSelection for non-user initiated events Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/render_widget_host_view_aura_unittest.cc
diff --git a/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc b/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc
index 3694c7968fe6c78e9840593a7210d5a710739db2..b2cfe0688e73324ec0615dae917dfa9d8ad6dc77 100644
--- a/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc
+++ b/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc
@@ -5455,7 +5455,7 @@ TEST_F(InputMethodStateAuraTest, GetSelectedText) {
for (auto index : active_view_sequence_) {
render_widget_host_delegate()->set_focused_widget(
RenderWidgetHostImpl::From(views_[index]->GetRenderWidgetHost()));
- views_[index]->SelectionChanged(text, offset, selection_range);
+ views_[index]->SelectionChanged(text, offset, selection_range, true);
base::string16 expected_text = text.substr(
selection_range.GetMin() - offset, selection_range.length());
@@ -5477,7 +5477,7 @@ TEST_F(InputMethodStateAuraTest, GetTextRange) {
render_widget_host_delegate()->set_focused_widget(
RenderWidgetHostImpl::From(views_[index]->GetRenderWidgetHost()));
gfx::Range expected_range(offset, offset + text.length());
- views_[index]->SelectionChanged(text, offset, selection_range);
+ views_[index]->SelectionChanged(text, offset, selection_range, true);
gfx::Range range_from_client;
// For aura this always returns true.
@@ -5498,7 +5498,7 @@ TEST_F(InputMethodStateAuraTest, GetSelectionRange) {
for (auto index : active_view_sequence_) {
render_widget_host_delegate()->set_focused_widget(
RenderWidgetHostImpl::From(views_[index]->GetRenderWidgetHost()));
- views_[index]->SelectionChanged(text, 0U, expected_range);
+ views_[index]->SelectionChanged(text, 0U, expected_range, true);
gfx::Range range_from_client;
// This method always returns true.
@@ -5528,7 +5528,7 @@ TEST_F(InputMethodStateAuraTest, SelectedTextCopiedToClipboard) {
// Change the selection of the currently focused widget. It suffices to just
// call the method on the view.
base::string16 expected_text = base::ASCIIToUTF16(texts[index]);
- views_[index]->SelectionChanged(expected_text, 0U, gfx::Range(0, 5));
+ views_[index]->SelectionChanged(expected_text, 0U, gfx::Range(0, 5), true);
// Retrieve the selected text from clipboard and verify it is as expected.
base::string16 result_text;

Powered by Google App Engine
This is Rietveld 408576698