Closed
Bug 242024
Opened 21 years ago
Closed 21 years ago
gtk_moz_embed_set_app_components doesn't run mRegisterSelfProc
Categories
(Core Graveyard :: Embedding: GTK Widget, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: chpe, Assigned: blizzard)
Details
Attachments
(2 files)
1.72 KB,
patch
|
bryner
:
review+
blizzard
:
superreview+
|
Details | Diff | Splinter Review |
1.72 KB,
patch
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.1) Gecko/20020823 Netscape/7.0
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.1) Gecko/20020823 Netscape/7.0
If you set components with gtk_moz_embed_set_app_components(), the components'
register proc (nsModuleComponentInfo.mRegisterSelfProc) isn't run when the
component's factory is created in EmbedPrivate::RegisterAppComponents().
Reproducible: Always
Steps to Reproduce:
1.
2.
3.
Reporter | ||
Comment 1•21 years ago
|
||
Reporter | ||
Updated•21 years ago
|
Attachment #147277 -
Flags: superreview?(blizzard)
Attachment #147277 -
Flags: review?(bryner)
Assignee | ||
Comment 2•21 years ago
|
||
Comment on attachment 147277 [details] [diff] [review]
proposed fix
Unsure if this is how it's supposed to work, but the code looks fine.
Attachment #147277 -
Flags: superreview?(blizzard) → superreview+
Updated•21 years ago
|
Attachment #147277 -
Flags: review?(bryner) → review+
Comment 3•21 years ago
|
||
Looks like this doesnt apply anymore :/ Can you submit an updated patch please?
Reporter | ||
Comment 4•21 years ago
|
||
Comment 5•21 years ago
|
||
Patch checked in the trunk
Status: UNCONFIRMED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•