Make VisibleUnits::ComputeTextRect() return unified IntRect

Make VisibleUnits::ComputeTextRect() return unified IntRect

Only Range uses Vector<IntRect> but it unifies and return a IntRect.
Just unify in VisibleUnits::ComputeTextRect() directly.

TEST=No change in behavior
BUG=712986
Review-Url: https://codereview.chromium.org/2891273003
Cr-Commit-Position: refs/heads/master@{#473185}
Committed: https://chromium.googlesource.com/chromium/src/+/d77cd716740a94334fc7c98749876f4a013b8322

[yoichio, 2017-05-19 08:18:37]

Description was changed from

==========
Change ComputeTextRect

BUG=712986
==========

to

==========
Make VisibleUnits::ComputeTextRect() return unified IntRect

Only Range uses Vector<IntRect> but it unifies and return a IntRect.
Just unify in VisibleUnits::ComputeTextRect() directly.

TEST=No change in behavior
BUG=712986
==========

[yoichio, 2017-05-19 08:18:37]

yoichio@chromium.org changed reviewers:
+ yosin@chromium.org

[yoichio, 2017-05-19 08:18:50]

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

[commit-bot: I haz the power, 2017-05-19 08:19:19]

Dry run: CQ is trying da patch.

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

[yosin_UTC9, 2017-05-19 08:42:30]

yosin@chromium.org changed reviewers:
+ tkent@chromium.org

[yosin_UTC9, 2017-05-19 08:42:30]

lgtm

+tkent@

[tkent, 2017-05-19 08:48:49]

lgtm

[commit-bot: I haz the power, 2017-05-19 10:07:26]

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

[commit-bot: I haz the power, 2017-05-19 10:07:27]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[yosin_UTC9, 2017-05-19 10:23:37]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2017-05-19 10:23:57]

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, 2017-05-19 14:44:23]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495189417685890, "parent_rev":
"fbb5549824aa0609e490d709480d5ee049072a65", "commit_rev":
"d77cd716740a94334fc7c98749876f4a013b8322"}

[commit-bot: I haz the power, 2017-05-19 14:44:33]

Description was changed from

==========
Make VisibleUnits::ComputeTextRect() return unified IntRect

Only Range uses Vector<IntRect> but it unifies and return a IntRect.
Just unify in VisibleUnits::ComputeTextRect() directly.

TEST=No change in behavior
BUG=712986
==========

to

==========
Make VisibleUnits::ComputeTextRect() return unified IntRect

Only Range uses Vector<IntRect> but it unifies and return a IntRect.
Just unify in VisibleUnits::ComputeTextRect() directly.

TEST=No change in behavior
BUG=712986

Review-Url: https://codereview.chromium.org/2891273003
Cr-Commit-Position: refs/heads/master@{#473185}
Committed:
https://chromium.googlesource.com/chromium/src/+/d77cd716740a94334fc7c9874987...
==========

[commit-bot: I haz the power, 2017-05-19 14:44:34]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/d77cd716740a94334fc7c9874987...