Fix the bug that virtual keyboard window doesn't work.

Fix the bug that virtual keyboard window doesn't work.

BUG=329756
TEST=Verified in sandbox.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=242064

[Shu Chen, 2013-12-19 03:43:36]

Biao/Kevin, mind take a look?

Thanks,
Shu

[Shu Chen, 2013-12-19 04:00:42]

Both Biao and Kevin are in vacation.

Peter/James, can you please help to review this cl? This cl is to fix a P1 bug.

[James Cook, 2013-12-19 17:30:21]

LGTM but you might want to add a comment as to why the opacity has to be set
after calling the base class.

[Peter Kasting, 2013-12-19 22:36:15]

I'm not a good reviewer here, but I agree with James' comment that this really
deserves some comments, since it's not at all obvious why the order here should
matter.

[Shu Chen, 2013-12-20 08:09:31]

Thanks for your reviews. I actually don't know the root cause of why calling
Hide before SetOpacity causes mis-function of the keyboard window (when clicking
on the window).

But removing SetOpacity calls can obviously fix the bug.
I'm going to fix the P1 bug as it's blocking some other tasks.

And actually the code intension is to animate keyboard hiding which doesn't seem
to work too.
Soon later I will created another cl to address the keyboard hiding animation.

[commit-bot: I haz the power, 2013-12-20 08:10:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shuchen@chromium.org/103163009/10001

[commit-bot: I haz the power, 2013-12-20 12:06:01]

Change committed as 242064