Closed
Bug 670040
Opened 13 years ago
Closed 13 years ago
Make nsISessionStartup.state a jsval to save repeatedly stringifying/parsing the initial state
Categories
(Firefox :: Session Restore, defect)
Firefox
Session Restore
Tracking
()
RESOLVED
FIXED
Firefox 8
People
(Reporter: mossop, Assigned: mossop)
References
Details
(Keywords: dev-doc-complete)
Attachments
(1 file)
11.23 KB,
patch
|
zpao
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•13 years ago
|
||
Attachment #544653 -
Flags: review?(paul)
Comment 2•13 years ago
|
||
Comment on attachment 544653 [details] [diff] [review] patch rev 1 yay jsvals!
Attachment #544653 -
Flags: review?(paul) → review+
Assignee | ||
Comment 3•13 years ago
|
||
Landed: http://hg.mozilla.org/mozilla-central/rev/a65a4165c236 I updated https://developer.mozilla.org/en/XPCOM_Interface_Reference/nsISessionStartup but this might want to go onto the Firefox 8 for developers page
Status: NEW → RESOLVED
Closed: 13 years ago
Flags: in-testsuite-
Keywords: dev-doc-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 8
Comment 4•13 years ago
|
||
Since http://hg.mozilla.org/mozilla-central/rev/a65a4165c236 it has been impossible to compile under VS2010. I can't quite figure out why, but, a bisection shows this changeset as the culprit. The crash will usually occur once shlibsign.exe is hit. From my Event log: Faulting application name: shlibsign.exe, version: 0.0.0.0, time stamp: 0x4e216d7d Faulting module name: jemalloc.dll, version: 8.0.0.4214, time stamp: 0x4e21707a Exception code: 0xc0000005 Fault offset: 0x00004672 Faulting process id: 0x16d8 Faulting application start time: 0x01cc43aa429bab0f Faulting application path: d:\build\mozilla-central\obj-i686-pc-mingw32\nss\shlibsign.exe Faulting module path: d:\build\mozilla-central\obj-i686-pc-mingw32\dist\bin\jemalloc.dll Report Id: 810dca07-af9d-11e0-b7f0-001b2112456b
Assignee | ||
Comment 5•13 years ago
|
||
(In reply to comment #4) > Since http://hg.mozilla.org/mozilla-central/rev/a65a4165c236 it has been > impossible to compile under VS2010. I can't quite figure out why, but, a > bisection shows this changeset as the culprit. The crash will usually occur > once shlibsign.exe is hit. That'd be pretty surprising, none of the code changed is compiled by VS. Could you file a new bug so we can track it though.
That is fallout from Bug 671441.
Comment 7•13 years ago
|
||
Pushed http://hg.mozilla.org/integration/fx-team/rev/0ffeb8ce63f1 to update a comment so it says jsval instead of string
Comment 8•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/0ffeb8ce63f1
Comment 9•13 years ago
|
||
Mossop updated the docs here: https://developer.mozilla.org/en/XPCOM_Interface_Reference/nsISessionStartup I added a note that the previous type was string. Also mentioned on Firefox 8 for developers.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•