Updated the fix-trayicon patch to include all the 4 commits by Kevin Funk available from http://krf.kollide.net/files/work/amarok/

Package-Manager: portage-2.2_rc67/cvs/Linux x86_64

4 files changed, 253 insertions, 74 deletions

diff --git a/media-sound/amarok/ChangeLog b/media-sound/amarok/ChangeLog
index 72227800c162..5e3feae60b86 100644
--- a/media-sound/amarok/ChangeLog
+++ b/media-sound/amarok/ChangeLog
@@ -1,6 +1,11 @@
 # ChangeLog for media-sound/amarok
 # Copyright 1999-2010 Gentoo Foundation; Distributed under the GPL v2
-# $Header: /var/cvsroot/gentoo-x86/media-sound/amarok/ChangeLog,v 1.382 2010/07/11 01:54:43 jmbsvicetto Exp $
+# $Header: /var/cvsroot/gentoo-x86/media-sound/amarok/ChangeLog,v 1.383 2010/07/11 16:00:08 jmbsvicetto Exp $
+
+  11 Jul 2010; Jorge Manuel B. S. Vicetto <jmbsvicetto@gentoo.org>
+  amarok-2.3.1-r2.ebuild, files/amarok-2.3.1-fix-trayicon.patch:
+  Updated the fix-trayicon patch to include all the 4 commits by Kevin Funk
+  available from http://krf.kollide.net/files/work/amarok/
 
 *amarok-2.3.1-r2 (11 Jul 2010)
 
diff --git a/media-sound/amarok/Manifest b/media-sound/amarok/Manifest
index 7f6b74d8aec5..cdaf802e7e25 100644
--- a/media-sound/amarok/Manifest
+++ b/media-sound/amarok/Manifest
@@ -1,14 +1,14 @@
 AUX amarok-2.2-mysql-libs.patch 540 RMD160 16b143a50c126ba69851e4c1d1524b1d6b7c175d SHA1 0358c91184604bf4587cfa755202a41ba9c4d346 SHA256 ace02a3f8d20295077de08d17af00382c19ad21ddf4ef324ff7c5e5ec4d666e1
 AUX amarok-2.2.2-nodebug.patch 16478 RMD160 8f8eedbd5a29f10da6243cc4ac586f59ca493a7d SHA1 c2316800d0632b473b14d683415f51a36879922a SHA256 022a655eff5cd918fc61999001a3251de7e406cbffb40127295820eb4e398a29
 AUX amarok-2.3.0-crash_fix_info_applet.patch 1820 RMD160 2ddbd8e30d7c9031eead648430bc8c9e2e6ec0cd SHA1 5aa6fd5ae30c956ea4f4be3d5bb6078af9e88ea3 SHA256 2f48faa43ad621616289b30da5b2febcd2861bc500a73bf9112916d1eeb3b41c
-AUX amarok-2.3.1-fix-trayicon.patch 7914 RMD160 0df9c611db33d11a4567f9ad4b5b1f3f8e449862 SHA1 c0c6a670b4fec60546fec80bd049306bb9139433 SHA256 8c52be2670fc3d60006eb8638e8cbd7ea52e623efc9f1a642629d1c3762eea47
+AUX amarok-2.3.1-fix-trayicon.patch 13469 RMD160 36f5ae424bdd6d865bfce203e7d74f08966ad379 SHA1 f0110088ecd826b652d558c7463a0a717ed5080b SHA256 4adb543ab25a86f85ea7b914e66374faeca0df8e02aa40682cee17d137a9b76a
 AUX amarok-fix-accessibility-dep.patch 687 RMD160 184a28eecb351117c8ce3c922112e7ac5c4efde1 SHA1 e9ea735d7431fb3c11f5aabab0935c637826059c SHA256 8d631d10dccce5a72eb24a366afc5672d7853fba8a5c667e300549d8ee85170d
 AUX amarok-qtscriptcheck.patch 1024 RMD160 0693ddd6bbaa2f15d381eec329cfeff1e8b176cd SHA1 0585c08f8159a06c95e89f22e788d6e039224fbb SHA256 8a2983f3bd5ec422f2ed0a90a49970fd13fed3852c66ed220c01b4d8418ce89d
 DIST amarok-2.2.2.tar.bz2 8788107 RMD160 0782c4b032d318179bd8bb91353f8e33772d2912 SHA1 2075299b856477cf55cba8ba983ea25a93dc139b SHA256 768d308572897a3100f0d87999f1bb1c68b2e2fa3b0961fd0b918bcfd33ae64d
 DIST amarok-2.3.1.tar.bz2 9823411 RMD160 70e581bd681361d7837f1ed295191174e92e9f8f SHA1 00a596384df8d155b91b40bb2484ec0a7f51da80 SHA256 3fef3f4d4b4ffa22faf4a3b97e2c2bcfde85f55a9973b1ad6d66df55068b4a72
 EBUILD amarok-2.2.2-r1.ebuild 3129 RMD160 7f2f764b4542d2447b7e1109c757a45ea8f0f403 SHA1 24f628aaf70a20f4160fa135df7d07e4f83f724f SHA256 fda21a42223a92bf47f68e2b2824b332017c480c3164389b76b1f8c5106459d1
 EBUILD amarok-2.3.1-r1.ebuild 4128 RMD160 ed941d9b1cdce3e9f900c0d39b21823740c26872 SHA1 27af017934c245dac5d84ff3b00b6d8b22303d1e SHA256 c3434e3d19d7679b2b8cb608de331f79fc05c0118f01dc4708d44051c2e8d9af
