Take size in ink drop masks instead of bounds

Reland: Take size in ink drop masks instead of bounds

Currently, constructors for ink drop masks, take layer bounds which is
expected to have (0, 0) as their origin. It's better for them to take
size instead.

BUG=612573
TEST=none

Committed: https://crrev.com/431883e99a2fcdc5e6e54868ac98d3474aaaf198
Committed: https://crrev.com/7c8b9453199a880668feab3a3f1bf39ca603a447
Cr-Original-Commit-Position: refs/heads/master@{#432898}
Cr-Commit-Position: refs/heads/master@{#432971}

[mohsen, 2016-11-17 05:05:31]

The CQ bit was checked by mohsen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-17 05:06:00]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[mohsen, 2016-11-17 05:08:32]

mohsen@chromium.org changed reviewers:
+ bruthig@chromium.org, jamescook@chromium.org

[mohsen, 2016-11-17 05:08:32]

+bruthig@ for changes in /ui/views/animation/.
+jamescook@ for changes in /ash/.

[mohsen, 2016-11-17 05:22:40]

Description was changed from

==========
Take size in ink drop masks instead of bounds

Currently, constructors for ink drop masks, take layer bounds which is
expected to have (0, 0) as their origin. It's better for them to take
size instead.

BUG=612573
==========

to

==========
Take size in ink drop masks instead of bounds

Currently, constructors for ink drop masks, take layer bounds which is
expected to have (0, 0) as their origin. It's better for them to take
size instead.

BUG=612573
TEST=none
==========

[bruthig, 2016-11-17 05:56:18]

lgtm

[commit-bot: I haz the power, 2016-11-17 05:59:46]

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

[commit-bot: I haz the power, 2016-11-17 05:59:47]

Dry run: This issue passed the CQ dry run.

[James Cook, 2016-11-17 16:49:54]

LGTM. Nice simplification.

[mohsen, 2016-11-17 16:54:55]

The CQ bit was checked by mohsen@chromium.org

[commit-bot: I haz the power, 2016-11-17 16:55:37]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-17 17:01:20]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-17 17:05:59]

Description was changed from

==========
Take size in ink drop masks instead of bounds

Currently, constructors for ink drop masks, take layer bounds which is
expected to have (0, 0) as their origin. It's better for them to take
size instead.

BUG=612573
TEST=none
==========

to

==========
Take size in ink drop masks instead of bounds

Currently, constructors for ink drop masks, take layer bounds which is
expected to have (0, 0) as their origin. It's better for them to take
size instead.

BUG=612573
TEST=none

Committed: https://crrev.com/431883e99a2fcdc5e6e54868ac98d3474aaaf198
Cr-Commit-Position: refs/heads/master@{#432898}
==========

[commit-bot: I haz the power, 2016-11-17 17:06:00]

Patchset 1 (id:??) landed as
https://crrev.com/431883e99a2fcdc5e6e54868ac98d3474aaaf198
Cr-Commit-Position: refs/heads/master@{#432898}

[Evan Stade, 2016-11-17 17:23:01]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2510633004/ by estade@chromium.org.

The reason for reverting is: compile failure on Windows -
https://build.chromium.org/p/chromium.win/builders/Win%20Builder/builds/32175...

'views::CircleInkDropMask::CircleInkDropMask(const views::CircleInkDropMask &)':
cannot convert argument 1 from 'const gfx::Rect' to 'const gfx::Size &'.

[mohsen, 2016-11-17 18:31:19]

The CQ bit was checked by mohsen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-17 18:32:59]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[mohsen, 2016-11-17 20:18:52]

Description was changed from

==========
Take size in ink drop masks instead of bounds

Currently, constructors for ink drop masks, take layer bounds which is
expected to have (0, 0) as their origin. It's better for them to take
size instead.

BUG=612573
TEST=none

Committed: https://crrev.com/431883e99a2fcdc5e6e54868ac98d3474aaaf198
Cr-Commit-Position: refs/heads/master@{#432898}
==========

to

==========
Reland: Take size in ink drop masks instead of bounds

Currently, constructors for ink drop masks, take layer bounds which is
expected to have (0, 0) as their origin. It's better for them to take
size instead.

BUG=612573
TEST=none

Committed: https://crrev.com/431883e99a2fcdc5e6e54868ac98d3474aaaf198
Cr-Commit-Position: refs/heads/master@{#432898}
==========

[commit-bot: I haz the power, 2016-11-17 20:36:38]

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

[commit-bot: I haz the power, 2016-11-17 20:36:39]

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

[mohsen, 2016-11-17 20:43:12]

The CQ bit was checked by mohsen@chromium.org

[mohsen, 2016-11-17 20:43:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
jamescook@chromium.org, bruthig@chromium.org
Link to the patchset: https://codereview.chromium.org/2508063004/#ps20001
(title: "Rebased")

[commit-bot: I haz the power, 2016-11-17 20:45:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-17 21:42:48]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-17 21:48:42]

Description was changed from

==========
Reland: Take size in ink drop masks instead of bounds

Currently, constructors for ink drop masks, take layer bounds which is
expected to have (0, 0) as their origin. It's better for them to take
size instead.

BUG=612573
TEST=none

Committed: https://crrev.com/431883e99a2fcdc5e6e54868ac98d3474aaaf198
Cr-Commit-Position: refs/heads/master@{#432898}
==========

to

==========
Reland: Take size in ink drop masks instead of bounds

Currently, constructors for ink drop masks, take layer bounds which is
expected to have (0, 0) as their origin. It's better for them to take
size instead.

BUG=612573
TEST=none

Committed: https://crrev.com/431883e99a2fcdc5e6e54868ac98d3474aaaf198
Committed: https://crrev.com/7c8b9453199a880668feab3a3f1bf39ca603a447
Cr-Original-Commit-Position: refs/heads/master@{#432898}
Cr-Commit-Position: refs/heads/master@{#432971}
==========

[commit-bot: I haz the power, 2016-11-17 21:48:44]

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