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

Review URL: https://codereview.chromium.org/506803004

Cr-Commit-Position: refs/heads/master@{#299771}
8 files changed