Skip to content
This repository was archived by the owner on Nov 1, 2022. It is now read-only.

EngineSessionState: Migrate components to writeTo(). #8370

Closed
pocmo opened this issue Sep 10, 2020 · 0 comments
Closed

EngineSessionState: Migrate components to writeTo(). #8370

pocmo opened this issue Sep 10, 2020 · 0 comments
Assignees
@pocmo
Labels
⌨️ code Technical debt, code clean up, small API change .. <session> Component: browser-session
Milestone
71.0.0 🐈

Comments

@pocmo
Copy link
Contributor

pocmo commented Sep 10, 2020 •
edited by data-sync-user
Loading

I am going to deprecated toJSON() in favor of a streaming API that uses JsonWriter (#8359). Eventually we should move all callers over to that and get rid of toJSON().

┆Issue is synchronized with this Jira Task
@pocmo pocmo added <session> Component: browser-session 🐞 bug Something isn't working ⌨️ code Technical debt, code clean up, small API change .. and removed 🐞 bug Something isn't working labels Sep 10, 2020
@pocmo pocmo mentioned this issue Sep 11, 2020
Merged
@pocmo pocmo self-assigned this Jan 7, 2021
pocmo added a commit to pocmo/android-components that referenced this issue Jan 7, 2021
@pocmo
Closes mozilla-mobile#8370: Remove EngineSessionState.toJSON().
f87fecd
pocmo added a commit to pocmo/android-components that referenced this issue Jan 8, 2021
@pocmo
Closes mozilla-mobile#8370: Remove EngineSessionState.toJSON().
338f326
@mergify mergify bot closed this as completed in e017b6f Jan 8, 2021
@Amejia481 Amejia481 added this to the 71.0.0 milestone Jan 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@pocmo pocmo

Labels
⌨️ code Technical debt, code clean up, small API change .. <session> Component: browser-session
Projects
None yet
Milestone
71.0.0 🐈
Development

No branches or pull requests
2 participants
@pocmo @Amejia481