-EBUILD amarok-2.3.1-r2.ebuild 4531 RMD160 cb56b2ad770859e2902edf0cbb685a8adea181ea SHA1 e4226a6feae889b74ea3a06fb33877a1362ab97e SHA256 d2b0fb91f63284462601431877cca240b2846cb9061d590027147cad630726c1
+EBUILD amarok-2.3.1-r2.ebuild 4920 RMD160 845942c28289a4534930ce39dece3f705b3a4e59 SHA1 3b38105a70d19e99daee689b7206cd8d74cc0db2 SHA256 7e4e235990bed3d7656ea4afe7a6e7093e121ebce45699f29cd824b47e47a797
 EBUILD amarok-2.3.1.ebuild 3465 RMD160 7834bed748ffc58ae9573eaad9f779de67332598 SHA1 ce11a10148980406634e9b494c6eeb530c647f7e SHA256 86b44855a9c8ea722406a13ef861350d18ba8e19ff7c11b202e9a3db9557a632
-MISC ChangeLog 64233 RMD160 b9c3b682e481d1ecef947a24a5757b6015417ec4 SHA1 dea61cf1741ea5fd07ebf3a6250302d6f6758318 SHA256 fd476fd45eacd61bdb4129cb6c4df41b6d5674061645eba463d90b7a2e74b0de
+MISC ChangeLog 64501 RMD160 f5f7450f02f4b81b871945bb956cb6aa1541c18a SHA1 680cf65b5068f1e421d305e68ef153167b4ddd1b SHA256 bd94159a9c274d772b4bd0d0f081974a926949f4afa3ac2ddc4c47909615372b
 MISC metadata.xml 1413 RMD160 62fc2ae4eae51f52f3ef02d86e8b244a3132ba86 SHA1 dda8b8bf344947925b93744cff693605f46c1006 SHA256 be46bcd7f470f66bb3afc8df3338aab204e055e10f042d4bae2be65eae7b3f8f
diff --git a/media-sound/amarok/amarok-2.3.1-r2.ebuild b/media-sound/amarok/amarok-2.3.1-r2.ebuild
index 5e818787729e..251f36e76a4d 100644
--- a/media-sound/amarok/amarok-2.3.1-r2.ebuild
+++ b/media-sound/amarok/amarok-2.3.1-r2.ebuild
@@ -1,6 +1,6 @@
 # Copyright 1999-2010 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/media-sound/amarok/amarok-2.3.1-r2.ebuild,v 1.1 2010/07/11 01:54:44 jmbsvicetto Exp $
+# $Header: /var/cvsroot/gentoo-x86/media-sound/amarok/amarok-2.3.1-r2.ebuild,v 1.2 2010/07/11 16:00:08 jmbsvicetto Exp $
 
 EAPI="2"
 
@@ -75,9 +75,14 @@ RDEPEND="${COMMONDEPEND}
 	player? ( >=kde-base/phonon-kde-${KDE_MINIMAL} )
 "
 
