Separate showing IME logic and setting properties 29/314029/7
authorhyoputer <hyoputer.kim@samsung.com>
Thu, 4 Jul 2024 04:40:57 +0000 (13:40 +0900)
committerBot Blink <blinkbot@samsung.com>
Tue, 16 Jul 2024 00:55:37 +0000 (00:55 +0000)
commit8dd5c75f7e736a4e7d195d88b156f947e99eb66d
tree7090bdeb7cad238578d2397547230be13941205d
parentad53c9c55a86ece6f7bdf17d7db927bdc263ee6f
Separate showing IME logic and setting properties

Separate the former one from |IMContextEfl::UpdateInputMthodState|
to the new function ShowIMEIfNeeded, and move the existing function
to use upstream functions and reduce the dependency of helpers.

The function had been called from |RenderWidgetHostViewAura
::TextInputStateChanged|, and this function would eventually call
|EflInputMethodContext::UpdateFocus|, where I move it to. It
would be always called before |ShowIMEIfNeeded| so the functionality
wouldn't be changed.

To make this, add the parameters of it in |LinuxInputMethodContext::
TextInputClientAttributes|, make the function counterparts of them in
|RenderWidgetHostViewAura| and |TextInputClient|. Three of the paramters
which are in |TextInputState| are returned like other ones in the class,
and |use_floating_ui| is returned by separated funtion in the helper.

Change-Id: I2c73c9e29ff773a22ad63250b3b440eef92683ad
Signed-off-by: hyoputer <hyoputer.kim@samsung.com>
content/browser/renderer_host/render_widget_host_view_aura.cc
content/browser/renderer_host/render_widget_host_view_aura.h
tizen_src/chromium_impl/content/browser/renderer_host/rwhv_aura_common_helper_efl.cc
tizen_src/chromium_impl/content/browser/renderer_host/rwhv_aura_common_helper_efl.h
tizen_src/chromium_impl/ui/ozone/platform/efl/efl_input_method_context.cc
tizen_src/chromium_impl/ui/ozone/platform/efl/im_context_efl.cc
tizen_src/chromium_impl/ui/ozone/platform/efl/im_context_efl.h
ui/base/ime/linux/input_method_auralinux.cc
ui/base/ime/linux/linux_input_method_context.h
ui/base/ime/text_input_client.cc
ui/base/ime/text_input_client.h