Fix focus ring damage sometimes being 1 pixel off.

Fix focus ring damage sometimes being 1 pixel off.

crrev.com/2386033002 changed the focus ring stroke from being 1 pixel to
being scaled based on the zoom level, which changed the type to a float.
When calculating the outdent (and thus damage) the width value lost
precsion and so at certain fractional zoom levels would leave a 1 pixel
line behind. This patch ceils the width value instead, to make sure that
pixel is redrawn when necessary.

R=bokan@chromium.org
BUG=658648
Committed: https://crrev.com/b18b97a7da62e85fc08eddaefbc95b49c4d7b648
Cr-Commit-Position: refs/heads/master@{#429153}

[bokan, 2016-10-27 21:40:42]

change looks fine, could we get a test?

[Bret, 2016-10-28 01:01:11]

On 2016/10/27 21:40:42, bokan wrote:
> change looks fine, could we get a test?

Added a test, PTAL

[bokan, 2016-10-28 12:59:25]

thank you, lgtm

[Bret, 2016-10-28 17:57:54]

The CQ bit was checked by bsep@chromium.org

[commit-bot: I haz the power, 2016-10-28 17:58: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-10-28 18:35:45]

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

[commit-bot: I haz the power, 2016-10-28 18:35:46]

Try jobs failed on following builders:
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)

[Bret, 2016-11-01 20:59:55]

The CQ bit was checked by bsep@chromium.org

[Bret, 2016-11-01 20:59:56]

The patchset sent to the CQ was uploaded after l-g-t-m from bokan@chromium.org
Link to the patchset: https://codereview.chromium.org/2460683002/#ps60001
(title: "fix linux component/debug link error")

[commit-bot: I haz the power, 2016-11-01 21:00:41]

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-11-01 23:33:49]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-11-01 23:35:49]

Description was changed from

==========
Fix focus ring damage sometimes being 1 pixel off.

crrev.com/2386033002 changed the focus ring stroke from being 1 pixel to
being scaled based on the zoom level, which changed the type to a float.
When calculating the outdent (and thus damage) the width value lost
precsion and so at certain fractional zoom levels would leave a 1 pixel
line behind. This patch ceils the width value instead, to make sure that
pixel is redrawn when necessary.

R=bokan@chromium.org
BUG=658648
==========

to

==========
Fix focus ring damage sometimes being 1 pixel off.

crrev.com/2386033002 changed the focus ring stroke from being 1 pixel to
being scaled based on the zoom level, which changed the type to a float.
When calculating the outdent (and thus damage) the width value lost
precsion and so at certain fractional zoom levels would leave a 1 pixel
line behind. This patch ceils the width value instead, to make sure that
pixel is redrawn when necessary.

R=bokan@chromium.org
BUG=658648

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

[commit-bot: I haz the power, 2016-11-01 23:35:50]

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