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•2 years ago
|
Reporter | ||
Updated•2 years ago
|
Assignee | ||
Comment 1•2 years 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•2 years 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•2 years ago
|
Updated•2 years ago
|
Reporter | ||
Updated•2 years ago
|
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 4•2 years ago
|
||
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Reporter | ||
Updated•2 years ago
|
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•2 years 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•2 years 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
Upstream PR merged by moz-wptsync-bot
Updated•2 years ago
|
Upstream PR merged by moz-wptsync-bot
Description
•