Migrate ui::Transform to gfx::Transform

Migrate ui::Transform to gfx::Transform

It is more appropriate for Transform class to belong with its fellow
geometry classes in the gfx namespace.

[shawnsingh, 2012-10-11 22:23:19]

I just simply changed the namespace used by Transform in ui/gfx/transform.* and
ui/gfx/transform_util.*  ... all other changes are just the fallout from the
namespace change.

[Ian Vollick, 2012-10-11 22:27:44]

On 2012/10/11 22:23:19, shawnsingh wrote:
> I just simply changed the namespace used by Transform in ui/gfx/transform.*
and
> ui/gfx/transform_util.*  ... all other changes are just the fallout from the
> namespace change.

LGTM. Thanks for doing this.

[tfarina, 2012-10-11 22:35:03]

http://codereview.chromium.org/11087093/diff/1/ui/gfx/transform_util.h
File ui/gfx/transform_util.h (right):

http://codereview.chromium.org/11087093/diff/1/ui/gfx/transform_util.h#newcode16
ui/gfx/transform_util.h:16: UI_EXPORT Transform GetScaleTransform(const
gfx::Point& anchor, float scale);
nit: no need of gfx:: anymore here so ;)

[tfarina, 2012-10-11 22:38:21]

http://codereview.chromium.org/11087093/diff/1/ui/gfx/transform.h
File ui/gfx/transform.h (right):

http://codereview.chromium.org/11087093/diff/1/ui/gfx/transform.h#newcode126
ui/gfx/transform.h:126: }// namespace gfx
nit: add two spaces between } and //

[sky, 2012-10-11 23:54:25]

LGTM - make sure history isn't lost.

[tfarina, 2012-10-11 23:57:32]

On Thu, Oct 11, 2012 at 8:54 PM,  <sky@chromium.org> wrote:
> LGTM - make sure history isn't lost.
>
i.e, either land through svn or cq. But not with git :/

-- 
Thiago

[shawnsingh, 2012-10-11 23:59:26]

On 2012/10/11 23:57:32, tfarina wrote:
> On Thu, Oct 11, 2012 at 8:54 PM,  <mailto:sky@chromium.org> wrote:
> > LGTM - make sure history isn't lost.
> >
> i.e, either land through svn or cq. But not with git :/
> 
> -- 
> Thiago

Just to be completely clear - using git locally, but using the cq on this will
be OK right?

[tfarina, 2012-10-12 00:02:40]

On 2012/10/11 23:59:26, shawnsingh wrote:
> On 2012/10/11 23:57:32, tfarina wrote:
> > On Thu, Oct 11, 2012 at 8:54 PM,  <mailto:sky@chromium.org> wrote:
> > > LGTM - make sure history isn't lost.
> > >
> > i.e, either land through svn or cq. But not with git :/
> > 
> > -- 
> > Thiago
> 
> Just to be completely clear - using git locally, but using the cq on this will
> be OK right?

Yes, but I think we don't have anything to worry about, since this does NOT move
any files around, right? Just the namespace is changing. 

If you had done 'git mv source dest', then you would have had to be careful to
not lose the history (you have land using svn or cq).

[shawnsingh, 2012-10-15 17:12:11]

This patch was already landed in http://codereview.chromium.org/11145005/
instead.