Closed
Bug 1544371
Opened 5 years ago
Closed 5 years ago
Convert content-sessionStore to C++ - [stageII] FormDataListener rewriting
Categories
(Firefox :: Session Restore, task, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox70 | --- | verified |
People
(Reporter: alchen, Assigned: alchen)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
This bug is from bug 1474130 comment 22.
In this bug, we will focus on "FormDataListener" and "SessionStorageListener" rewriting.
|
Assignee | |
Updated•5 years ago
|
Priority: -- → P2
|
||
Updated•5 years ago
|
Type: enhancement → task
|
||
Updated•5 years ago
|
No longer blocks: ss-feature
No longer blocks: Session_managers
|
|
Assignee | |
Comment 1•5 years ago
|
||
|
|
Assignee | |
Comment 2•5 years ago
|
||
Since the patch is too big, I create another bug 1549975 for sessionStorageListener rewriting.
No longer depends on: 1549973
Summary: Convert content-sessionStore to C++ - stage II → Convert content-sessionStore to C++ - [stageII] FormDataListener rewriting
|
||
Updated•5 years ago
|
Assignee: nobody → alchen
Status: NEW → ASSIGNED
Fission Milestone: --- → M3
|
|
||
Updated•5 years ago
|
Blocks: memshrink-content
|
|
Assignee | |
Comment 3•5 years ago
|
||
Current try result. It looks fine.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=25ac8372a21a57a58535593f66950483b0b35bc4
|
|
||
Updated•5 years ago
|
Fission Milestone: M3 → M4
|
|
Assignee | |
Updated•5 years ago
|
Keywords: checkin-needed
|
|
Assignee | |
Comment 4•5 years ago
|
||
|
||
Comment 5•5 years ago
|
||
Reviewbot in Phabricator says the (eslint) linting failures are still not fixed.
Flags: needinfo?(alchen)
Keywords: checkin-needed
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e78733e0ffcf Implement FormDataListener for contentSessionStore in C++ r=peterv
|
||
Comment 8•5 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox70:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
|
||
Comment 9•5 years ago
|
||
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.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•