Closed Bug 305439 Opened 19 years ago Closed 19 years ago

nsLayoutModule's Initialize is recursive

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 305310

People

(Reporter: peterv, Assigned: peterv)

Details

Attachments

(1 file)

nsLayoutModule's Initialize calls nsContentUtils::Init which does
CallGetService(NS_CONTENTPOLICY_CONTRACTID, &sContentPolicyService) which tries
to Initialize the layout module because it provides NS_CONTENTPOLICY_CONTRACTID.
Switching to NS_NewContentPolicy seems to make more sense.
Attached patch v1Splinter Review
Attachment #193378 - Flags: superreview?(bzbarsky)
Attachment #193378 - Flags: review?(bzbarsky)
Comment on attachment 193378 [details] [diff] [review]
v1

No, this is no good.  People need to be able to override this contract...
Attachment #193378 - Flags: superreview?(bzbarsky)
Attachment #193378 - Flags: superreview-
Attachment #193378 - Flags: review?(bzbarsky)
Attachment #193378 - Flags: review-

*** This bug has been marked as a duplicate of 305310 ***
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → DUPLICATE
Product: Core → Core Graveyard
Component: Layout: Misc Code → Layout
Product: Core Graveyard → Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: