[ash-md] Fixed TriView to relayout child Views when their preferred size changed.

[ash-md] Fixed TriView to relayout child Views when their preferred size changed.

Previously TriView child views would be allocated their preferred size
during the TriView layout.  However, if a child view's preferred size
later changed it did not trigger a TriView layout and the child view
would not be allocated space based on its new preferred size.

This change fixes it so that a TriView's child will be reallocated space
if its preferred size changes. This will not cause the TriView
container views to be resized when a child views preferred size changes.

BUG=667955
TEST=ash_unittests --gtest_filter=TriViewTest.ChildViewsPreferredSizeChanged

Committed: https://crrev.com/0f07a3024a009d9a367d07a2740a3ac96558ab79
Cr-Commit-Position: refs/heads/master@{#434818}

[bruthig, 2016-11-28 20:10:47]

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

[bruthig, 2016-11-28 20:11:15]

bruthig@chromium.org changed reviewers:
+ tdanderson@chromium.org

[bruthig, 2016-11-28 20:11:16]

tdanderson@, can you PTAL?

[commit-bot: I haz the power, 2016-11-28 20:12:40]

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

[tdanderson, 2016-11-28 21:47:37]

LGTM

https://codereview.chromium.org/2538513002/diff/1/ash/common/system/tray/tri_...
File ash/common/system/tray/tri_view.cc (right):

https://codereview.chromium.org/2538513002/diff/1/ash/common/system/tray/tri_...
ash/common/system/tray/tri_view.cc:31: class RelayoutView : public views::View {
Hm, I'm a bit surprised there isn't something like this in use somewhere else
already.

[bruthig, 2016-11-28 22:00:57]

The CQ bit was unchecked by bruthig@chromium.org

[bruthig, 2016-11-28 22:01:17]

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

[bruthig, 2016-11-28 22:01:17]

sky@, can you take a quick look at ui/views/*?

[sky, 2016-11-28 22:46:14]

LGTM

[bruthig, 2016-11-29 01:12:22]

The CQ bit was checked by bruthig@chromium.org

[commit-bot: I haz the power, 2016-11-29 01:13:22]

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-29 01:44:08]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480381942536390, "parent_rev":
"139d22f27f7dd4ac55b63e1409cfd060d173d01b", "commit_rev":
"f34bb743bb4a18d059c527300220a4039034a53d"}

[commit-bot: I haz the power, 2016-11-29 01:44:33]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-29 01:48:03]

Description was changed from

==========
[ash-md] Fixed TriView to relayout child Views when their preferred size
changed.

Previously TriView child views would be allocated their preferred size
during the TriView layout.  However, if a child view's preferred size
later changed it did not trigger a TriView layout and the child view
would not be allocated space based on its new preferred size.

This change fixes it so that a TriView's child will be reallocated space
if its preferred size changes. This will not cause the TriView
container views to be resized when a child views preferred size changes.

BUG=667955
TEST=ash_unittests --gtest_filter=TriViewTest.ChildViewsPreferredSizeChanged
==========

to

==========
[ash-md] Fixed TriView to relayout child Views when their preferred size
changed.

Previously TriView child views would be allocated their preferred size
during the TriView layout.  However, if a child view's preferred size
later changed it did not trigger a TriView layout and the child view
would not be allocated space based on its new preferred size.

This change fixes it so that a TriView's child will be reallocated space
if its preferred size changes. This will not cause the TriView
container views to be resized when a child views preferred size changes.

BUG=667955
TEST=ash_unittests --gtest_filter=TriViewTest.ChildViewsPreferredSizeChanged

Committed: https://crrev.com/0f07a3024a009d9a367d07a2740a3ac96558ab79
Cr-Commit-Position: refs/heads/master@{#434818}
==========

[commit-bot: I haz the power, 2016-11-29 01:48:04]

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