Support CancelCompositionText/ComfirmCompositionText in TSF

Support CancelCompositionText/ComfirmCompositionText in TSF.

Currently CancelCompositionText is not fully implemented in
InputMethodTSF. This CL fixes the issue.

Note:
CUAS is also implementing the protocol conversions from
  ImmNotifyIME(NI_COMPOSITIONSTR, CPS_CANCEL, 0)
and
  ImmNotifyIME(NI_COMPOSITIONSTR, CPS_COMPLETE, 0)
into TSF operations, respectively. To minimize the risk of
compatibility issues, this CL uses the the same protocol
conversions logic to CUAS.

BUG=246534
TEST=Tested with GoogleJapaneseInput

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

[Seigo Nonaka, 2013-06-21 08:09:53]

https://codereview.chromium.org/16901011/diff/3001/ui/base/ime/win/tsf_bridge.cc
File ui/base/ime/win/tsf_bridge.cc (right):

https://codereview.chromium.org/16901011/diff/3001/ui/base/ime/win/tsf_bridge...
ui/base/ime/win/tsf_bridge.cc:239: 
nit: please remove

https://codereview.chromium.org/16901011/diff/3001/ui/base/ime/win/tsf_text_s...
File ui/base/ime/win/tsf_text_store.cc (right):

https://codereview.chromium.org/16901011/diff/3001/ui/base/ime/win/tsf_text_s...
ui/base/ime/win/tsf_text_store.cc:836: if (edit_flag_ != 0)
!edit_flag?

https://codereview.chromium.org/16901011/diff/3001/ui/base/ime/win/tsf_text_s...
ui/base/ime/win/tsf_text_store.cc:863: if (edit_flag_ != 0)
ditto

https://codereview.chromium.org/16901011/diff/3001/ui/base/ime/win/tsf_text_s...
File ui/base/ime/win/tsf_text_store.h (right):

https://codereview.chromium.org/16901011/diff/3001/ui/base/ime/win/tsf_text_s...
ui/base/ime/win/tsf_text_store.h:205: // Cancels the ongoing composition if
exists.
nit: please describe the returned value. true on succeeded?

[Yohei Yukawa, 2013-06-21 08:26:54]

PTAL?

https://codereview.chromium.org/16901011/diff/3001/ui/base/ime/win/tsf_bridge.cc
File ui/base/ime/win/tsf_bridge.cc (right):

https://codereview.chromium.org/16901011/diff/3001/ui/base/ime/win/tsf_bridge...
ui/base/ime/win/tsf_bridge.cc:239: 
On 2013/06/21 08:09:53, Seigo Nonaka wrote:
> nit: please remove

Done.

https://codereview.chromium.org/16901011/diff/3001/ui/base/ime/win/tsf_text_s...
File ui/base/ime/win/tsf_text_store.cc (right):

https://codereview.chromium.org/16901011/diff/3001/ui/base/ime/win/tsf_text_s...
ui/base/ime/win/tsf_text_store.cc:836: if (edit_flag_ != 0)
On 2013/06/21 08:09:53, Seigo Nonaka wrote:
> !edit_flag?

Done.

https://codereview.chromium.org/16901011/diff/3001/ui/base/ime/win/tsf_text_s...
ui/base/ime/win/tsf_text_store.cc:863: if (edit_flag_ != 0)
On 2013/06/21 08:09:53, Seigo Nonaka wrote:
> ditto

Done.

https://codereview.chromium.org/16901011/diff/3001/ui/base/ime/win/tsf_text_s...
File ui/base/ime/win/tsf_text_store.h (right):

https://codereview.chromium.org/16901011/diff/3001/ui/base/ime/win/tsf_text_s...
ui/base/ime/win/tsf_text_store.h:205: // Cancels the ongoing composition if
exists.
On 2013/06/21 08:09:53, Seigo Nonaka wrote:
> nit: please describe the returned value. true on succeeded?

Done.

[Seigo Nonaka, 2013-06-21 08:40:57]

lgtm

[commit-bot: I haz the power, 2013-06-22 14:33:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/Yukawa@chromium.org/16901011/16001

[commit-bot: I haz the power, 2013-06-22 15:18:48]

Sorry for I got bad news for ya.
Compile failed with a clobber build on win7_aura.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-06-23 02:25:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/Yukawa@chromium.org/16901011/22003

[commit-bot: I haz the power, 2013-06-23 07:26:38]

The commit queue went berserk retrying too often for a
seemingly flaky test on builder android_dbg:
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...

[commit-bot: I haz the power, 2013-06-23 11:33:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/Yukawa@chromium.org/16901011/22003

[commit-bot: I haz the power, 2013-06-23 14:10:47]

Change committed as 208124