Rename SkBitmapProcStateAutoMapper methods

Rename SkBitmapProcStateAutoMapper methods

x(),y() -> fractionalIntX(), fractionalIntY()

(to clarify the return type)

Also add fixed & int helpers.

R=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1666433003
CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot

Committed: https://skia.googlesource.com/skia/+/be5cfa9a9e1cfb77beba27727df7149ab5e7d422

[f(malita), 2016-02-03 16:57:38]

Description was changed from

==========
Rename SkBitmapProcStateAutoMapper methods

x(),y() -> fractionalIntX(), fractionalIntY()

(to clarify the return type)

R=reed@google.com
==========

to

==========
Rename SkBitmapProcStateAutoMapper methods

x(),y() -> fractionalIntX(), fractionalIntY()

(to clarify the return type)

R=reed@google.com
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot
==========

[f(malita), 2016-02-03 16:59:03]

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

[commit-bot: I haz the power, 2016-02-03 16:59:11]

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

[reed1, 2016-02-03 17:18:21]

Thanks for experimenting. Probably slightly better, but no biggie I think either
way.

Another "possible" clarification could be to embedded the various conversions in
the getters, so the callers can be slightly more succinct:

mapper.fractionalIntX()
mapper.fixedX()    or asFixedX()
mapper.intX()

Just a crazy thought. I'm fine to land, ignore, or revise, at your judgement.

[f(malita), 2016-02-03 17:23:49]

On 2016/02/03 17:18:21, reed1 wrote:
> Another "possible" clarification could be to embedded the various conversions
in
> the getters, so the callers can be slightly more succinct:
> 
> mapper.fractionalIntX()
> mapper.fixedX()    or asFixedX()
> mapper.intX()

Good idea, let me see what that looks like.

[f(malita), 2016-02-03 17:41:15]

Description was changed from

==========
Rename SkBitmapProcStateAutoMapper methods

x(),y() -> fractionalIntX(), fractionalIntY()

(to clarify the return type)

R=reed@google.com
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot
==========

to

==========
Rename SkBitmapProcStateAutoMapper methods

x(),y() -> fractionalIntX(), fractionalIntY()

(to clarify the return type)

Also add fixed & int helpers.

R=reed@google.com
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot
==========

[f(malita), 2016-02-03 17:45:08]

On 2016/02/03 17:23:49, f(malita) wrote:
> On 2016/02/03 17:18:21, reed1 wrote:
> > Another "possible" clarification could be to embedded the various
conversions
> in
> > the getters, so the callers can be slightly more succinct:
> > 
> > mapper.fractionalIntX()
> > mapper.fixedX()    or asFixedX()
> > mapper.intX()
> 
> Good idea, let me see what that looks like.

It certainly helps declutter the clients, I like it.  PTAL.

[f(malita), 2016-02-03 17:45:13]

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

[commit-bot: I haz the power, 2016-02-03 17:45:19]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1666433003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1666433003/20001

[reed1, 2016-02-03 17:48:15]

lgtm

[commit-bot: I haz the power, 2016-02-03 17:48:33]

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

[commit-bot: I haz the power, 2016-02-03 17:48:33]

Dry run: Try jobs failed on following builders:
  Build-Ubuntu-GCC-Arm64-Debug-Android-Trybot on client.skia.compile
(JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-Arm...)

[f(malita), 2016-02-03 17:50:02]

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

[commit-bot: I haz the power, 2016-02-03 17:50:10]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1666433003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1666433003/40001

[f(malita), 2016-02-03 18:15:01]

The CQ bit was unchecked by fmalita@chromium.org

[f(malita), 2016-02-03 18:15:07]

The CQ bit was checked by fmalita@chromium.org

[f(malita), 2016-02-03 18:15:08]

The patchset sent to the CQ was uploaded after l-g-t-m from reed@google.com
Link to the patchset: https://codereview.chromium.org/1666433003/#ps40001
(title: "neon build fix")

[commit-bot: I haz the power, 2016-02-03 18:15:09]

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

[commit-bot: I haz the power, 2016-02-03 18:21:36]

Description was changed from

==========
Rename SkBitmapProcStateAutoMapper methods

x(),y() -> fractionalIntX(), fractionalIntY()

(to clarify the return type)

Also add fixed & int helpers.

R=reed@google.com
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot
==========

to

==========
Rename SkBitmapProcStateAutoMapper methods

x(),y() -> fractionalIntX(), fractionalIntY()

(to clarify the return type)

Also add fixed & int helpers.

R=reed@google.com
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot

Committed:
https://skia.googlesource.com/skia/+/be5cfa9a9e1cfb77beba27727df7149ab5e7d422
==========

[commit-bot: I haz the power, 2016-02-03 18:21:37]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/be5cfa9a9e1cfb77beba27727df7149ab5e7d422