gfx: Don't template gfx::Rect and gfx::RectF

gfx: Don't template gfx::Rect and gfx::RectF

The extern template class declaration prevents the compiler
from ever inlining methods even when they are in the .h file
and it makes sense to do so.

Stop templating the classes to remove the extern template class
and allow the compiler to make better decisions about inlining.

Performance numbers are at https://crbug.com/407444#c13 This
drops up to 200us off our total thread times per frame.

It also decreases the size of the android binary as shown in
the https://crbug.com/407444#c12 by 3613 bytes before even
removing the RectBase class from the build.

This also makes finding and reading the public API for Rect and
RectF classes much easier for developers.

BUG=407444
Committed: https://crrev.com/a5cc021b6ee19af6a931db1042ec2ab562506010
Cr-Commit-Position: refs/heads/master@{#299771}

[danakj, 2014-08-27 22:01:17]

Patchset #1 (id:1) has been deleted

[danakj, 2014-10-15 19:42:22]

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

[sky, 2014-10-15 20:19:53]

LGTM

[danakj, 2014-10-15 20:45:25]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2014-10-15 20:46:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/506803004/60001

[danakj, 2014-10-15 20:46:58]

The CQ bit was unchecked by danakj@chromium.org

[danakj, 2014-10-15 20:47:08]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2014-10-15 20:47:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/506803004/60001

[commit-bot: I haz the power, 2014-10-15 21:28:02]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2014-10-15 21:28:50]

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