Always inline SkFloatToFixed_arm.

Always inline SkFloatToFixed_arm.

We have "inline" assembly for speed on ARM, but the compiler when
told to optimize for space wasn't inlining it, destroying any
possible performance improvement.

BUG=skbug:2550
R=reed@google.com,mtklein@google.com

Committed: http://code.google.com/p/skia/source/detail?r=14745

[tomhudson, 2014-05-14 15:17:52]

Confirmed by inspection that using SK_ALWAYS_INLINE for SkFloatToFixed_arm in
Chromium gets rid of those bl opcodes inside SkDraw::drawPosText() and produces
relatively clean assembly code.

[reed1, 2014-05-14 15:18:28]

lgtm

[reed1, 2014-05-14 15:19:21]

hopefully we also have a micro-bench to confirm that SkFloatToFixed_arm is
indeed faster than just SkFloatToFixed, to justify the extra complexity.

[mtklein, 2014-05-14 15:20:17]

Was just thinking yesterday that we should take a survey of all the places we
write "inline".  Most of the time we don't mind things getting inlined when they
aren't tagged, but when we write inline I have a hunch we really may mean
SK_ALWAYS_INLINE.

lgtm

[reed1, 2014-05-14 15:35:16]

lgtm

[tomhudson, 2014-05-15 09:39:44]

The CQ bit was checked by tomhudson@google.com

[commit-bot: I haz the power, 2014-05-15 09:39:57]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/tomhudson@chromium.org/281143002/1

[commit-bot: I haz the power, 2014-05-15 09:46:22]

Change committed as 14745