Move mojo/converters/geometry -> ui/gfx/geometry/mojo

Move mojo/converters/geometry -> ui/gfx/geometry/mojo

R=sky@chromium.org
TBR=tsepez@chromium.org
BUG=

Committed: https://crrev.com/7a77e85c2dbd440d718702b2580109eabe422f22
Cr-Commit-Position: refs/heads/master@{#396128}

[Ben Goodger (Google), 2016-05-25 20:27:34]

Description was changed from

==========
Move mojo/converters/geometry -> ui/gfx/geometry/mojo

BUG=
==========

to

==========
Move mojo/converters/geometry -> ui/gfx/geometry/mojo

R=sky@chromium.org
TBR=tsepez@chromium.org
BUG=
==========

[Ben Goodger (Google), 2016-05-25 20:27:34]

ben@chromium.org changed reviewers:
+ tsepez@google.com

[Ben Goodger (Google), 2016-05-25 20:27:47]

ben@chromium.org changed reviewers:
+ sky@chromium.org
- tsepez@google.com

[sky, 2016-05-25 20:31:41]

Rubber stamp LGTM

[Ben Goodger (Google), 2016-05-25 20:53:43]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-05-25 20:53:44]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2009073003/#ps40001
(title: ".")

[commit-bot: I haz the power, 2016-05-25 20:54:36]

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

[commit-bot: I haz the power, 2016-05-25 21:02:53]

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

[commit-bot: I haz the power, 2016-05-25 21:02:54]

Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Ben Goodger (Google), 2016-05-25 23:03:18]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-05-25 23:03:19]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2009073003/#ps60001
(title: ".")

[commit-bot: I haz the power, 2016-05-25 23:04:53]

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

[commit-bot: I haz the power, 2016-05-25 23:46:27]

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

[commit-bot: I haz the power, 2016-05-25 23:46:29]

Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Ben Goodger (Google), 2016-05-26 00:00:49]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-05-26 00:00:50]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2009073003/#ps80001
(title: ".")

[commit-bot: I haz the power, 2016-05-26 00:02:01]

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

[commit-bot: I haz the power, 2016-05-26 05:50:56]

Description was changed from

==========
Move mojo/converters/geometry -> ui/gfx/geometry/mojo

R=sky@chromium.org
TBR=tsepez@chromium.org
BUG=
==========

to

==========
Move mojo/converters/geometry -> ui/gfx/geometry/mojo

R=sky@chromium.org
TBR=tsepez@chromium.org
BUG=
==========

[commit-bot: I haz the power, 2016-05-26 05:50:57]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-05-26 05:51:57]

Description was changed from

==========
Move mojo/converters/geometry -> ui/gfx/geometry/mojo

R=sky@chromium.org
TBR=tsepez@chromium.org
BUG=
==========

to

==========
Move mojo/converters/geometry -> ui/gfx/geometry/mojo

R=sky@chromium.org
TBR=tsepez@chromium.org
BUG=

Committed: https://crrev.com/7a77e85c2dbd440d718702b2580109eabe422f22
Cr-Commit-Position: refs/heads/master@{#396128}
==========

[commit-bot: I haz the power, 2016-05-26 05:51:59]

Patchset 5 (id:??) landed as
https://crrev.com/7a77e85c2dbd440d718702b2580109eabe422f22
Cr-Commit-Position: refs/heads/master@{#396128}

[nodir, 2016-05-26 17:57:56]

nodir@chromium.org changed reviewers:
+ nodir@chromium.org

[nodir, 2016-05-26 17:57:57]

This is probably a cause of
https://bugs.chromium.org/p/chromium/issues/detail?id=615066

https://codereview.chromium.org/2009073003/diff/80001/ui/gfx/gfx.gyp
File ui/gfx/gfx.gyp (right):

https://codereview.chromium.org/2009073003/diff/80001/ui/gfx/gfx.gyp#newcode7
ui/gfx/gfx.gyp:7: '../../mojo/mojo_variables.gypi',
should be a list?

[stgao, 2016-05-26 18:28:06]

stgao@chromium.org changed reviewers:
+ stgao@chromium.org

[stgao, 2016-05-26 18:28:07]

https://codereview.chromium.org/2009073003/diff/80001/ui/gfx/gfx.gyp
File ui/gfx/gfx.gyp (right):

https://codereview.chromium.org/2009073003/diff/80001/ui/gfx/gfx.gyp#newcode7
ui/gfx/gfx.gyp:7: '../../mojo/mojo_variables.gypi',
On 2016/05/26 17:57:57, nodir wrote:
> should be a list?

It seems this cause a lot of failures in lkgr.
https://build.chromium.org/p/chromium.lkgr/console