1507286 - Convert SessionStorage.jsm to C++

Status: RESOLVED FIXED

(Firefox :: Session Restore, enhancement, P2)

Description

[Alphan Chen [:alchen]]

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.

Comment 1

[Alphan Chen [:alchen]]

Attachment: Bug 1507286 - Convert content-sessionStore.jsm to C++

Test Plani: browser/components/sessionstore/test/

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0509a9edc58a
Convert content-sessionStore.jsm to C++ r=mikedeboer,nika

Comment 3

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/0509a9edc58a

Comment 4

[Pulsebot]

Backout by [e-mail]:
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

Comment 5

[Andrei Ciure[:aciure]]

Backed out 2 changesets (bug 1507286, bug 1497146) for causing multiple crashes in nsFocusManager::GetRedirectedFocus

push that caused the backout:
https://hg.mozilla.org/mozilla-central/rev/21ad3aeb636f
https://hg.mozilla.org/mozilla-central/rev/0509a9edc58a

backout: https://hg.mozilla.org/mozilla-central/rev/b08b9f22ad06e55aa83e9c85c74db82c50552094

Comment 6

[Alphan Chen [:alchen]]

I am checking the crashes.

Comment 7

[Alphan Chen [:alchen]]

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

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1efa956efdff
Convert content-sessionStore.jsm to C++ r=mikedeboer,nika

Comment 9

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/1efa956efdff