1593708 - Persist "inRDMPane" attribute set on the document while in RDM mode

Status: RESOLVED FIXED

(DevTools :: Responsive Design Mode, defect, P1)

Description

[Micah [:mtigley] (she/her)]

When RDM is open and the user either refreshes or performs a "OnLocationChange" event, attributes on the document (such as the inRDMPane) need to persist.

Steps to Reproduce

1. Make sure devtools.responsive.browserUI.enabled
2. Open RDM, and select the iPad from the devices list.
3. Check screen.height in the DevTools console. You should see a value of 1022 (which is supposed to be 1024, but that's another issue)
4. Refresh the page and check the value screen.height again.

ER:
The value should still be 1022

AR:
The value is the actual height of the physical device.

Comment 1

[Micah [:mtigley] (she/her)]

For context, RDM is currently persisting these attributes by overriding the WebProgressListener in its frame content script.

Since we're removing the use of the <iframe mozbrowser> as part of the DevTools fission effort, this also means we will removing its injected content frame script. So we need to find another way to persist the document attributes RDM manipulates when refreshed or navigated. In particular, RDM is using these WebIDL Document extensions to set inRDMPane and screen.orientation:

• inRDMPane
• setInRDMPaneOrientation

Comment 2

[Micah [:mtigley] (she/her)]

After discussing this issue with the Fission team, we need to be setting the inRDMPane attribute on the BrowsingContext and not the Document if we need it to persist. I can delve into this more.

Comment 3

[Micah [:mtigley] (she/her)]

Attachment: Bug 1593708 - Part 1: Add inRDMPane property to BrowsingContext and expose it to JS.

Comment 4

[Micah [:mtigley] (she/her)]

Attachment: Bug 1593708 - Part 2: Get inRDMPane value from BrowsingContext instead of Document.

Comment 5

[Micah [:mtigley] (she/her)]

Attachment: Bug 1593708 - Part 3: Remove inRDMPane attribute from Document.

Comment 6

[Micah [:mtigley] (she/her)]

Attachment: Bug 1593708 - Part 4: Update RDM to set inRDMPane property on the BrowsingContext.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8dd6e18ae2af
Part 1: Add inRDMPane property to BrowsingContext and expose it to JS. r=smaug
https://hg.mozilla.org/integration/autoland/rev/157248203e4a
Part 2: Get inRDMPane value from BrowsingContext instead of Document. r=smaug
https://hg.mozilla.org/integration/autoland/rev/33d4fb0808c9
Part 3: Remove inRDMPane attribute from Document. r=bradwerth,smaug
https://hg.mozilla.org/integration/autoland/rev/32913780ebb3
Part 4: Update RDM to set inRDMPane property on the BrowsingContext. r=bradwerth

Comment 8

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/8dd6e18ae2af
https://hg.mozilla.org/mozilla-central/rev/157248203e4a
https://hg.mozilla.org/mozilla-central/rev/33d4fb0808c9
https://hg.mozilla.org/mozilla-central/rev/32913780ebb3