Closed Bug 1507286 Opened 2 years ago Closed 2 years ago

Convert SessionStorage.jsm to C++

Categories

(Firefox :: Session Restore, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
Firefox 67
Tracking Status
firefox67 --- fixed

People

(Reporter: alchen, Assigned: alchen)

References

Details

Attachments

(1 file)

Before doing bug 1474130, I would like to rewrite the js modules which are used in ContentSessionStore.jsm.

In this bug, I will focus on SessionStorage.jsm.
Blocks: 1474130
Test Plani: browser/components/sessionstore/test/
Assignee: nobody → alchen
Priority: -- → P2
Status: NEW → ASSIGNED
Pushed by kjang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0509a9edc58a
Convert content-sessionStore.jsm to C++ r=mikedeboer,nika
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66
Backout by aciure@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/b08b9f22ad06
Backed out 2 changesets (bug 1507286, bug 1497146) for causing multiple crashes in nsFocusManager::GetRedirectedFocus a=backout
Flags: needinfo?(alchen)

I am checking the crashes.

Flags: needinfo?(alchen)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: Firefox 66 → ---

After investigating, the crash is not related to this patch.
I will rebase the patch and submit it today.

Pushed by xeonchen@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/1efa956efdff
Convert content-sessionStore.jsm to C++ r=mikedeboer,nika
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
Blocks: 1564412
You need to log in before you can comment on or make changes to this bug.