This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 26602 - Cross-process damage control in requestFullscreen() and exitFullscreen()?
Summary: Cross-process damage control in requestFullscreen() and exitFullscreen()?
Status: RESOLVED INVALID
Alias: None
Product: WebAppsWG
Classification: Unclassified
Component: Fullscreen (show other bugs)
Version: unspecified
Hardware: PC All
: P2 normal
Target Milestone: ---
Assignee: Anne
QA Contact: public-webapps-bugzilla
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-08-18 22:20 UTC by Philip Jägenstedt
Modified: 2014-08-26 13:16 UTC (History)
2 users (show)

See Also:


Attachments

Description Philip Jägenstedt 2014-08-18 22:20:26 UTC
Here's an idea that doesn'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't have to be done synchronously before requestFullscreen() returns. "Fullscreen is not supported" 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?
Comment 1 Philip Jägenstedt 2014-08-21 11:48:18 UTC
On the other hand, this change would require specifying in greater detail when exactly the async steps are run, so that it's well defined which scripts and event handlers run between the requestFullscreen() call and the first async steps.

That might not be an improvement.
Comment 2 Anne 2014-08-26 09:21:32 UTC
Not sure what to do here. Did you experiment with this?
Comment 3 Philip Jägenstedt 2014-08-26 13:16:16 UTC
I didn't experiment, unfortunately I'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't know how to improve it.

Resolving as invalid until the problem turns out to be real.