Track composition range and character bounds on the browser side (Mac)

Track composition range and character bounds on the browser side (Mac)

This CL will route updates in composition range through TextInputManager on Mac.
This is similar to how the tracking works in Aura. The corresponding test,
'TrackCompositionRangeForAllFrames' is also activated on Mac.

BUG=578168, 602723
Committed: https://crrev.com/d773ff47652910acdd309eca657aec6133d513b1
Cr-Commit-Position: refs/heads/master@{#411732}

[EhsanK, 2016-08-11 19:56:36]

The CQ bit was checked by ekaramad@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-11 19:57:29]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[EhsanK, 2016-08-12 03:18:25]

The CQ bit was checked by ekaramad@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-12 03:18:42]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-12 04:21:57]

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

[commit-bot: I haz the power, 2016-08-12 04:21:58]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[EhsanK, 2016-08-12 05:33:37]

The CQ bit was checked by ekaramad@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-12 05:33:47]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-12 06:15:51]

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

[commit-bot: I haz the power, 2016-08-12 06:15:52]

Dry run: This issue passed the CQ dry run.

[EhsanK, 2016-08-12 06:19:08]

ekaramad@chromium.org changed reviewers:
+ avi@chromium.org, creis@chromium.org, kenrb@chromium.org

[EhsanK, 2016-08-12 06:19:08]

PTAL. This should be that last CL for MAC-OOPIF-IME.

[Avi (use Gerrit), 2016-08-12 15:56:04]

lgtm

[Charlie Reis, 2016-08-12 19:14:13]

Great, LGTM too.

[Charlie Reis, 2016-08-12 19:14:27]

Description was changed from

==========
Track composition range and character bounds on the browser side (Mac)

This CL will route updates in composition range through TextInputManager on Mac.
This is similar to how the tracking works in Aura. The corresponding test,
'TrackCompositionRangeForAllFrames' is also activated on Mac.

BUG=578168, 602723
==========

to

==========
Track composition range and character bounds on the browser side (Mac)

This CL will route updates in composition range through TextInputManager on Mac.
This is similar to how the tracking works in Aura. The corresponding test,
'TrackCompositionRangeForAllFrames' is also activated on Mac.

BUG=578168, 602723
==========

[kenrb, 2016-08-12 19:14:56]

lgtm

[EhsanK, 2016-08-12 19:25:20]

Thank you for the reviews. I will CQ after dry-run.

[EhsanK, 2016-08-12 19:25:28]

The CQ bit was checked by ekaramad@chromium.org to run a CQ dry run

[EhsanK, 2016-08-12 19:25:35]

The CQ bit was unchecked by ekaramad@chromium.org

[EhsanK, 2016-08-12 19:25:38]

The CQ bit was checked by ekaramad@chromium.org

[commit-bot: I haz the power, 2016-08-12 19:26:13]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-12 19:30:58]

Description was changed from

==========
Track composition range and character bounds on the browser side (Mac)

This CL will route updates in composition range through TextInputManager on Mac.
This is similar to how the tracking works in Aura. The corresponding test,
'TrackCompositionRangeForAllFrames' is also activated on Mac.

BUG=578168, 602723
==========

to

==========
Track composition range and character bounds on the browser side (Mac)

This CL will route updates in composition range through TextInputManager on Mac.
This is similar to how the tracking works in Aura. The corresponding test,
'TrackCompositionRangeForAllFrames' is also activated on Mac.

BUG=578168, 602723
==========

[commit-bot: I haz the power, 2016-08-12 19:31:00]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-12 19:33:04]

Description was changed from

==========
Track composition range and character bounds on the browser side (Mac)

This CL will route updates in composition range through TextInputManager on Mac.
This is similar to how the tracking works in Aura. The corresponding test,
'TrackCompositionRangeForAllFrames' is also activated on Mac.

BUG=578168, 602723
==========

to

==========
Track composition range and character bounds on the browser side (Mac)

This CL will route updates in composition range through TextInputManager on Mac.
This is similar to how the tracking works in Aura. The corresponding test,
'TrackCompositionRangeForAllFrames' is also activated on Mac.

BUG=578168, 602723

Committed: https://crrev.com/d773ff47652910acdd309eca657aec6133d513b1
Cr-Commit-Position: refs/heads/master@{#411732}
==========

[commit-bot: I haz the power, 2016-08-12 19:33:05]

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