[Android] Enable composited selection updates

content/renderer/render_widget.cc

Index: content/renderer/render_widget.cc
diff --git a/content/renderer/render_widget.cc b/content/renderer/render_widget.cc
index 8cf0ae0735652ea30717f1baa56af0806975a309..5c42ba291e8f0a47b52ff0055948a196e1962d9a 100644
--- a/content/renderer/render_widget.cc
+++ b/content/renderer/render_widget.cc
@@ -1818,6 +1818,12 @@ void RenderWidget::UpdateSelectionBounds() {
   if (handling_ime_event_)
     return;
 
+  // With composited selection updates, the selection bounds will be reported
+  // directly by the compositor, in which case explicit IPC selection
+  // notifications should be suppressed.
+  // TODO(jdduke): Remove this when all platforms use composited updates for
+  // selection bounds, crbug.com/399721.
+#if !defined(OS_ANDROID)

[jochen (gone - plz use gerrit), 2014/09/19 13:25:11]

why not add WebRuntimeFeatures::isCompositedSelectionUpdatesEnabled() and skip
this code if it returns true?

[jdduke (slow), 2014/09/19 15:46:07]

What, and make the code both less brittle and easier to follow!?! ;) Done.

   ViewHostMsg_SelectionBounds_Params params;
   GetSelectionBounds(&params.anchor_rect, &params.focus_rect);
   if (selection_anchor_rect_ != params.anchor_rect ||
@@ -1828,6 +1834,8 @@ void RenderWidget::UpdateSelectionBounds() {
     params.is_anchor_first = webwidget_->isSelectionAnchorFirst();
     Send(new ViewHostMsg_SelectionBoundsChanged(routing_id_, params));
   }
+#endif
+
 #if defined(OS_MACOSX) || defined(USE_AURA)
   UpdateCompositionInfo(false);
 #endif