Closed
Bug 1330403
Opened 8 years ago
Closed 8 years ago
Properly support remoteType in RDM
Categories
(DevTools :: Responsive Design Mode, defect, P3)
DevTools
Responsive Design Mode
Tracking
(firefox53 affected)
RESOLVED
INVALID
Tracking | Status | |
---|---|---|
firefox53 | --- | affected |
People
(Reporter: jryans, Unassigned)
References
Details
(Whiteboard: [rdm-v2][sb?])
After some changes in bug 1318767, RDM is no longer passing `remoteType` correctly to `updateBrowserRemoteness`.
I would also think the <iframe mozbrowser> needs to be passed the `remoteType` as well.
However, in limited manual testing, it does seem we are correctly swapping content in file process, so more investigation is needed...
Reporter | ||
Comment 1•8 years ago
|
||
Bug 1331601 fixes part of this by passing the correct arg so that the remoteType is restored on RDM exit.
However, RDM currently assumes remoteType is always "web", which wouldn't be true for file content, for example.
Depends on: 1331601
Reporter | ||
Comment 2•8 years ago
|
||
Bug 1317293 will enforce matching remoteTypes when swapping, which is good for correctness.
I am pretty sure this will break remoteType="file" in RDM for the moment, since the viewport is always created with "web".
Depends on: 1317293
Updated•8 years ago
|
Whiteboard: [rdm-v2] → [rdm-v2][sb?]
Comment 3•8 years ago
|
||
:jryans, what are the implications of this bug? I'm having trouble following what happens as a result of this.
For future reference for myself, or others, https://dxr.mozilla.org/mozilla-central/source/devtools/client/responsive.html/components/browser.js?q=path%3Adevtools%2Fclient%2Fresponsive.html%2Fcomponents%2Fbrowser.js&redirect_type=single#139-145 appears to be the "root" place where we actually set a value for |remoteType|, all the other instances of |remoteType| in |devtools/| appear to be passing an existing value through.
Flags: needinfo?(jryans)
Reporter | ||
Comment 4•8 years ago
|
||
Responsive Design Mode creates a separate browser (the one you found in components/browser.js) and uses a docshell swap to move existing content into that browser.
Initially, I expected that `remoteType` would have to match between the browsers for swapping to work (since that's true of most other browser attributes). Instead however, after bug 1338241, we swap between browsers of any `remoteType`, and then move the `remoteType` labels around afterwards to match up.
Here's STR to make sure things appear okay:
1. Open some file content in a normal tab
2. Double check using the Browser Console that `remoteType` is file (gBrowser.selectedBrowser)
3. Open Responsive Design Mode (Tools -> Web Dev -> RDM)
ER:
The content should appear inside the RDM viewport.
So, I think there's nothing left to do here.
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(jryans)
Resolution: --- → INVALID
Comment 5•8 years ago
|
||
Thanks!
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•