<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 22, 2016 at 2:24 PM, Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Le 2016-03-22 17:49, Darrell Walisser a écrit :<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Are you suggesting that every call to check window state<br>
(isFullscreen(), isMaximized(), ...) be removed, since the UI should<br>
not care about it?<br>
</blockquote>
<br></span>
Actually yes. The only valid case for resizing is if trying to adapt to new video size (and not every user agrees there). </blockquote><div><br></div><div>I agree it would be better if these checks are removed, they will always be subject to races such as this. However there are a bunch of these checks,  I don't really get the motivation for doing them. Mostly it seems they want to avoid changing the size of the video viewport when a toolbar or other UI element is hidden (by resizing the parent window), but I don't see what the big deal is with letting it grow or shrink.</div></div></div></div>