Fixed i-cursor height calculation.

Fixed i-cursor height calculation.

In previous solution only AND part of cursor mask
was taken into account. Now both AND and XOR parts
are considered to properly determine bottom point
of cursor pointer.

BUG=446810
Committed: https://crrev.com/990e693cd5eece2e7a92007281b8e14ce2feec0a
Cr-Commit-Position: refs/heads/master@{#364026}

[Mateusz Szymański, 2015-04-08 07:12:33]

mateuszs@opera.com changed reviewers:
+ sky@chromium.org

[sky, 2015-04-09 15:23:01]

sky@chromium.org changed reviewers:
+ ananta@chromium.org

[sky, 2015-04-09 15:23:01]

+ananta

[Mateusz Szymański, 2015-04-16 07:36:24]

Any comments/remarks on this review?

[Mateusz Szymański, 2015-07-02 13:02:56]

~ping

Any news on this one?

[sky, 2015-07-20 17:29:25]

On 2015/07/02 13:02:56, Mateusz Szymański wrote:
> ~ping
> 
> Any news on this one?

Ananta, the ping is for you.

[Mateusz Szymański, 2015-12-07 08:10:06]

~ping.

Please, would someone kindly review this or can we just revert my changes back
to hardcoded "usually good" solution or something?
There are two issues hanging on this one that I keep getting notifications
about.

[ananta, 2015-12-08 00:47:46]

lgtm

[sky, 2015-12-08 00:58:48]

I'm assuming Ananta reviewed this, so rubber stamp LGTM

[Mateusz Szymański, 2015-12-08 08:19:19]

The CQ bit was checked by mateuszs@opera.com

[commit-bot: I haz the power, 2015-12-08 08:19:38]

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

[commit-bot: I haz the power, 2015-12-08 08:56:38]

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

[commit-bot: I haz the power, 2015-12-08 08:56:39]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_arm64_d...)

[Mateusz Szymański, 2015-12-08 09:04:42]

The CQ bit was checked by mateuszs@opera.com

[commit-bot: I haz the power, 2015-12-08 09:05:42]

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

[commit-bot: I haz the power, 2015-12-08 11:17:56]

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

[commit-bot: I haz the power, 2015-12-08 11:17:56]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Mateusz Szymański, 2015-12-08 15:30:35]

The CQ bit was checked by mateuszs@opera.com

[commit-bot: I haz the power, 2015-12-08 15:33:09]

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

[commit-bot: I haz the power, 2015-12-08 18:10:04]

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

[commit-bot: I haz the power, 2015-12-08 18:10:05]

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

[Mateusz Szymański, 2015-12-09 08:49:14]

The CQ bit was checked by mateuszs@opera.com

[commit-bot: I haz the power, 2015-12-09 08:49:52]

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

[commit-bot: I haz the power, 2015-12-09 09:48:25]

Description was changed from

==========
Fixed i-cursor height calculation.

In previous solution only AND part of cursor mask
was taken into account. Now both AND and XOR parts
are considered to properly determine bottom point
of cursor pointer.

BUG=446810
==========

to

==========
Fixed i-cursor height calculation.

In previous solution only AND part of cursor mask
was taken into account. Now both AND and XOR parts
are considered to properly determine bottom point
of cursor pointer.

BUG=446810
==========

[commit-bot: I haz the power, 2015-12-09 09:48:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-09 09:49:38]

Description was changed from

==========
Fixed i-cursor height calculation.

In previous solution only AND part of cursor mask
was taken into account. Now both AND and XOR parts
are considered to properly determine bottom point
of cursor pointer.

BUG=446810
==========

to

==========
Fixed i-cursor height calculation.

In previous solution only AND part of cursor mask
was taken into account. Now both AND and XOR parts
are considered to properly determine bottom point
of cursor pointer.

BUG=446810

Committed: https://crrev.com/990e693cd5eece2e7a92007281b8e14ce2feec0a
Cr-Commit-Position: refs/heads/master@{#364026}
==========

[commit-bot: I haz the power, 2015-12-09 09:49:38]

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