Refactor LayoutText::AbsoluteQuadsForRange()

Refactor LayoutText::AbsoluteQuadsForRange()

Following code flow doesn't make sense:
bool has_checked_box_in_range = !quads.IsEmpty();
if (!has_checked_box_in_range) {
  has_checked_box_in_range = true;	
  quads.clear();	
}

Because if |quads| is empty, we don't need to clear it.

BUG=

[yoichio, 2017-05-24 05:03:28]

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

[commit-bot: I haz the power, 2017-05-24 05:04:14]

Dry run: CQ is trying da patch.

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

[yoichio, 2017-05-24 05:50:08]

Description was changed from

==========
Clear text::abs variables

BUG=
==========

to

==========
Refactor LayoutText::AbsoluteQuadsForRange()

Following code flow doesn't make sense:
bool has_checked_box_in_range = !quads.IsEmpty();
if (!has_checked_box_in_range) {
  has_checked_box_in_range = true;	
  quads.clear();	
}

Because if |quads| is empty, we don't need to clear it.

BUG=
==========

[yoichio, 2017-05-24 05:50:08]

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

[commit-bot: I haz the power, 2017-05-24 07:05:32]

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

[commit-bot: I haz the power, 2017-05-24 07:05:33]

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_...)