Closed Bug 1151753 Opened 9 years ago Closed 7 years ago

WM_CLASS should be different between Mozilla Firefox and Firefox Developer Edition, even in private browsing

Categories

(Firefox :: Shell Integration, defect)

All
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 58
Tracking Status
firefox58 --- fixed

People

(Reporter: cm-t, Assigned: m_kato)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

Attached image ubuntu-firefox.png
User Agent: Mozilla/5.0 (Windows NT 5.1; rv:39.0) Gecko/20100101 Firefox/39.0
Build ID: 20150403004008

Steps to reproduce:


 * Running Ubuntu 14.04 LTS (Unity Desktop) fully up to date (including Mozilla Firefox)
 * Installed Firefox Developer Edition using the developer tools "ubuntu-make"[1]
 * Both Mozilla Firefox and Firefox Developer Edition's shortcuts (unity's launcher on sidebar) are pinned
 * Right click on the Mozilla Firefox launcher
 * Select "New Private browsing windows"
 * Launch optionnaly new windows from Mozilla Firefox and Firefox Developer Edition



[1]: ubuntu-make: https://github.com/ubuntu/ubuntu-make



Actual results:

On the Unity's sidebar and ALT+TAB "popup", you will see the Firefox Developer Edition telling that all windows are opened as Firefox Developer Edition. All windows are launched as Firefox Developer Edition (as seen by Ubuntu Unity).


Note: There are more informations and screenshots on this bug report

 * https://github.com/ubuntu/ubuntu-make/issues/78


Expected results:

All those opened windows should be in their own icons (sidebar/alt+tab unity's icons): 

 * Firefox Developer Edition windows under the Firefox Developer Edition icon
 * Mozilla Firefox under the Mozilla Firefox icon
OS: Windows XP → Linux
Hardware: x86 → All
Status: UNCONFIRMED → NEW
Component: Untriaged → Shell Integration
Ever confirmed: true
See Also: → 849571
We should call gdk_set_program_class.  If gdk_prgclass is overriden by --class, it isn't updated according to GTK+ 3.20 code.
Comment on attachment 8922739 [details]
Bug 1151753 - Set program class name when not using --class command line option.

https://reviewboard.mozilla.org/r/193900/#review199862

Thanks for sharing and tidying up the existing code.

::: widget/WidgetUtils.cpp:148
(Diff revision 1)
> +WidgetUtils::GetBrandShortName(nsAString& aBrandName)
> +{
> +    aBrandName.Truncate();
> +
> +    nsCOMPtr<nsIStringBundleService> bundleService =
> +        do_GetService(NS_STRINGBUNDLE_CONTRACTID);

Please use GetStringBundleService() as it caches the result of the service look-up.

::: widget/gtk/nsAppShell.cpp:180
(Diff revision 1)
>          } else {
>              screenManager.SetHelper(mozilla::MakeUnique<ScreenHelperGTK>());
>          }
>      }
>  
> +    if (!gtk_check_version(3, 16, 3)) {

Please compare the result of gtk_check_version() with nullptr, so that it is clear that this function does not return a boolean, as a hint that the result is opposite to what is expected.
Attachment #8922739 - Flags: review?(karlt) → review+
Pushed by m_kato@ga2.so-net.ne.jp:
https://hg.mozilla.org/integration/autoland/rev/cd2a5b5cf752
Set program class name when not using --class command line option. r=karlt
https://hg.mozilla.org/mozilla-central/rev/cd2a5b5cf752
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
Blocks: 1387859
Depends on: 1414881
Assignee: nobody → m_kato
For anyone wanting to take advantage of this, the .desktop filename must match the channel's "brandShortName" which can be found in chrome://branding/locale/brand.properties with current values:

> Nightly: "Nightly"
> Developer Edition: "Firefox Developer Edition"
> Beta: "Firefox"
> Release: "Firefox"

These are mostly obvious but Developer Edition is a bit tricky and Beta cannot be distinguished from Release.
(In reply to Kestrel from comment #7)
> For anyone wanting to take advantage of this, the .desktop filename must
> match the channel's "brandShortName" [...]

What about case and space characters? Does it have to be named "Firefox Developer Edition.desktop" or would "firefox developer edition" or "firefox_developer_edition" work, too?
In my testing, I found that the letter case must be equal or lower (not higher) and you can substitute space with dash (not underscore) but then all must be lower case.

Good:
Firefox Developer Edition.desktop
firefox developer edition.desktop
firefox-developer-edition.desktop

Bad:
FIREFOX developer edition.desktop
Firefox-Developer-Edition.desktop
firefox_developer_edition.desktop
Depends on: 1465814
No longer depends on: 1465814
Regressions: 1465814
You need to log in before you can comment on or make changes to this bug.