Fix the order of the event dispatch.

Fix the order of the event dispatch.

UpdateTextInputType should be called before SyncSelectionIfRequired.
UpdateTextInputType may send TextInputTypeChanged to notify the focus
was changed, and SyncSelectionIfRequired may send SelectionChanged
to notify the selection was changed.  Focus change should be notified
before selection change.

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

[Seigo Nonaka, 2013-10-23 11:20:24]

lgtm, please wait aurimas's response :)

[aurimas (slooooooooow), 2013-10-24 06:27:42]

On 2013/10/23 11:20:24, Seigo Nonaka wrote:
> lgtm, please wait aurimas's response :)

lgtm if Android IME tests pass. Locally it seems to work.

[Hiro Komatsu, 2013-10-24 06:41:13]

Hi Jochen,

Would you review this patch.  I also received LGTMs from nona and aurimas who
are owners of text input for desktop and android.

Thank you,

[jochen (gone - plz use gerrit), 2013-10-24 12:35:47]

can you add a test that failed without this patch?

[Hiro Komatsu, 2013-11-01 12:58:52]

On 2013/10/24 12:35:47, jochen - OOO 1. - 3. 11 wrote:
> can you add a test that failed without this patch?

Added a test case.
Thanks,

[jochen (gone - plz use gerrit), 2013-11-04 09:09:04]

lgtm, thanks

[commit-bot: I haz the power, 2013-11-04 11:51:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/komatsu@chromium.org/33323003/150001

[commit-bot: I haz the power, 2013-11-04 14:51:50]

Retried try job too often on win7_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...

[commit-bot: I haz the power, 2013-11-05 01:35:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/komatsu@chromium.org/33323003/150001

[commit-bot: I haz the power, 2013-11-05 03:01:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/komatsu@chromium.org/33323003/150001

[commit-bot: I haz the power, 2013-11-05 06:54:49]

Retried try job too often on win_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-11-06 01:49:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/komatsu@chromium.org/33323003/150001

[commit-bot: I haz the power, 2013-11-06 05:41:00]

Retried try job too often on win_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-11-06 23:45:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/komatsu@chromium.org/33323003/150001

[commit-bot: I haz the power, 2013-11-07 01:00:49]

Change committed as 233449