Extract opening and closing tab related helpers from browser.js to the TabManager module
Categories
(Remote Protocol :: Marionette, task, P3)
Tracking
(Not tracked)
People
(Reporter: jdescottes, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [webdriver:backlog])
See https://phabricator.services.mozilla.com/D109431#inline-619034:
It would be good to get these extra helper modules out of browser.js.
But maybe we should do it for the tab-manager.js module (which we already have in Remote Agent).
about:
let tabBrowser = browser.getTabBrowser(win);
See also https://phabricator.services.mozilla.com/D109431#inline-619052 which suggests to also move the getIdForBrowser created in window-manager.js to another helper.
Comment 1•4 years ago
|
||
This should only be about tab related methods. For window specific helpers we have a separate shared module.
Reporter | ||
Comment 2•4 years ago
|
||
(In reply to Henrik Skupin (:whimboo) [⌚️UTC+1] from comment #1)
This should only be about tab related methods. For window specific helpers we have a separate shared module.
Sorry, I meant browser & tab.
Comment 3•4 years ago
|
||
Not fully sure if we need this for M1 but adding it to the list for now so that we can get it discussed.
Reporter | ||
Updated•4 years ago
|
Reporter | ||
Comment 4•4 years ago
|
||
As discussed in the triage meeting, should move to this tab-manager module:
We should also attempts to extract tab-related methods from the Context class in browser.js: openTab and closeTab:
Updated•3 years ago
|
Updated•3 years ago
|
Comment 5•3 years ago
|
||
We don't need it right away but would be good to have support for Firefox on Android for sure.
Reporter | ||
Updated•3 years ago
|
Reporter | ||
Updated•3 years ago
|
Comment 6•3 years ago
|
||
(In reply to Julian Descottes [:jdescottes] from comment #4)
As discussed in the triage meeting, should move to this tab-manager module:
All this items have been done by bug 1747222.
We should also attempts to extract tab-related methods from the Context class in browser.js: openTab and closeTab:
We cannot fully extract these given that WebDriver HTTP will need extra code (waiting for the tab to finish loading) but maybe we can have the custom code in driver.js instead. I've made some initial changes also on bug 1723919.
Comment 7•3 years ago
|
||
I'm doing some updates for opening and selecting a tab over on bug 1749666.
Comment 8•2 years ago
|
||
Lets see what else we can simplify once bug 1533058 has been fixed.
Updated•2 years ago
|
Updated•2 years ago
|
Description
•