-# The fix trayicon patch is amarok.git-26104cd35fd50222c354f3afc9fce6bba093c05f.patch
-# from https://bugs.kde.org/show_bug.cgi?id=232578#c13 and
-# http://gitweb.kde.org/amarok/amarok.git/commitdiff/26104cd35fd50222c354f3afc9fce6bba093c05f
+# The fix trayicon patch was assembled from the 4 patches committed by Kevin Funk to fix
+# upstream bug at https://bugs.kde.org/show_bug.cgi?id=232578#c13 and available from
+# http://krf.kollide.net/files/work/amarok/
+# They correspond to the following 4 git commits:
+# http://gitweb.kde.org/amarok/amarok.git/commit/e959e75a8f028eb36406d65118885c32e3eff3c8
+# http://gitweb.kde.org/amarok/amarok.git/commit/26104cd35fd50222c354f3afc9fce6bba093c05f
+# http://gitweb.kde.org/amarok/amarok.git/commit/4995f14cefbbe78e9dd3c42af00188e6c82e6f94
+# http://gitweb.kde.org/amarok/amarok.git/commit/74ea4c1f9e69952ac274be44ab37ed073e61c1e6
 PATCHES=( "${FILESDIR}/${PN}-fix-accessibility-dep.patch" "${FILESDIR}/${P}-fix-trayicon.patch")
 
 src_prepare() {
diff --git a/media-sound/amarok/files/amarok-2.3.1-fix-trayicon.patch b/media-sound/amarok/files/amarok-2.3.1-fix-trayicon.patch
index c971ef48a4c0..db423c9ec82f 100644
--- a/media-sound/amarok/files/amarok-2.3.1-fix-trayicon.patch
+++ b/media-sound/amarok/files/amarok-2.3.1-fix-trayicon.patch
@@ -1,29 +1,33 @@
-From 26104cd35fd50222c354f3afc9fce6bba093c05f Mon Sep 17 00:00:00 2001
-From: Kevin Funk <krf@electrostorm.net>
-Date: Thu, 8 Jul 2010 12:13:00 +0200
-Subject: [PATCH] Some TrayIcon changes:
- * Remove track progress effect (this is because KSNI has some bogus
- implementation of the setIconByPixmap() function causing the overlay
- icon being wrong sized
- * Fix overlay icon size
- * Tooltip album cover is now updated if changed in Amarok
- * Cleanup
- CCBUG: 231539
- CCBUG: 232578
- CCBUG: 232312
- BUG: 233506
- BUG: 240463
-
----
- src/TrayIcon.cpp |  106 ++++++++++++------------------------------------------
- src/TrayIcon.h   |   11 +-----
- 2 files changed, 25 insertions(+), 92 deletions(-)
-
-diff --git a/src/TrayIcon.cpp b/src/TrayIcon.cpp
-index fb97483..4bd46e4 100644
---- a/src/TrayIcon.cpp
-+++ b/src/TrayIcon.cpp
-@@ -52,7 +52,6 @@
+diff -urN amarok-2.3.1/src/TrayIcon.cpp amarok-2.3.1-new//src/TrayIcon.cpp
+--- amarok-2.3.1/src/TrayIcon.cpp	2010-05-27 18:37:20.000000000 +0000
++++ amarok-2.3.1-new//src/TrayIcon.cpp	2010-07-11 15:21:36.000000000 +0000
+@@ -26,33 +26,24 @@
+ #include "EngineController.h"
+ #include "amarokconfig.h"
+ #include "GlobalCurrentTrackActions.h"
+-#include "core/meta/support/MetaConstants.h"
+ #include "core/capabilities/CurrentTrackActionsCapability.h"
+ #include "playlist/PlaylistActions.h"
+-#include "playlist/PlaylistModelStack.h"
+ #include "SvgHandler.h"
+ #include <KAboutData>
+ #include <KAction>
+ #include <KCmdLineArgs>
+-#include <KIcon>
+-#include <KIconEffect>
+ #include <KLocale>
+ #include <KMenu>
+ #include <KStandardDirs>
+ 
+ #include <QAction>
+-#include <QEvent>
+ #include <QFontMetrics>
+-#include <QMouseEvent>
+-#include <QPainter>
+ #include <QPixmap>
+-#include <QTime>
+ #include <QToolTip>
+ 
  Amarok::TrayIcon::TrayIcon( QObject *parent )
          : KStatusNotifierItem( parent )
          , Engine::EngineObserver( The::engineController() )
@@ -31,26 +35,54 @@ index fb97483..4bd46e4 100644
          , m_separator( 0 )
  {
      DEBUG_BLOCK
-@@ -80,7 +79,7 @@ Amarok::TrayIcon::TrayIcon( QObject *parent )
+@@ -64,13 +55,13 @@
+ 
+     PERF_LOG( "Before adding actions" );
+ 
+-    #ifdef Q_WS_MAC
++#ifdef Q_WS_MAC
+     // Add these functions to the dock icon menu in OS X
+     extern void qt_mac_set_dock_menu(QMenu *);
+     qt_mac_set_dock_menu( contextMenu() );
+     contextMenu()->addAction( ac->action( "playlist_playmedia" ) );
+     contextMenu()->addSeparator();
+-    #endif
++#endif
+ 
+     contextMenu()->addAction( ac->action( "prev"       ) );
+     contextMenu()->addAction( ac->action( "play_pause" ) );
+@@ -79,105 +70,87 @@
+     contextMenu()->setObjectName( "TrayIconContextMenu" );
  
      PERF_LOG( "Adding system tray icon" );
-+    setIconByName( "amarok" );
 -    paintIcon();
  
-     setupToolTip();
+-    setupToolTip();
++    setIconByName( "amarok" );
++
++    setupToolTip( true );
  
      connect( this, SIGNAL( scrollRequested( int, Qt::Orientation ) ), SLOT( slotScrollRequested(int, Qt::Orientation) ) );
-@@ -102,22 +95,13 @@ Amarok::TrayIcon::setupToolTip()
+     connect( this, SIGNAL( secondaryActivateRequested( const QPoint & ) ), SLOT( slotActivated() ) );
+ }
+ 
+ void
+-Amarok::TrayIcon::setupToolTip()
++Amarok::TrayIcon::setupToolTip( bool updateIcon )
+ {
+     if( m_track )
+     {
          setToolTipTitle( The::engineController()->prettyNowPlaying() );
  
-         QStringList tooltip;
+-        QStringList tooltip;
 -        // TODO: Use Observer to get notified about changed album art
 -        if( m_track->album() )
-+        if( m_track->album() && m_track->album()->hasImage() )
++        // check if we really need to update the icon (performance tweak)
++        if( updateIcon )
          {
-             const QString uid = m_track->uidUrl();
-             if ( uid != m_toolTipIconUid ) {
-                 const QPixmap image = The::svgHandler()->imageWithBorder( m_track->album(), KIconLoader::SizeLarge, 5 );
+-            const QString uid = m_track->uidUrl();
+-            if ( uid != m_toolTipIconUid ) {
+-                const QPixmap image = The::svgHandler()->imageWithBorder( m_track->album(), KIconLoader::SizeLarge, 5 );
 -                if ( image.isNull() )
 -                {
 -                    setToolTipIconByName( "amarok" );
@@ -61,24 +93,99 @@ index fb97483..4bd46e4 100644
 -                    setToolTipIconByPixmap( image );
 -                    m_toolTipIconUid = uid;
 -                }
++            if( m_track->album() && m_track->album()->hasImage() )
++            {
++                QPixmap image = The::svgHandler()->imageWithBorder( m_track->album(), KIconLoader::SizeLarge, 5 );
 +                setToolTipIconByPixmap( image );
-+                m_toolTipIconUid = uid;
++            }
++            else
++            {
++                setToolTipIconByName( "amarok" );
              }
+-        }
+-        else
+-        {
+-            setToolTipIconByName( "amarok" );
+-            m_toolTipIconUid.clear();
          }
+ 
+-        QStringList left, right;
++        QStringList tooltip;
+ 
+-        // TODO: Replace block by some other useful information
+         QString volume;
+         if ( The::engineController()->isMuted() )
++        {
+             volume = i18n( "Muted" );
++        }
          else
-@@ -184,9 +168,10 @@ Amarok::TrayIcon::setupToolTip()
++        {
+             volume = QString( "%1%" ).arg( The::engineController()->volume() );
+-        right << QString("<i>%1</i>").arg( volume );
+-        left << QString( "<i>%1</i>" ).arg( i18n( "Volume" ) );
++        }
++        tooltip << QString("<i>%1: %2</i>").arg( i18n( "Volume" ) ).arg( volume );
+ 
+         const float score = m_track->score();
+         if( score > 0.f )
+         {
+-            right << QString::number( score, 'f', 2 );  // 2 digits after decimal point
+-            left << i18n( "Score" );
++            tooltip << QString( "%1: %2" ).arg( i18n( "Score" ) ).arg( QString::number( score, 'f', 2 ) );
+         }
+ 
+         const int rating = m_track->rating();
+         if( rating > 0 )
+         {
+-            QString s;
++            QString stars;
+             for( int i = 0; i < rating / 2; ++i )
+-                s += QString( "<img src=\"%1\" height=\"%2\" width=\"%3\">" )
++                stars += QString( "<img src=\"%1\" height=\"%2\" width=\"%3\">" )
+                         .arg( KStandardDirs::locate( "data", "amarok/images/star.png" ) )
+                         .arg( QFontMetrics( QToolTip::font() ).height() )
+                         .arg( QFontMetrics( QToolTip::font() ).height() );
+             if( rating % 2 )
+-                s += QString( "<img src=\"%1\" height=\"%2\" width=\"%3\">" )
++                stars += QString( "<img src=\"%1\" height=\"%2\" width=\"%3\">" )
+                         .arg( KStandardDirs::locate( "data", "amarok/images/smallstar.png" ) )
+                         .arg( QFontMetrics( QToolTip::font() ).height() )
+                         .arg( QFontMetrics( QToolTip::font() ).height() );
+-            right << s;
+-            left << i18n( "Rating" );
++
++            tooltip << QString( "%1: %2" ).arg( i18n( "Rating" ) ).arg( stars );
+         }
+ 
+         const int count = m_track->playCount();
+         if( count > 0 )
+         {
+-            right << QString::number( count );
+-            left << i18n( "Play Count" );
++            tooltip << QString( "%1: %2" ).arg( i18n( "Play Count" ) ).arg( QString::number( count ) );
+         }
+ 
+         const uint lastPlayed = m_track->lastPlayed();
+-        right << Amarok::verboseTimeSince( lastPlayed );
+-        left << i18n( "Last Played" );
+-
+-        // NOTE: It seems to be necessary to <center> each element indivdually
+-        const QString row = "- %1: %2";
+-        for( int x = 0; x < left.count(); ++x )
+-            if ( !right[x].isEmpty() )
+-                tooltip << row.arg( left[x] ).arg( right[x] );
++        tooltip << QString( "%1: %2" ).arg( i18n( "Last played" ) ).arg( Amarok::verboseTimeSince( lastPlayed ) );
+ 
+         setToolTipSubTitle( tooltip.join("<br>") );
+     }
      else
      {
          setToolTipIconByName( "amarok" );
 -        m_toolTipIconUid.clear();
          setToolTipTitle( KCmdLineArgs::aboutData()->programName() );
          setToolTipSubTitle( The::engineController()->prettyNowPlaying() );
-+
-+        m_toolTipIconUid.clear();
      }
- }
- 
-@@ -206,20 +191,26 @@ Amarok::TrayIcon::engineStateChanged( Phonon::State state, Phonon::State /*oldSt
+@@ -199,20 +172,26 @@
      switch( state )
      {
          case Phonon::PlayingState:
@@ -111,7 +218,7 @@ index fb97483..4bd46e4 100644
              setupMenu(); // remove custom track actions on stop
              break;
  
-@@ -230,6 +221,7 @@ Amarok::TrayIcon::engineStateChanged( Phonon::State state, Phonon::State /*oldSt
+@@ -223,21 +202,19 @@
          case Phonon::LoadingState:
          case Phonon::ErrorState:
          case Phonon::BufferingState:
@@ -119,7 +226,11 @@ index fb97483..4bd46e4 100644
              break;
      }
  
-@@ -240,9 +232,6 @@ void
+-    setupToolTip();
++    setupToolTip( true );
+ }
+ 
+ void
  Amarok::TrayIcon::engineNewTrackPlaying()
  {
      m_track = The::engineController()->currentTrack();
@@ -127,9 +238,18 @@ index fb97483..4bd46e4 100644
 -
 -    paintIcon( 0 );
  
-     setupToolTip();
+-    setupToolTip();
++    setupToolTip( true );
+     setupMenu();
+ }
+ 
+@@ -246,17 +223,17 @@
+ {
+     Q_UNUSED( track )
+ 
+-    setupToolTip();
++    setupToolTip( false );
      setupMenu();
-@@ -258,12 +247,12 @@ Amarok::TrayIcon::metadataChanged( Meta::TrackPtr track )
  }
  
  void
@@ -141,15 +261,28 @@ index fb97483..4bd46e4 100644
  
 -    if( m_trackLength )
 -        paintIcon( position );
-+    setupToolTip();
++    setupToolTip( true );
 +    setupMenu();
  }
  
  void
-@@ -283,71 +272,6 @@ Amarok::TrayIcon::engineMuteStateChanged( bool mute )
+@@ -264,7 +241,7 @@
+ {
+     Q_UNUSED( percent );
+ 
+-    setupToolTip();
++    setupToolTip( false );
  }
  
  void
+@@ -272,72 +249,7 @@
+ {
+     Q_UNUSED( mute );
+ 
+-    setupToolTip();
+-}
+-
+-void
 -Amarok::TrayIcon::paletteChange( const QPalette & op )
 -{
 -    Q_UNUSED( op );
@@ -212,17 +345,53 @@ index fb97483..4bd46e4 100644
 -
 -    setIconByPixmap( m_icon );
 -    setOverlayIconByName( "media-playback-start" );
--}
++    setupToolTip( false );
+ }
+ 
+ void
+diff -urN amarok-2.3.1/src/TrayIcon.h amarok-2.3.1-new//src/TrayIcon.h
+--- amarok-2.3.1/src/TrayIcon.h	2010-05-27 18:37:20.000000000 +0000
++++ amarok-2.3.1-new//src/TrayIcon.h	2010-07-11 15:21:36.000000000 +0000
+@@ -1,7 +1,7 @@
+ /****************************************************************************************
+  * Copyright (c) 2003 Stanislav Karchebny <berkus@users.sf.net>                         *
+- * Copyright (c) 2009 Kevin Funk <krf@electrostorm.net>                                 *
+  * Copyright (c) 2009 Mark Kretschmann <kretschmann@kde.org>                            *
++ * Copyright (c) 2009,2010 Kevin Funk <krf@electrostorm.net>                            *
+  *                                                                                      *
+  * This program is free software; you can redistribute it and/or modify it under        *
+  * the terms of the GNU General Public License as published by the Free Software        *
+@@ -19,18 +19,15 @@
+ #ifndef AMAROK_TRAYICON_H
+ #define AMAROK_TRAYICON_H
+ 
+-#include "core/engine/EngineObserver.h" //baseclass
++#include "core/engine/EngineObserver.h" // baseclass
+ #include "core/meta/Meta.h"
+ #include "core/support/SmartPointerList.h"
+ 
+-#include <KStatusNotifierItem> //baseclass
++#include <KStatusNotifierItem> // baseclass
+ 
+ #include <QAction>
+ #include <QPointer>
+ 
+-class QEvent;
+-class App;
 -
--void
- Amarok::TrayIcon::setupMenu()
- {
-     foreach( QAction* action, m_extraActions )
-diff --git a/src/TrayIcon.h b/src/TrayIcon.h
-index c2602df..d590b5f 100644
---- a/src/TrayIcon.h
-+++ b/src/TrayIcon.h
-@@ -49,16 +49,13 @@ protected:
+ 
+ namespace Amarok {
+ 
+@@ -40,24 +37,20 @@
+ 
+ public:
+     TrayIcon( QObject *parent );
+-    friend class ::App;
+-    
++
+     void setVisible( bool visible );
+ 
+ protected:
      // reimplemented from engineobserver
      virtual void engineStateChanged( Phonon::State state, Phonon::State oldState = Phonon::StoppedState );
      virtual void engineNewTrackPlaying();
@@ -241,20 +410,20 @@ index c2602df..d590b5f 100644
  
  private slots:
      void slotActivated();
-@@ -68,13 +65,9 @@ private:
-     void setupMenu();
-     void setupToolTip();
+@@ -65,15 +58,10 @@
  
--    void paintIcon( qint64 trackPosition = -1 );
+ private:
+     void setupMenu();
+-    void setupToolTip();
 -
+-    void paintIcon( qint64 trackPosition = -1 );
++    void setupToolTip( bool updateIcon );
+ 
      Meta::TrackPtr m_track;
 -    qint64 m_trackLength;
-     QString m_toolTipIconUid;
+-    QString m_toolTipIconUid;
  
 -    QPixmap m_baseIcon, m_grayedIcon, m_icon;
      SmartPointerList<QAction> m_extraActions;
      QPointer<QAction> m_separator;
  };
--- 
-1.7.1
-