525306 - MOZ_APP_DISPLAYNAME should be Firefox when official branding is enabled

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[:Gavin Sharp [email: [e-mail]]]

This in turn requires changing a couple of places that use it when they really want to always be Fennec:

1) UA string (hardcoded)
2) application.ini (hardcoded)
3) desktop.in - GenericName isn't needed, X-Osso-Service can use MOZ_APP_NAME which is always "fennec"
4) fennec.service.in to match 4)
5) prefs.js, for the on-disk cache location (via app/Makefile.in's -DAPP_NAME) - note that this changes it from "Fennec" to "fennec", I think that's OK?

The other users of MOZ_APP_DISPLAYNAME are:
1) Mac packaging stuff in app/Makefile.in (e.g. app bundle name)
2) desktop.in Name and Comment
3) cab installer shortcut
4) updater.ini strings

I contend that they are all OK being branding-dependent.

Comment 1

[:Gavin Sharp [email: [e-mail]]]

Attachment: patch

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to comment #0)
> 3) desktop.in - GenericName isn't needed, X-Osso-Service can use MOZ_APP_NAME
> which is always "fennec"
> 4) fennec.service.in to match 4)

agreed

> 5) prefs.js, for the on-disk cache location (via app/Makefile.in's -DAPP_NAME)
> - note that this changes it from "Fennec" to "fennec", I think that's OK?

should be fine

> The other users of MOZ_APP_DISPLAYNAME are:
> 1) Mac packaging stuff in app/Makefile.in (e.g. app bundle name)
> 2) desktop.in Name and Comment
> 3) cab installer shortcut
> 4) updater.ini strings
> 
> I contend that they are all OK being branding-dependent.

agreed

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 409164 [details] [diff] [review]
patch


> #filter substitution
> [D-BUS Service]
>-Name=Mozilla.@MOZ_APP_DISPLAYNAME@
>+Name=Mozilla.@MOZ_APP_NAME@
 
Name=mozilla.@MOZ_APP_NAME@

Comment 4

[:Gavin Sharp [email: [e-mail]]]

Attachment: comment addressed

Comment 5

[:Gavin Sharp [email: [e-mail]]]

https://hg.mozilla.org/mobile-browser/rev/7fabd9fbc27c

Comment 6

[Aakash Desai [:aakashd]]

verified FIXED on build:

Mozilla/5.0 (X11; U; Linux armv7l; Nokia N900; en-US; rv:1.9.2b2pre) Gecko/20091102 Namoroka/3.6b2pre Fennec/1.0b5