Add empty implementation of ThreadedInputConnection.deleteSurroundingTextInCodePoints().

Add empty implementation of ThreadedInputConnection.deleteSurroundingTextInCodePoints().

This is needed to allow building with the N preview SDK.

BUG=595525, 595522
Committed: https://crrev.com/75d7160d39b1928b8afe65323d9fed7fe7a7e379
Cr-Commit-Position: refs/heads/master@{#381652}

[newt (away), 2016-03-17 01:33:26]

newt@chromium.org changed reviewers:
+ changwan@chromium.org

[newt (away), 2016-03-17 01:33:27]

PTAL

This short-term fix is needed to use the N preview SDK (see linked bug).

Is this a reasonable approach? We'll need to actually implement this before N
launches.

[Changwan Ryu, 2016-03-17 01:40:19]

lgtm

actually this should not happen because there are apps who directly implement
InputConnection interface. but this is needed for us anyways. please change it
to return false for now.

https://codereview.chromium.org/1808073002/diff/1/content/public/android/java...
File
content/public/android/java/src/org/chromium/content/browser/input/ThreadedInputConnection.java
(right):

https://codereview.chromium.org/1808073002/diff/1/content/public/android/java...
content/public/android/java/src/org/chromium/content/browser/input/ThreadedInputConnection.java:405:
throw new UnsupportedOperationException("Not yet implemented");
could you return false instead? IME apps should be able to understand that it's
not being handled for now.

[newt (away), 2016-03-17 01:55:43]

Thanks for the quick review :)

https://codereview.chromium.org/1808073002/diff/1/content/public/android/java...
File
content/public/android/java/src/org/chromium/content/browser/input/ThreadedInputConnection.java
(right):

https://codereview.chromium.org/1808073002/diff/1/content/public/android/java...
content/public/android/java/src/org/chromium/content/browser/input/ThreadedInputConnection.java:405:
throw new UnsupportedOperationException("Not yet implemented");
On 2016/03/17 01:40:19, Changwan Ryu wrote:
> could you return false instead? IME apps should be able to understand that
it's
> not being handled for now.

Done.

[newt (away), 2016-03-17 01:58:30]

The CQ bit was checked by newt@chromium.org

[newt (away), 2016-03-17 01:58:31]

The patchset sent to the CQ was uploaded after l-g-t-m from
changwan@chromium.org
Link to the patchset: https://codereview.chromium.org/1808073002/#ps20001
(title: "returned false")

[commit-bot: I haz the power, 2016-03-17 01:58:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1808073002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1808073002/20001

[commit-bot: I haz the power, 2016-03-17 03:14:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-17 03:16:08]

Description was changed from

==========
Add empty implementation of
ThreadedInputConnection.deleteSurroundingTextInCodePoints().

This is needed to allow building with the N preview SDK.

BUG=595525,595522
==========

to

==========
Add empty implementation of
ThreadedInputConnection.deleteSurroundingTextInCodePoints().

This is needed to allow building with the N preview SDK.

BUG=595525,595522

Committed: https://crrev.com/75d7160d39b1928b8afe65323d9fed7fe7a7e379
Cr-Commit-Position: refs/heads/master@{#381652}
==========

[commit-bot: I haz the power, 2016-03-17 03:16:09]

Patchset 2 (id:??) landed as
https://crrev.com/75d7160d39b1928b8afe65323d9fed7fe7a7e379
Cr-Commit-Position: refs/heads/master@{#381652}