Show composition window for IME-unaware PPAPI plugins on Windows.

Show composition window for IME-unaware PPAPI plugins on Windows.

When a PPAPI plugin does not support drawing composition text by themselves, 
outer environment is expected to do it instead. This patch implements it on Windows.

BUG=97189
TEST=Manual test by a dummy plugin.
1. ninja -C out/Debug chrome ppapi_example_ime
2. out/Debug/chrome --register-pepper-plugins='/path/to/ppapi_example_ime.dll;application/x-ppapi-example-ime' file:///path/to/ppapi/example/ime/ime.html
3. type some text via IME on 2nd and 3rd plugin widget.
4. verify pre-edit composition text is displayed.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=158756

[kinaba, 2012-09-24 10:57:08]

+suzhe

Could you take a look at this patch?
I'm sorry for asking this time-leaping CL; this is a left-over work from the
last year, about supporting IME on PPAPI plugins. The counterpart for Linux
Chrome was in http://codereview.chromium.org/7041003, which you've kindly
reviewed.

Thanks in advance.

[James Su, 2012-09-24 15:35:54]

LGTM, assuming you have tested it.

On 2012/09/24 10:57:08, kinaba wrote:
> +suzhe
> 
> Could you take a look at this patch?
> I'm sorry for asking this time-leaping CL; this is a left-over work from the
> last year, about supporting IME on PPAPI plugins. The counterpart for Linux
> Chrome was in http://codereview.chromium.org/7041003, which you've kindly
> reviewed.
> 
> Thanks in advance.

[kinaba, 2012-09-25 00:55:02]

+sky, for OWNER approval for ui/ and chrome/.

Thanks, James.
I've manually verified it working on MS-{CCJK}-IMEs and several other
third-party IMEs including Google's, on XP and 7 (Win8 style UI using TSF is not
covered in this patch, I'm discussing with the guys working on TSF support).

[sky, 2012-09-25 15:46:18]

LGTM

http://codereview.chromium.org/10907170/diff/11001/content/browser/renderer_h...
File content/browser/renderer_host/render_widget_host_view_win.cc (right):

http://codereview.chromium.org/10907170/diff/11001/content/browser/renderer_h...
content/browser/renderer_host/render_widget_host_view_win.cc:767: ||
can_compose_inline_ != params.can_compose_inline) {
|| on previous line.

[kinaba, 2012-09-26 01:59:53]

http://codereview.chromium.org/10907170/diff/11001/content/browser/renderer_h...
File content/browser/renderer_host/render_widget_host_view_win.cc (right):

http://codereview.chromium.org/10907170/diff/11001/content/browser/renderer_h...
content/browser/renderer_host/render_widget_host_view_win.cc:767: ||
can_compose_inline_ != params.can_compose_inline) {
On 2012/09/25 15:46:18, sky wrote:
> || on previous line.

Done.

[commit-bot: I haz the power, 2012-09-26 02:00:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinaba@chromium.org/10907170/18002

[commit-bot: I haz the power, 2012-09-26 05:47:43]

Change committed as 158756