Closed Bug 532941 Opened 16 years ago Closed 16 years ago

[Collector Extension] Use nsIXULAppInfo.ID instead of name

Categories

(addons.mozilla.org Graveyard :: Collector Extension, defect, P1)

Product:
addons.mozilla.org Graveyard
Component:
Collector Extension
Platform:
x86
All
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
BW-1.1

People

(Reporter: kinger, Assigned: kinger)

References

Details

From bug 517291: Bug 517291#15 Bloody mess… Previosly I use "case "Seamonkey": {", and it work fine. Now I see "nothing working", and, when I trace with venkman, look into, and see that appname="SeaMonkey". Wtf? It remember me to my previous patch for weave. See https://bugzilla.mozilla.org/show_bug.cgi?id=526521#c17, last paragraph. Edward Lee wrote: "We've run into issues before when switching on the AppInfo.name instead of AppInfo.ID". Maybe, bandwagon also affected by "issues"? Bug 517291#16 nsIXULAppInfo has returned "SeaMonkey" (capital M) as the appname since SeaMonkey 1.0. But as you say now that Bandwagon supports Fennec which at some point will start identifying itself as "Firefox", the bandwagon developers (like the weave developers) will have to bite the bullet and switch to using appId from appname.
No longer blocks: 517291
Status: NEW → ASSIGNED
Summary: [Collector Extension] Use nsIXULAppInfo.appID instead of appName → [Collector Extension] Use nsIXULAppInfo.ID instead of name
Fixed. http://viewvc.svn.mozilla.org/vc?revision=57715&view=revision The add-on can now better distinguish between Firefox desktop and mobile.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Component: Collections → Collector Extension
QA Contact: collections → collector-extension
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.