Revert "Plumb composition character bounds from C++ to Java"

Revert "Plumb composition character bounds from C++ to Java"

This reverts commit 6d6a279a1cf7d9e8b5072740a494c4031eaa133d.

Turned out that this CL is not necessary.

BUG=424866
TBR=aelias@chromium.org

Committed: https://crrev.com/cf1b16837d5482473d390a23e664ceaeba78421a
Cr-Commit-Position: refs/heads/master@{#308038}

[yukawa, 2014-12-05 07:35:34]

yukawa@chromium.org changed reviewers:
+ aelias@chromium.org, aurimas@chromium.org, jdduke@chromium.org

[yukawa, 2014-12-05 07:35:34]

Hi Alexandre, Aurimas. Can you take a look?

Reverting this CL also makes it easy to revert r301384 as Jared suggested in
https://codereview.chromium.org/757233003/.

[jdduke (slow), 2014-12-05 16:07:52]

On 2014/12/05 07:35:34, yukawa wrote:
> Hi Alexandre, Aurimas. Can you take a look?
> 
> Reverting this CL also makes it easy to revert r301384 as Jared suggested in
> https://codereview.chromium.org/757233003/.

So we'll never need this code? Or you'll add it back later? If you have definite
plans to add it back, you might consider leaving it, and just reverting r301384
as that is what turns the plumbing on.

[yukawa, 2014-12-05 23:18:46]

On 2014/12/05 16:07:52, jdduke wrote:
> On 2014/12/05 07:35:34, yukawa wrote:
> > Hi Alexandre, Aurimas. Can you take a look?
> > 
> > Reverting this CL also makes it easy to revert r301384 as Jared suggested in
> > https://codereview.chromium.org/757233003/.
> 
> So we'll never need this code? Or you'll add it back later? If you have
definite
> plans to add it back, you might consider leaving it, and just reverting
r301384
> as that is what turns the plumbing on.

Probably we don't need this, but it still depends on the discussion in
https://codereview.chromium.org/699333003/.
In the latest plan, I'm implementing that functionality mostly in native side,
hence we no longer need to send character bounds to Java side.  If the latest
plan is OK, we'll never need this code.

[jdduke (slow), 2014-12-05 23:20:08]

On 2014/12/05 23:18:46, yukawa wrote:
> On 2014/12/05 16:07:52, jdduke wrote:
> > On 2014/12/05 07:35:34, yukawa wrote:
> > > Hi Alexandre, Aurimas. Can you take a look?
> > > 
> > > Reverting this CL also makes it easy to revert r301384 as Jared suggested
in
> > > https://codereview.chromium.org/757233003/.
> > 
> > So we'll never need this code? Or you'll add it back later? If you have
> definite
> > plans to add it back, you might consider leaving it, and just reverting
> r301384
> > as that is what turns the plumbing on.
> 
> Probably we don't need this, but it still depends on the discussion in
> https://codereview.chromium.org/699333003/.
> In the latest plan, I'm implementing that functionality mostly in native side,
> hence we no longer need to send character bounds to Java side.  If the latest
> plan is OK, we'll never need this code.

ok then lgtm

[jdduke (slow), 2014-12-12 01:02:58]

As a straightforward revert that has no user-facing effect, feel free to TBR
owners if you need them.

[jdduke (slow), 2014-12-12 01:05:31]

The CQ bit was checked by jdduke@chromium.org

[commit-bot: I haz the power, 2014-12-12 01:06:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/784503002/1

[commit-bot: I haz the power, 2014-12-12 02:52:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-12-12 02:54:17]

Patchset 1 (id:??) landed as
https://crrev.com/cf1b16837d5482473d390a23e664ceaeba78421a
Cr-Commit-Position: refs/heads/master@{#308038}