Unity groups separate Java webstart app windows
Bug #1098186 reported by
Timo Jyrinki
This bug affects 2 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
BAMF |
Fix Released
|
Medium
|
Marco Trevisan (Treviño) | ||
0.2 |
Fix Committed
|
Medium
|
Marco Trevisan (Treviño) | ||
0.3 |
Fix Committed
|
Medium
|
Marco Trevisan (Treviño) | ||
bamf (Ubuntu) |
Fix Released
|
Medium
|
Unassigned | ||
Precise |
Fix Released
|
Medium
|
Timo Jyrinki | ||
Quantal |
Fix Released
|
Medium
|
Łukasz Zemczak |
Bug Description
[Impact]
Applications launched via Java Webstart are grouped under the same icon on the unity launcher. Unable to activate individual javawebstart apps via launcher button.
[Test Case]
Download different jnlp's (for example from http://
Automated tests are included into the diff.
[Regression Potential]
Windows of applications with no .desktop files could be matched as different applications if their execution path differs.
Gnome Control Center and LibreOffice matching codepath has been slightly changed but is now potentially improved and more solid (both unit-tested).
Related branches
lp://staging/~3v1n0/bamf/javaws-matching-0.2
- Brandon Schaefer (community): Approve
-
Diff: 1011 lines (+448/-211)9 files modifiedsrc/bamf-legacy-screen.c (+8/-27)
src/bamf-legacy-window.c (+2/-2)
src/bamf-matcher-private.h (+1/-0)
src/bamf-matcher.c (+193/-182)
tests/bamfdaemon/Makefile.am (+1/-0)
tests/bamfdaemon/data/gnome-control-center.desktop (+9/-0)
tests/bamfdaemon/data/gnome-display-panel.desktop (+10/-0)
tests/bamfdaemon/data/gnome-mouse-panel.desktop (+10/-0)
tests/bamfdaemon/test-matcher.c (+214/-0)
lp://staging/~3v1n0/bamf/javaws-matching
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
- Brandon Schaefer (community): Approve
-
Diff: 332 lines (+140/-45)3 files modifiedsrc/bamf-legacy-window.c (+25/-41)
src/bamf-matcher.c (+34/-4)
tests/bamfdaemon/test-matcher.c (+81/-0)
lp://staging/~3v1n0/bamf/javaws-matching-0.3
- Brandon Schaefer (community): Approve
-
Diff: 1011 lines (+448/-211)9 files modifiedsrc/bamf-legacy-screen.c (+8/-27)
src/bamf-legacy-window.c (+2/-2)
src/bamf-matcher-private.h (+1/-0)
src/bamf-matcher.c (+193/-182)
tests/bamfdaemon/Makefile.am (+1/-0)
tests/bamfdaemon/data/gnome-control-center.desktop (+9/-0)
tests/bamfdaemon/data/gnome-display-panel.desktop (+10/-0)
tests/bamfdaemon/data/gnome-mouse-panel.desktop (+10/-0)
tests/bamfdaemon/test-matcher.c (+214/-0)
lp://staging/~3v1n0/bamf/javaws-ignore-desktop
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
- Brandon Schaefer (community): Approve
-
Diff: 558 lines (+252/-70)9 files modifiedsrc/bamf-legacy-window-test.c (+21/-0)
src/bamf-legacy-window-test.h (+1/-0)
src/bamf-legacy-window.c (+47/-0)
src/bamf-legacy-window.h (+4/-0)
src/bamf-matcher-private.h (+1/-0)
src/bamf-matcher.c (+49/-55)
tests/bamfdaemon/data/icedtea-netx-javaws.desktop (+12/-0)
tests/bamfdaemon/test-matcher.c (+107/-3)
tests/run-xvfb.sh (+10/-12)
lp://staging/~3v1n0/bamf/javaws-ignore-desktop-0.2
- Brandon Schaefer (community): Approve
-
Diff: 473 lines (+241/-21)8 files modifiedsrc/bamf-legacy-window-test.c (+21/-0)
src/bamf-legacy-window-test.h (+1/-0)
src/bamf-legacy-window.c (+47/-0)
src/bamf-legacy-window.h (+4/-0)
src/bamf-matcher-private.h (+1/-0)
src/bamf-matcher.c (+48/-18)
tests/bamfdaemon/data/icedtea-netx-javaws.desktop (+12/-0)
tests/bamfdaemon/test-matcher.c (+107/-3)
lp://staging/~3v1n0/bamf/javaws-ignore-desktop-0.3
- Brandon Schaefer (community): Approve
-
Diff: 477 lines (+243/-22)8 files modifiedsrc/bamf-legacy-window-test.c (+21/-0)
src/bamf-legacy-window-test.h (+1/-0)
src/bamf-legacy-window.c (+47/-0)
src/bamf-legacy-window.h (+4/-0)
src/bamf-matcher-private.h (+1/-0)
src/bamf-matcher.c (+50/-19)
tests/bamfdaemon/data/icedtea-netx-javaws.desktop (+12/-0)
tests/bamfdaemon/test-matcher.c (+107/-3)
Changed in bamf: | |
status: | New → Fix Committed |
Changed in bamf: | |
milestone: | none → 0.3.6 |
milestone: | 0.3.6 → 0.4.0 |
Changed in bamf (Ubuntu): | |
status: | New → In Progress |
status: | In Progress → Triaged |
Changed in bamf: | |
importance: | Undecided → Medium |
Changed in bamf: | |
assignee: | nobody → Marco Trevisan (Treviño) (3v1n0) |
Changed in bamf (Ubuntu): | |
importance: | Undecided → Medium |
description: | updated |
description: | updated |
Changed in bamf (Ubuntu): | |
status: | Triaged → Fix Released |
Changed in bamf (Ubuntu Quantal): | |
status: | New → In Progress |
Changed in bamf (Ubuntu Precise): | |
status: | New → In Progress |
assignee: | nobody → Timo Jyrinki (timo-jyrinki) |
Changed in bamf (Ubuntu Quantal): | |
assignee: | nobody → Łukasz Zemczak (sil2100) |
Changed in bamf (Ubuntu Precise): | |
importance: | Undecided → Medium |
Changed in bamf (Ubuntu Quantal): | |
importance: | Undecided → Medium |
Changed in bamf: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Oops... I've just noticed that the fix committed does not cover all the cases. I've just pushed a fix that will cover that too.