Closed
Bug 909642
Opened 11 years ago
Closed 11 years ago
Include nsNPAPIPluginInstance a bit less in headers
Categories
(Core Graveyard :: Plug-ins, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla26
People
(Reporter: bzbarsky, Assigned: bzbarsky)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
|
(deleted),
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
It pulls in a bunch of code that ends up all over the place (e.g. in bindings) if it's included in headers.
|
Assignee | |
Comment 1•11 years ago
|
||
Attachment #795836 -
Flags: review?(benjamin)
|
||
Updated•11 years ago
|
Attachment #795836 -
Flags: review?(benjamin) → review+
|
||
Comment 2•11 years ago
|
||
Yay, this fixes something that I hit in my patch for bug 912702!
|
|
||
Comment 3•11 years ago
|
||
I need this badly, so I went ahead and landed it. Thanks for fixing this, Boris!
https://hg.mozilla.org/integration/mozilla-inbound/rev/080a54213c6e
|
|
Assignee | |
Comment 4•11 years ago
|
||
No problem. Thanks for pushing it!
Flags: in-testsuite-
Whiteboard: [need review]
Target Milestone: --- → mozilla26
|
||
Comment 5•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
|
||
Updated•2 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•