[Media Router WebUI] Handle header height changes.

[Media Router WebUI] Handle header height changes.

Previously, there were two issues:
- The header height is expanded to 62px when the email is shown, but not reduced back to 52px when the email is later hidden.
- The positioning update is only done for the sink view.

This change includes:
- Removing the height styling from CSS and setting it explicitly in the header element.
- Wrapping everything under the first run flow and header in a div for positioning updating purposes.

BUG=597414, 597411
Committed: https://crrev.com/aa5f0bcbe659641c6902d69a0ca33e8eb785533d
Cr-Commit-Position: refs/heads/master@{#383216}

[apacible, 2016-03-24 01:29:15]

Description was changed from

==========
header height

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

to

==========
[Work In Progress] [Media Router WebUI] Handle header height changes.

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

[apacible, 2016-03-24 18:04:39]

Patchset #1 (id:1) has been deleted

[apacible, 2016-03-24 18:46:46]

Patchset #1 (id:20001) has been deleted

[apacible, 2016-03-24 18:49:27]

Patchset #1 (id:40001) has been deleted

[apacible, 2016-03-24 18:52:35]

Description was changed from

==========
[Work In Progress] [Media Router WebUI] Handle header height changes.

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

to

==========
[Media Router WebUI] Handle header height changes.

BUG=597414, 597411
==========

[apacible, 2016-03-24 18:54:18]

Description was changed from

==========
[Media Router WebUI] Handle header height changes.

BUG=597414, 597411
==========

to

==========
[Media Router WebUI] Handle header height changes.

This change includes:
- Removing the height styling from CSS and setting it explicitly in the header
element.
- Wrapping everything under the first run flow and header in a div for
positioning updating purposes.

BUG=597414, 597411
==========

[apacible, 2016-03-24 18:54:18]

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

[apacible, 2016-03-24 18:58:43]

Description was changed from

==========
[Media Router WebUI] Handle header height changes.

This change includes:
- Removing the height styling from CSS and setting it explicitly in the header
element.
- Wrapping everything under the first run flow and header in a div for
positioning updating purposes.

BUG=597414, 597411
==========

to

==========
[Media Router WebUI] Handle header height changes.

Previously, there were two issues:
- The header height is expanded to 62px when the email is shown, but not reduced
back to 52px when the email is later hidden.
- The positioning update is only done for the sink view.

This change includes:
- Removing the height styling from CSS and setting it explicitly in the header
element.
- Wrapping everything under the first run flow and header in a div for
positioning updating purposes.

BUG=597414, 597411
==========

[apacible, 2016-03-24 18:59:19]

PTAL, thanks!

FYI the html file has an addition in lines 63 and 218 with everything between
indented, but the rietveld diff makes it look like there are more changes.

[amp, 2016-03-24 19:06:53]

https://codereview.chromium.org/1834573002/diff/60001/chrome/browser/resource...
File
chrome/browser/resources/media_router/elements/media_router_header/media_router_header.js
(right):

https://codereview.chromium.org/1834573002/diff/60001/chrome/browser/resource...
chrome/browser/resources/media_router/elements/media_router_header/media_router_header.js:230:
this.headerWithoutEmailHeight_ + 'px';
Thanks for updating this.  I thought I had verified that setting this back to
undefined properly reset the css back to the default, but maybe I missed it or
something changed since this first went in.

[mark a. foltz, 2016-03-24 23:45:32]

lgtm

[apacible, 2016-03-25 01:13:49]

The CQ bit was checked by apacible@chromium.org

[commit-bot: I haz the power, 2016-03-25 01:13:58]

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

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

Description was changed from

==========
[Media Router WebUI] Handle header height changes.

Previously, there were two issues:
- The header height is expanded to 62px when the email is shown, but not reduced
back to 52px when the email is later hidden.
- The positioning update is only done for the sink view.

This change includes:
- Removing the height styling from CSS and setting it explicitly in the header
element.
- Wrapping everything under the first run flow and header in a div for
positioning updating purposes.

BUG=597414, 597411
==========

to

==========
[Media Router WebUI] Handle header height changes.

Previously, there were two issues:
- The header height is expanded to 62px when the email is shown, but not reduced
back to 52px when the email is later hidden.
- The positioning update is only done for the sink view.

This change includes:
- Removing the height styling from CSS and setting it explicitly in the header
element.
- Wrapping everything under the first run flow and header in a div for
positioning updating purposes.

BUG=597414, 597411
==========

[commit-bot: I haz the power, 2016-03-25 01:19:23]

Committed patchset #1 (id:60001)

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

Description was changed from

==========
[Media Router WebUI] Handle header height changes.

Previously, there were two issues:
- The header height is expanded to 62px when the email is shown, but not reduced
back to 52px when the email is later hidden.
- The positioning update is only done for the sink view.

This change includes:
- Removing the height styling from CSS and setting it explicitly in the header
element.
- Wrapping everything under the first run flow and header in a div for
positioning updating purposes.

BUG=597414, 597411
==========

to

==========
[Media Router WebUI] Handle header height changes.

Previously, there were two issues:
- The header height is expanded to 62px when the email is shown, but not reduced
back to 52px when the email is later hidden.
- The positioning update is only done for the sink view.

This change includes:
- Removing the height styling from CSS and setting it explicitly in the header
element.
- Wrapping everything under the first run flow and header in a div for
positioning updating purposes.

BUG=597414, 597411

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

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

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