Closed
Bug 680881
Opened 13 years ago
Closed 13 years ago
Add a className parameter to SHELL_WRAPPER's and GETFUNC macros
Categories
(Core Graveyard :: Widget: Android, defect)
Core Graveyard
Widget: Android
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: mounir, Unassigned)
Details
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
With WebSMS (bug 674725), I'm going to introduce new Java to C++ calls and I would like them to be in GeckoSmsManager instead of GeckoAppShell (no need to make GeckoAppShell grow insanely). Likely, it might be useful for other calls. I did extract this patch from my patch queue because any new GET_FUNC or SHELL_WRAPPER call breaks the entire patch and re-basing it is quite error-prone.
Attachment #554843 -
Flags: review?(mwu)
Comment 1•13 years ago
|
||
the pattern has been to have GeckoAppShell proxy calls to other classes, so the implementation can be in GeckoSmsManager, but it would be called by way of GeckoAppShell
Reporter | ||
Comment 2•13 years ago
|
||
(In reply to Brad Lassey [:blassey](On vacation 8/22-9/20) from comment #1) > the pattern has been to have GeckoAppShell proxy calls to other classes, so > the implementation can be in GeckoSmsManager, but it would be called by way > of GeckoAppShell With the WebAPI work, this might proxy dozen(s) of calls. Is that really what we want?
Reporter | ||
Comment 4•13 years ago
|
||
It's no longer in my patch queue, let's mark this WONTFIX...
No longer blocks: websms
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
Whiteboard: [needs review]
Reporter | ||
Updated•13 years ago
|
Attachment #554843 -
Flags: review?(mwu)
Updated•3 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•