1544371 - Convert content-sessionStore to C++ - [stageII] FormDataListener rewriting

Status: VERIFIED FIXED

(Firefox :: Session Restore, task, P2)

Description

[Alphan Chen [:alchen]]

This bug is from bug 1474130 comment 22.

In this bug, we will focus on "FormDataListener" and "SessionStorageListener" rewriting.

Comment 1

[Alphan Chen [:alchen]]

Attachment: Bug 1544371 - Implement FormDataListener for contentSessionStore in C++

Comment 2

[Alphan Chen [:alchen]]

Since the patch is too big, I create another bug 1549975 for sessionStorageListener rewriting.

Comment 3

[Alphan Chen [:alchen]]

Current try result. It looks fine.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=25ac8372a21a57a58535593f66950483b0b35bc4

Comment 4

[Alphan Chen [:alchen]]

Try:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=4a77d4f5a039f439ba9dac4dcd962625edbc5b47&selectedJob=256951271

Comment 5

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Reviewbot in Phabricator says the (eslint) linting failures are still not fixed.

Comment 6

[Alphan Chen [:alchen]]

I am working on it.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e78733e0ffcf
Implement FormDataListener for contentSessionStore in C++ r=peterv

Comment 8

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/e78733e0ffcf

Comment 9

[Emil Ghitta, QA [:emilghitta]]

This has been covered by our recent manual testing efforts Test Report.

Marking this as verified fixed since no regressions were encountered during our test run.