Support serialization of complex objects with container value fields
Categories
(Remote Protocol :: WebDriver BiDi, task, P1)
Tracking
(firefox107 fixed)
Tracking | Status | |
---|---|---|
firefox107 | --- | fixed |
People
(Reporter: whimboo, Assigned: Sasha)
References
(Blocks 1 open bug, )
Details
(Whiteboard: [webdriver:m5], [wptsync upstream])
Attachments
(3 files, 2 obsolete files)
With this bug we want to extend our serialization for types that have a container type in their value field. This means adding support for the max depth
argument.
Note that this is not about using the strong handle
field which is bug 1770736.
Reporter | ||
Updated•1 year ago
|
Reporter | ||
Updated•1 year ago
|
Assignee | ||
Comment 1•10 months ago
|
||
In the scope of this bug, we also should remove the if condition introduced in https://bugzilla.mozilla.org/show_bug.cgi?id=1779226
Comment 2•9 months ago
|
||
Depends on D155446
This is missing tests, and might need to move to Bug 1770754, but since I brought it up when looking at supporting handle, I wanted to experiment with the
implementation.
Updated•9 months ago
|
Comment 3•9 months ago
|
||
Depends on D155518
Updated•9 months ago
|
Reporter | ||
Updated•9 months ago
|
Assignee | ||
Updated•9 months ago
|
Assignee | ||
Comment 4•8 months ago
|
||
Assignee | ||
Comment 5•8 months ago
|
||
Depends on D157634
Updated•8 months ago
|
Updated•8 months ago
|
Assignee | ||
Comment 6•8 months ago
|
||
Depends on D157634
Pushed by aborovova@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2faeb47919ce [bidi] Add internal ids to the duplicated objects for a given serialization. r=webdriver-reviewers,jdescottes,whimboo https://hg.mozilla.org/integration/autoland/rev/1fa586b566a7 [wdspec] Add tests for internal ids. r=webdriver-reviewers,jdescottes,whimboo https://hg.mozilla.org/integration/autoland/rev/a8c3d932b570 [bidi] Support serialization of complex objects with container value fields. r=webdriver-reviewers,jdescottes,whimboo
Comment 8•8 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/2faeb47919ce
https://hg.mozilla.org/mozilla-central/rev/1fa586b566a7
https://hg.mozilla.org/mozilla-central/rev/a8c3d932b570
Reporter | ||
Comment 9•8 months ago
|
||
James, can you please check why no upstream PR for the second patch has been created? Thanks!
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/36181 for changes under testing/web-platform/tests
Updated•8 months ago
|
Upstream PR merged by moz-wptsync-bot
Updated•8 months ago
|
Upstream PR merged by moz-wptsync-bot
Description
•