Add origin() method on View and use it to simplify some callers.

Add origin() method on View and use it to simplify some callers.

I only looked for calls to "bounds().origin()".  We could probably change a lot
more spots by fixing places that call both x() and y().

BUG=none
TEST=none

Committed: https://crrev.com/edf5fb1403295bb60531db023307944364f4d725
Cr-Commit-Position: refs/heads/master@{#441413}

[Peter Kasting, 2016-12-21 22:54:13]

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

[commit-bot: I haz the power, 2016-12-21 22:54:47]

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

[Peter Kasting, 2016-12-21 22:56:19]

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

[Peter Kasting, 2016-12-21 22:56:20]

Not blocking anything.  Should be easy to review.

https://codereview.chromium.org/2595163002/diff/1/chrome/browser/ui/views/tab...
File chrome/browser/ui/views/tabs/tab_strip.cc (right):

https://codereview.chromium.org/2595163002/diff/1/chrome/browser/ui/views/tab...
chrome/browser/ui/views/tabs/tab_strip.cc:2596: gfx::Size
tab_size(Tab::GetTouchWidth(), GetLayoutConstant(TAB_HEIGHT));
This isn't related to the rest of the CL, it snuck in as a separate
simplification.

It makes no behavior change (GetMinimumActiveSize() already used TAB_HEIGHT for
the height), it's just shorter and more efficient.

[commit-bot: I haz the power, 2016-12-22 00:10:17]

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

[commit-bot: I haz the power, 2016-12-22 00:10:17]

Dry run: This issue passed the CQ dry run.

[sky, 2017-01-03 22:05:19]

LGTM

[Peter Kasting, 2017-01-04 17:35:14]

The CQ bit was checked by pkasting@chromium.org

[Peter Kasting, 2017-01-04 17:35:15]

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/2595163002/#ps20001
(title: "Resync")

[commit-bot: I haz the power, 2017-01-04 17:35: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, 2017-01-04 18:30:22]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483551314645950,
"parent_rev": "bd3fd2bef4fa15b93a13b5175da7a5a83901581b", "commit_rev":
"a96081b9d41458a682d72bb9f010a8698a969aa5"}

[commit-bot: I haz the power, 2017-01-04 18:30:51]

Description was changed from

==========
Add origin() method on View and use it to simplify some callers.

I only looked for calls to "bounds().origin()".  We could probably change a lot
more spots by fixing places that call both x() and y().

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

to

==========
Add origin() method on View and use it to simplify some callers.

I only looked for calls to "bounds().origin()".  We could probably change a lot
more spots by fixing places that call both x() and y().

BUG=none
TEST=none

Review-Url: https://codereview.chromium.org/2595163002
==========

[commit-bot: I haz the power, 2017-01-04 18:30:52]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2017-01-04 18:33:16]

Description was changed from

==========
Add origin() method on View and use it to simplify some callers.

I only looked for calls to "bounds().origin()".  We could probably change a lot
more spots by fixing places that call both x() and y().

BUG=none
TEST=none

Review-Url: https://codereview.chromium.org/2595163002
==========

to

==========
Add origin() method on View and use it to simplify some callers.

I only looked for calls to "bounds().origin()".  We could probably change a lot
more spots by fixing places that call both x() and y().

BUG=none
TEST=none

Committed: https://crrev.com/edf5fb1403295bb60531db023307944364f4d725
Cr-Commit-Position: refs/heads/master@{#441413}
==========

[commit-bot: I haz the power, 2017-01-04 18:33:17]

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