<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>33741</bug_id>
          
          <creation_ts>2010-01-15 14:13:44 -0800</creation_ts>
          <short_desc>Fullscreen video stays visible after switching to another window</short_desc>
          <delta_ts>2010-03-17 17:14:07 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Media</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows XP</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar, PlatformOnly</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Adam Roben (:aroben)">aroben</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bdakin</cc>
    
    <cc>cmarrin</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>181386</commentid>
    <comment_count>0</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2010-01-15 14:13:44 -0800</bug_when>
    <thetext>To reproduce:

1. Go to a page with &lt;video controls&gt;
2. Click the fullscreen button in the video&apos;s controls
3. Switch to another window via Alt-Tab (e.g., the Safari window that contained
the &lt;video&gt;)

The fullscreen video is still visible in the background. On Mac, switching to another window in the same application is disallowed, but switching to another application causes the fullscreen video to go away. (Somewhat oddly, it comes back when you switch back to Safari!)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>181388</commentid>
    <comment_count>1</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2010-01-15 14:14:42 -0800</bug_when>
    <thetext>It&apos;s possible we&apos;ll want to make the fullscreen video go away whenever you switch to another window. One reason for this is that you can currently switch back to the window that contained the webpage with the &lt;video&gt; element while the fullscreen video is still playing in the background, which is just weird!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>181391</commentid>
    <comment_count>2</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2010-01-15 14:15:54 -0800</bug_when>
    <thetext>&lt;rdar://problem/7547579&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>199926</commentid>
    <comment_count>3</comment_count>
      <attachid>50750</attachid>
    <who name="Beth Dakin">bdakin</who>
    <bug_when>2010-03-15 16:47:18 -0700</bug_when>
    <thetext>Created attachment 50750
Patch

