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

<bugzilla version="5.0.4"
          urlbase="https://www.w3.org/Bugs/Public/"
          
          maintainer="sysbot+bugzilla@w3.org"
>

    <bug>
          <bug_id>26602</bug_id>
          
          <creation_ts>2014-08-18 22:20:26 +0000</creation_ts>
          <short_desc>Cross-process damage control in requestFullscreen() and exitFullscreen()?</short_desc>
          <delta_ts>2014-08-26 13:16:16 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebAppsWG</product>
          <component>Fullscreen</component>
          <version>unspecified</version>
          <rep_platform>PC</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>INVALID</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Philip Jägenstedt">philipj</reporter>
          <assigned_to name="Anne">annevk</assigned_to>
          <cc>mike</cc>
    
    <cc>public-webapps</cc>
          
          <qa_contact>public-webapps-bugzilla</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>110269</commentid>
    <comment_count>0</comment_count>
    <who name="Philip Jägenstedt">philipj</who>
    <bug_when>2014-08-18 22:20:26 +0000</bug_when>
    <thetext>Here&apos;s an idea that doesn&apos;t fix any bug but may be worthwhile.

In requestFullscreen(), wait with the first ready check until just before the resize, so that if it requires IPC, that won&apos;t have to be done synchronously before requestFullscreen() returns. &quot;Fullscreen is not supported&quot; could be moved as well, just in case establishing that actually requires inspecting each document. Together with bug 26601 that should leave nothing fancy in the synchronous section.

In exitFullscreen(), return immediately after step 1 or 2, or otherwise move the population of parentDocs to the async section.

WDYT?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>110400</commentid>
    <comment_count>1</comment_count>
    <who name="Philip Jägenstedt">philipj</who>
    <bug_when>2014-08-21 11:48:18 +0000</bug_when>
    <thetext>On the other hand, this change would require specifying in greater detail when exactly the async steps are run, so that it&apos;s well defined which scripts and event handlers run between the requestFullscreen() call and the first async steps.

That might not be an improvement.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>110588</commentid>
    <comment_count>2</comment_count>
    <who name="Anne">annevk</who>
    <bug_when>2014-08-26 09:21:32 +0000</bug_when>
    <thetext>Not sure what to do here. Did you experiment with this?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>110604</commentid>
    <comment_count>3</comment_count>
    <who name="Philip Jägenstedt">philipj</who>
    <bug_when>2014-08-26 13:16:16 +0000</bug_when>
    <thetext>I didn&apos;t experiment, unfortunately I&apos;m just speculating on what might be a problem.

Doing the cross-process checks synchronously in requestFullscreen() could turn out to be a mistake, but the check needs to be done before the actual resize at a well-defined point for interop, so I don&apos;t know how to improve it.

Resolving as invalid until the problem turns out to be real.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>