[WIP] Add support for Android SuggestionSpans when editing text

content/common/input_messages.h

Index: content/common/input_messages.h
diff --git a/content/common/input_messages.h b/content/common/input_messages.h
index 7221f95d3d31c18cd2cfd6877c938d8ccab62dbe..19d4bf7b2b36beee1a8b6bb973153bd61e94698c 100644
--- a/content/common/input_messages.h
+++ b/content/common/input_messages.h
@@ -28,6 +28,7 @@
 #include "content/common/input/touch_action.h"
 #include "ipc/ipc_message_macros.h"
 #include "third_party/WebKit/public/platform/WebInputEvent.h"
+#include "third_party/WebKit/public/web/WebTextSuggestionInfo.h"
 #include "ui/events/blink/did_overscroll_params.h"
 #include "ui/events/ipc/latency_info_param_traits.h"
 #include "ui/gfx/geometry/point.h"
@@ -135,6 +136,18 @@ IPC_STRUCT_TRAITS_BEGIN(content::InputEventAck)
   IPC_STRUCT_TRAITS_MEMBER(unique_touch_event_id)
 IPC_STRUCT_TRAITS_END()
 
+IPC_STRUCT_TRAITS_BEGIN(blink::WebTextSuggestionInfo)
+  IPC_STRUCT_TRAITS_MEMBER(documentMarkerID)
+  IPC_STRUCT_TRAITS_MEMBER(suggestionStart)
+  IPC_STRUCT_TRAITS_MEMBER(suggestionEnd)
+  IPC_STRUCT_TRAITS_MEMBER(spanStart)
+  IPC_STRUCT_TRAITS_MEMBER(spanEnd)
+  IPC_STRUCT_TRAITS_MEMBER(suggestionIndex)
+  IPC_STRUCT_TRAITS_MEMBER(prefix)
+  IPC_STRUCT_TRAITS_MEMBER(suggestion)
+  IPC_STRUCT_TRAITS_MEMBER(suffix)
+IPC_STRUCT_TRAITS_END()
+
 // Sends an input event to the render widget. The input event in general
 // contains a list of coalesced events and one event that is representative of
 // all those events (https://w3c.github.io/pointerevents/extension.html).
@@ -183,6 +196,14 @@ IPC_MESSAGE_ROUTED5(
     int, /* selectiont_start */
     int /* selection_end */)
 
+IPC_MESSAGE_ROUTED2(InputMsg_ImeApplySuggestionReplacement,

[aelias_OOO_until_Jul13, 2017/01/25 03:34:26]

These do not originate from the IME per se so I'd prefer "Ime" not be in the
name.

+                    int /* document_marker_ID */,
+                    int /* suggestion_index */)
+
+IPC_MESSAGE_ROUTED0(InputMsg_ImeDeleteSuggestionHighlight)

[aelias_OOO_until_Jul13, 2017/01/25 03:34:26]

Can this take a marker ID as well instead of inferring it from current
selection?  Or, if the ID doesn't provide any information on top of what Blink
has to keep state on anyway, I guess neither message should have it.

[rlanday, 2017/01/25 18:48:10]

This message doesn't take an ID because:
  - At any point there's only one span of text highlighted in this manner
  - The highlighted span is actually a union of SuggestionSpan ranges rather
than necessarily corresponding to any individual range

The messages to replace an individual SuggestionSpan do need the ID because
neither of these two reasons applies.

+
+IPC_MESSAGE_ROUTED0(InputMsg_ImeCloseSuggestionMenu)

[aelias_OOO_until_Jul13, 2017/01/25 03:34:26]

This is more of a notification for something that already happened, so I'd
prefer "InputMsg_SuggestionMenuClosed"

+
 // This message deletes the current composition, inserts specified text, and
 // moves the cursor.
 IPC_MESSAGE_ROUTED4(
@@ -327,6 +348,9 @@ IPC_MESSAGE_ROUTED2(InputHostMsg_ImeCompositionRangeChanged,
                     gfx::Range /* composition range */,
                     std::vector<gfx::Rect> /* character bounds */)
 
+IPC_MESSAGE_ROUTED1(InputHostMsg_ShowTextSuggestionMenu,
+                    std::vector<blink::WebTextSuggestionInfo>)
+
 // Adding a new message? Stick to the sort order above: first platform
 // independent InputMsg, then ifdefs for platform specific InputMsg, then
 // platform independent InputHostMsg, then ifdefs for platform specific