[Media Router] Fix timing of initially setting margins.

[Media Router] Fix timing of initially setting margins.

Currently, the sink list view is initially clipped at the top since the top margin styling is being set before the container header is rendered. The first time updateElementPositioning_() is called when the dialog initially is opened should be after the element is attached.

BUG=580394
Committed: https://crrev.com/96424f368f476053e4bd957ca5d637299fabef6c
Cr-Commit-Position: refs/heads/master@{#371295}

[apacible, 2016-01-22 23:36:19]

Description was changed from

==========
fix height top

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

to

==========
[Media Router] Fix timing of initially setting margins.

Currently, the sink list view is initially clipped at the top since the top
margin styling is being set before the container header is rendered.
updateElementPositioning_() should be called after the element is attached.

BUG=580394
==========

[apacible, 2016-01-22 23:36:19]

apacible@chromium.org changed reviewers:
+ imcheng@chromium.org

[apacible, 2016-01-22 23:36:46]

Description was changed from

==========
[Media Router] Fix timing of initially setting margins.

Currently, the sink list view is initially clipped at the top since the top
margin styling is being set before the container header is rendered.
updateElementPositioning_() should be called after the element is attached.

BUG=580394
==========

to

==========
[Media Router] Fix timing of initially setting margins.

Currently, the sink list view is initially clipped at the top since the top
margin styling is being set before the container header is rendered. The first
time updateElementPositioning_() is called when the dialog initially is opened
should be after the element is attached.

BUG=580394
==========

[apacible, 2016-01-22 23:37:11]

PTAL, thanks!

[imcheng, 2016-01-25 18:54:41]

lgtm

[apacible, 2016-01-25 18:55:06]

The CQ bit was checked by apacible@chromium.org

[commit-bot: I haz the power, 2016-01-25 18:55:46]

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

[commit-bot: I haz the power, 2016-01-25 20:16:05]

Description was changed from

==========
[Media Router] Fix timing of initially setting margins.

Currently, the sink list view is initially clipped at the top since the top
margin styling is being set before the container header is rendered. The first
time updateElementPositioning_() is called when the dialog initially is opened
should be after the element is attached.

BUG=580394
==========

to

==========
[Media Router] Fix timing of initially setting margins.

Currently, the sink list view is initially clipped at the top since the top
margin styling is being set before the container header is rendered. The first
time updateElementPositioning_() is called when the dialog initially is opened
should be after the element is attached.

BUG=580394
==========

[commit-bot: I haz the power, 2016-01-25 20:16:06]

Committed patchset #1 (id:1)

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

Description was changed from

==========
[Media Router] Fix timing of initially setting margins.

Currently, the sink list view is initially clipped at the top since the top
margin styling is being set before the container header is rendered. The first
time updateElementPositioning_() is called when the dialog initially is opened
should be after the element is attached.

BUG=580394
==========

to

==========
[Media Router] Fix timing of initially setting margins.

Currently, the sink list view is initially clipped at the top since the top
margin styling is being set before the container header is rendered. The first
time updateElementPositioning_() is called when the dialog initially is opened
should be after the element is attached.

BUG=580394

Committed: https://crrev.com/96424f368f476053e4bd957ca5d637299fabef6c
Cr-Commit-Position: refs/heads/master@{#371295}
==========

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

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