Preload IME for the flash sandbox plugin...

Preload IME for the flash sandbox plugin

Before we lock down (call LowerToken()) we force the IME subsystem to load, so the
required resources are acquired.

Typical sites that show the issue are weathernews.jp and www.nicovideo.jp

BUG=66605
TEST=see bug



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

[cpu_(ooo_6.6-7.5), 2011-02-21 22:02:46]

.

[Hironori Bono, 2011-02-22 01:39:47]

LGTM.
I'm a little wondering if we may need to call ImmDestroyContext() to delete the
context created by the ImmCreateContext(), though.

Regards,

Hironori Bono

[cpu_(ooo_6.6-7.5), 2011-02-22 02:31:15]

Ideally we would, but I am concerned that under some conditions the resources
needed for IME get closed and we end up in the same situation.


On 2011/02/22 01:39:47, hbono wrote:
> LGTM.
> I'm a little wondering if we may need to call ImmDestroyContext() to delete
the
> context created by the ImmCreateContext(), though.
> 
> Regards,
> 
> Hironori Bono

[Hironori Bono, 2011-02-22 02:47:44]

Thank you so much for your description. It makes sense. :)

Regards,

Hironori Bono

On 2011/02/22 02:31:15, cpu wrote:
> Ideally we would, but I am concerned that under some conditions the resources
> needed for IME get closed and we end up in the same situation.
> 
> 
> On 2011/02/22 01:39:47, hbono wrote:
> > LGTM.
> > I'm a little wondering if we may need to call ImmDestroyContext() to delete
> the
> > context created by the ImmCreateContext(), though.
> > 
> > Regards,
> > 
> > Hironori Bono