Caret is showing after do a Cut operation from editable field.

Hide the insertion handle when performing a Cut

Make behavior uniform with other apps on Android,
where after a Cut from an editable field, the 
insertion handle is hidden and only the cursor
remains visible.
https://codereview.chromium.org/394953005/

BUG:394781

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

[Cyan, 2014-07-17 10:20:17]

@jdduke
PTAL

[jdduke (slow), 2014-07-17 14:53:21]

I think aurimas@ may have added this code original (though that was probably
just from upstreaming?), so I'll defer review to him. In any case, this looks
good and may have simply been an oversight to make the call in the original
implementation?

Could you also update the title/description according to the comments I left at
https://codereview.chromium.org/394953005/? Note that when you go to "Edit
Issue", the first line in the "Description" field will be used as the submitted
patch title, and the remaining text is used in the patch description. Thanks.

[Cyan, 2014-07-17 15:16:24]

On 2014/07/17 14:53:21, jdduke wrote:
> I think aurimas@ may have added this code original (though that was probably
> just from upstreaming?), so I'll defer review to him. In any case, this looks
> good and may have simply been an oversight to make the call in the original
> implementation?
> 
> Could you also update the title/description according to the comments I left
at
> https://codereview.chromium.org/394953005/ Note that when you go to "Edit
> Issue", the first line in the "Description" field will be used as the
submitted
> patch title, and the remaining text is used in the patch description. Thanks.

@aurimas: PTAL

[aurimas (slooooooooow), 2014-07-17 16:14:58]

lgtm

[Cyan, 2014-07-18 05:11:50]

The CQ bit was checked by sayan.nayak@samsung.com

[commit-bot: I haz the power, 2014-07-18 05:13:24]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sayan.nayak@samsung.com/399613008/1

[commit-bot: I haz the power, 2014-07-18 07:36:01]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[commit-bot: I haz the power, 2014-07-18 07:39:15]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-18 07:39:15]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[Cyan, 2014-07-18 09:16:00]

On 2014/07/18 07:39:15, I haz the power (commit-bot) wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on tryserver.chromium
>
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

@jdduke,aurimas:
I'm getting this error during commit.
** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
   
content/public/android/java/src/org/chromium/content/browser/SelectActionModeCallback.java

[jdduke (slow), 2014-07-18 12:47:00]

On 2014/07/18 09:16:00, sayan.nayak wrote:
> On 2014/07/18 07:39:15, I haz the power (commit-bot) wrote:
> > Try jobs failed on following builders:
> >   chromium_presubmit on tryserver.chromium
> >
>
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)
> 
> @jdduke,aurimas:
> I'm getting this error during commit.
> ** Presubmit ERRORS **
> Missing LGTM from an OWNER for these files:
>    
>
content/public/android/java/src/org/chromium/content/browser/SelectActionModeCallback.java

lgtm

[Cyan, 2014-07-18 12:48:29]

The CQ bit was checked by sayan.nayak@samsung.com

[commit-bot: I haz the power, 2014-07-18 12:49:07]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sayan.nayak@samsung.com/399613008/1

[commit-bot: I haz the power, 2014-07-18 12:51:58]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[commit-bot: I haz the power, 2014-07-18 12:57:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-18 12:57:56]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[Cyan, 2014-07-18 13:18:06]

@jdduke
added my name in AUTHORS file and uploaded new patchset.
PTAL

[jdduke (slow), 2014-07-18 14:55:29]

On 2014/07/18 13:18:06, sayan.nayak wrote:
> @jdduke
> added my name in AUTHORS file and uploaded new patchset.
> PTAL

lgtm, thanks.

[Cyan, 2014-07-18 15:02:24]

The CQ bit was checked by sayan.nayak@samsung.com

[commit-bot: I haz the power, 2014-07-18 15:03:14]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sayan.nayak@samsung.com/399613008/20001

[commit-bot: I haz the power, 2014-07-18 17:33:34]

Change committed as 284138