Currently, we update the height of the header to accommodate the email if there's a cloud sink available. When the email is hidden, we don't update the height again.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/aa5f0bcbe659641c6902d69a0ca33e8eb785533d commit aa5f0bcbe659641c6902d69a0ca33e8eb785533d Author: apacible <apacible@chromium.org> Date: Fri Mar 25 01:19:12 2016 [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 Review URL: https://codereview.chromium.org/1834573002 Cr-Commit-Position: refs/heads/master@{#383216} [modify] https://crrev.com/aa5f0bcbe659641c6902d69a0ca33e8eb785533d/chrome/browser/resources/media_router/elements/media_router_container/media_router_container.css [modify] https://crrev.com/aa5f0bcbe659641c6902d69a0ca33e8eb785533d/chrome/browser/resources/media_router/elements/media_router_container/media_router_container.html [modify] https://crrev.com/aa5f0bcbe659641c6902d69a0ca33e8eb785533d/chrome/browser/resources/media_router/elements/media_router_container/media_router_container.js [modify] https://crrev.com/aa5f0bcbe659641c6902d69a0ca33e8eb785533d/chrome/browser/resources/media_router/elements/media_router_header/media_router_header.css [modify] https://crrev.com/aa5f0bcbe659641c6902d69a0ca33e8eb785533d/chrome/browser/resources/media_router/elements/media_router_header/media_router_header.js [modify] https://crrev.com/aa5f0bcbe659641c6902d69a0ca33e8eb785533d/chrome/browser/resources/media_router/media_router_common.css [modify] https://crrev.com/aa5f0bcbe659641c6902d69a0ca33e8eb785533d/chrome/test/data/webui/media_router/media_router_header_tests.js
Comment 1 by sko...@chromium.org
, Mar 23 2016