Fix static initializers in DOM etc so that XPCOM can be restarted

RESOLVED FIXED in mozilla1.8alpha2

Status

()

Core
DOM: Core & HTML
RESOLVED FIXED
14 years ago
3 years ago

People

(Reporter: Benjamin Smedberg, Assigned: Benjamin Smedberg)

Tracking

Trunk
mozilla1.8alpha2
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

14 years ago
This is the last of my "xpcom-restart" static initializer bugs, I think... I now
have a usable-restartable tree.

Note to myself: there might be statics in non-gtk toolkits that I haven't found.
(Assignee)

Comment 1

14 years ago
Created attachment 145644 [details] [diff] [review]
DOM/layout/libpr0n (DOM is the only interesting one)
(Assignee)

Updated

14 years ago
Attachment #145644 - Flags: superreview?(brendan)
Attachment #145644 - Flags: review?(jst)
(Assignee)

Updated

14 years ago
Blocks: 239929
Comment on attachment 145644 [details] [diff] [review]
DOM/layout/libpr0n (DOM is the only interesting one)

- In dom/src/base/nsJSEnvironment.cpp:

+static PRBool isInitialized;

name this sIsInitialized, not isInitialized.

r=jst
Attachment #145644 - Flags: review?(jst) → review+

Comment 3

14 years ago
Comment on attachment 145644 [details] [diff] [review]
DOM/layout/libpr0n (DOM is the only interesting one)

sr=darin
Attachment #145644 - Flags: superreview?(brendan) → superreview+
(Assignee)

Comment 4

14 years ago
fixed on trunk
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.8alpha2

Updated

10 years ago
Component: DOM: Core → DOM: Core & HTML
QA Contact: ian → general
Created attachment 8594533 [details] [diff] [review]
patch 2 - Add BUG_COMPONENT metadata for tests in layout/reftests/
Attachment #8594533 - Flags: review?(gps)
Assignee: benjamin → dbaron
Fixing damage from broken "hg bzexport" when you tell it to bzexport something by numeric revision.
Assignee: dbaron → benjamin
Attachment #8594533 - Attachment is obsolete: true
Attachment #8594533 - Flags: review?(gps)
You need to log in before you can comment on or make changes to this bug.