Closed
Bug 696875
Opened 14 years ago
Closed 14 years ago
Widget component interfaces not getting registered
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kats, Assigned: kats)
References
Details
Attachments
(1 file, 1 obsolete file)
|
1.83 KB,
patch
|
dougt
:
review+
|
Details | Diff | Splinter Review |
Trying to access widget component interfaces in javascript (e.g. doing Components.interfaces.nsIPrintSettingsService) doesn't work as expected. The reason for this appears to be that the widget/public/Makefile.in defines XPIDL_MODULE to be "widget", as does the widget/src/android/Makefile.in. This causes the android XPT file to clobber the widget XPT file, and so the only interface getting picked up is nsIAndroidBridge.
|
Assignee | |
Comment 1•14 years ago
|
||
Attachment #569177 -
Flags: review?(doug.turner)
|
||
Comment 2•14 years ago
|
||
why widget_fennec and not widget_android?
|
|
Assignee | |
Comment 3•14 years ago
|
||
Oh, whoops. I wasn't sure if widget_android would conflict with LIBRARY_NAME in some way (I looked at the widget/src/cocoa/ folder for inspiration and it had different names for LIBRARY_NAME and XPIDL_MODULE) so I used _fennec and forgot to test _android too. I can try _android as well to see if that works.
|
||
Comment 4•14 years ago
|
||
Comment on attachment 569177 [details] [diff] [review]
Rename XPIDL_MODULE
can't use the product name here. can we use widget_android?
Attachment #569177 -
Flags: review?(doug.turner) → review-
|
|
Assignee | |
Comment 5•14 years ago
|
||
Updated to widget_android
Attachment #569177 -
Attachment is obsolete: true
Attachment #569395 -
Flags: review?(doug.turner)
|
|
||
Updated•14 years ago
|
Attachment #569395 -
Flags: review?(doug.turner) → review+
|
|
||
Comment 6•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
|
||
Updated•5 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•