This patch makes the fullscreen video totally go away in Windows when you switch to another window. If you want to see the video fullscreen again, you have to click the fullscreen button again. I think this behavior makes the most sense, but I suppose this could be a matter of debate.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200301</commentid>
    <comment_count>4</comment_count>
    <who name="Chris Marrin">cmarrin</who>
    <bug_when>2010-03-16 11:07:04 -0700</bug_when>
    <thetext>I looked at this a while back and on Mac, you go back to full screen when you go back to the Safari Window. I don&apos;t think it&apos;s that big of a deal personally, but I can imagine HI having issues with it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200329</commentid>
    <comment_count>5</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2010-03-16 11:32:23 -0700</bug_when>
    <thetext>(In reply to comment #4)
&gt; I looked at this a while back and on Mac, you go back to full screen when you
&gt; go back to the Safari Window. I don&apos;t think it&apos;s that big of a deal personally

I agree it&apos;s not that big of a deal. Maybe we should investigate how other software behaves on Windows in this regard.

It looks like when you switch away from a full-screen (Flash) YouTube video, the video exits full-screen mode (on both platforms).

When you switch away from a full-screen instance of Windows Media Player, the video exits full-screen mode.

When you switch away from a full-screen instance of QuickTime Player (on Windows), the video remains in full-screen mode.

The Flash YouTube example seems the most applicable, since it&apos;s the only other case that has a &quot;source&quot; window that the video originated from (and is still present in in non-full-screen mode). But that doesn&apos;t mean we necessarily want to copy Flash&apos;s behavior.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201060</commentid>
    <comment_count>6</comment_count>
      <attachid>50750</attachid>
    <who name="Beth Dakin">bdakin</who>
    <bug_when>2010-03-17 17:13:54 -0700</bug_when>
    <thetext>Comment on attachment 50750
Patch

We talked this out over email, and we decided to keep the current behavior for the time being.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>50750</attachid>
            <date>2010-03-15 16:47:18 -0700</date>
            <delta_ts>2010-03-17 17:13:54 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>forReview.txt</filename>
            <type>text/plain</type>
            <size>1445</size>
            <attacher name="Beth Dakin">bdakin</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdC93aW4vQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYktpdC93aW4vQ2hh
bmdlTG9nCShyZXZpc2lvbiA1NjAyNSkKKysrIFdlYktpdC93aW4vQ2hhbmdlTG9nCSh3b3JraW5n
IGNvcHkpCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTAtMDMtMTUgIEJldGggRGFraW4gIDxiZGFraW5A
YXBwbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAg
ICAgIEZpeCBmb3IgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTMzNzQx
IEZ1bGxzY3JlZW4gCisgICAgICAgIHZpZGVvIHN0YXlzIHZpc2libGUgYWZ0ZXIgc3dpdGNoaW5n
IHRvIGFub3RoZXIgd2luZG93CisgICAgICAgIC1hbmQgY29ycmVzcG9uZGluZy0KKyAgICAgICAg
PHJkYXI6Ly9wcm9ibGVtLzc1NDc1Nzk+CisKKyAgICAgICAgKiBGdWxsc2NyZWVuVmlkZW9Db250
cm9sbGVyLmNwcDoKKyAgICAgICAgKEZ1bGxzY3JlZW5WaWRlb0NvbnRyb2xsZXI6OmRyYXcpOiBF
eGl0IGZ1bGxzY3JlZW4gaWYgdGhlIAorICAgICAgICBmb3JlZ3JvdW5kV2luZG93IGlzIG5laXRo
ZXIgdGhlIHZpZGVvIG5vciB0aGUgSFVELgorCiAyMDEwLTAzLTE0ICBEYW4gQmVybnN0ZWluICA8
bWl0ekBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgRGFyaW4gQWRsZXIuCkluZGV4
OiBXZWJLaXQvd2luL0Z1bGxzY3JlZW5WaWRlb0NvbnRyb2xsZXIuY3BwCj09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0t
IFdlYktpdC93aW4vRnVsbHNjcmVlblZpZGVvQ29udHJvbGxlci5jcHAJKHJldmlzaW9uIDU2MDA0
KQorKysgV2ViS2l0L3dpbi9GdWxsc2NyZWVuVmlkZW9Db250cm9sbGVyLmNwcAkod29ya2luZyBj
b3B5KQpAQCAtMzk3LDYgKzM5NywxMyBAQCBzdGF0aWMgU3RyaW5nIHRpbWVUb1N0cmluZyhmbG9h
dCB0aW1lKQogCiB2b2lkIEZ1bGxzY3JlZW5WaWRlb0NvbnRyb2xsZXI6OmRyYXcoKQogeworICAg
IEhXTkQgZm9yZWdyb3VuZFdpbmRvdyA9IEdldEZvcmVncm91bmRXaW5kb3coKTsKKyAgICBpZiAo
Zm9yZWdyb3VuZFdpbmRvdyAhPSBtX3ZpZGVvV2luZG93ICYmIGZvcmVncm91bmRXaW5kb3cgIT0g
bV9odWRXaW5kb3cpIHsKKyAgICAgICAgaWYgKG1fbWVkaWFFbGVtZW50KQorICAgICAgICAgICAg
bV9tZWRpYUVsZW1lbnQtPmV4aXRGdWxsc2NyZWVuKCk7CisgICAgICAgIHJldHVybjsKKyAgICB9
CisKICAgICBIREMgd2luZG93REMgPSBHZXREQyhtX2h1ZFdpbmRvdyk7CiAgICAgSERDIGJpdG1h
cERDID0gQ3JlYXRlQ29tcGF0aWJsZURDKHdpbmRvd0RDKTsKICAgICA6OlJlbGVhc2VEQyhtX2h1
ZFdpbmRvdywgd2luZG93REMpOwo=
</data>
<flag name="review"
          id="33918"
          type_id="1"
          status="-"
          setter="bdakin"
    />
          </attachment>
      

    </bug>

</bugzilla>