Use a custom Module base class for WebDriver BiDi
Categories
(Remote Protocol :: WebDriver BiDi, enhancement, P3)
Tracking
(firefox114 fixed)
| Tracking | Status | |
|---|---|---|
| firefox114 | --- | fixed |
People
(Reporter: whimboo, Assigned: victoria.o.ajala, Mentored)
Details
(Whiteboard: [lang=js][webdriver:m7][webdriver:external])
Attachments
(1 file)
It might be helpful to have a custom base class for the MessageHandler's Module class just dedicated for WebDriver BiDi. Hereby we could add more properties like getting the nodeCache or the processActor which should not be done in the MessageHandler's implementation because that's WebDriver related.
We should discuss if that is an option and if yes which other properties and methods might be helpful to have as well.
|
Reporter | |
Comment 1•1 year ago
|
||
We agreed on that a dedicated Module base class for WebDriver BiDi would be helpful. We can get started with sub-classing the message manager class and provide access to the processActor, and additionally a shortcut to the nodeCache.
The implementation should take place in the remote/webdriver-bidi folder and the file name can be as well stay with Module.sys.jsm.
|
Assignee | |
Comment 2•1 year ago
|
||
Hi, I would be working on this bug @whimboo
|
|
Reporter | |
Comment 3•1 year ago
|
||
Sounds good! Let me know if you have questions. Happy to discuss / explain on Matrix.
|
|
Assignee | |
Comment 4•1 year ago
|
||
Sorry for the delay @whimboo, If I understand you correctly in comment 2, I would be adding a Module base class for WebDriver BiDi. Could you elaborate on that for me?
|
|
Reporter | |
Comment 5•1 year ago
|
||
Yes, that is correct. This BiDi specific Module class would inherit from the MessageManager module base class as referenced in comment 0. With this addition all the BiDi specific module implementations would then make use of the new class instead.
|
|
Assignee | |
Comment 6•1 year ago
|
||
|
||
Updated•1 year ago
|
|
|
Assignee | |
Comment 7•1 year ago
|
||
Hi @whimboo, I attached a patch to this, I have only created the Module.sys.jsm file in remote/webdriver-bidi How do I get a base bidi module working like you mentioned in the comment?
|
||
Updated•1 year ago
|
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/903008b27dd0 Used a custom Module base class for WebDriver BiDi. r=jdescottes
|
||
Comment 9•1 year ago
|
||
| bugherder | ||
|
|
Reporter | |
Updated•11 months ago
|
Description
•