1196163 - [e10s] Window.fullScreen always returns false for content window

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Xidorn Quan [:xidorn] UTC+10]

Step to reproduce:
1. open the Web Console
2. press F11 to enter fullscreen mode
3. in the console, execute `window.fullScreen`

Expected result:
the expression returns true

Actual result:
the expression returns false


This is not a regression. It is always this way as we do not store the fullscreen state in the content process at all.

Also, Window.fullScreen is not a standard attribute, hence this is probably not important. The only issue is that, the fullscreen-api-race test relies on this. I want to have a way to detect whether the window is actually exit fullscreen state in that test.

Comment 1

[Xidorn Quan [:xidorn] UTC+10]

Attachment: MozReview Request: Bug 1196163 part 1 - Use nsSizeMode instead of int32_t for nsIWidget::{SizeMode,SetSizeMode}.

Bug 1196163 part 1 - Use nsSizeMode instead of int32_t for nsIWidget::{SizeMode,SetSizeMode}.

Comment 2

[Xidorn Quan [:xidorn] UTC+10]

Attachment: MozReview Request: Bug 1196163 part 2 - Send sizemode as part of UpdateDimensions message to TabChild.

Bug 1196163 part 2 - Send sizemode as part of UpdateDimensions message to TabChild.

Comment 3

[Xidorn Quan [:xidorn] UTC+10]

Attachment: MozReview Request: Bug 1196163 part 3 - Derive value of window.fullScreen for content process from the sizemode of puppet widget.

Bug 1196163 part 3 - Derive value of window.fullScreen for content process from the sizemode of puppet widget.

Comment 4

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 8653784 [details]
MozReview Request: Bug 1196163 part 1 - Use nsSizeMode instead of int32_t for nsIWidget::{SizeMode,SetSizeMode}.

https://reviewboard.mozilla.org/r/17519/#review15655

Comment 5

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 8653785 [details]
MozReview Request: Bug 1196163 part 2 - Send sizemode as part of UpdateDimensions message to TabChild.

https://reviewboard.mozilla.org/r/17521/#review15657

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b8d3872064c7
https://hg.mozilla.org/integration/mozilla-inbound/rev/7e1086bf6018
https://hg.mozilla.org/integration/mozilla-inbound/rev/05c0212b9f16

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/9c38b505ef57

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d229498e103a

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/b8d3872064c7
https://hg.mozilla.org/mozilla-central/rev/7e1086bf6018
https://hg.mozilla.org/mozilla-central/rev/05c0212b9f16
https://hg.mozilla.org/mozilla-central/rev/9c38b505ef57
https://hg.mozilla.org/mozilla-central/rev/d229498e103a