Review Request 109695: JJ#241066: Added a prepareToQuit() signal to amarokWindowScript

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109695/
-----------------------------------------------------------

(Updated March 28, 2013, 8:48 p.m.)


Status
------

This change has been marked as submitted.


Review request for Amarok.


Description
-------

Bug 241066 - JJ: Add Signals trackStop and amarokShutdown to Amarok scripting interface

Changes:
1. Added a prepareToQuit() signal to amarokWindowScript
2. Replaced kapp macro calls with App::instance() because quit() is not virtual

Note:
Signal trackFinished()[trackStop] already exists, so only added prepareToQuit()[amarokShutdown]


Diffs
-----

  src/scriptengine/AmarokScript.cpp 922e71d 
  src/dbus/mpris2/MediaPlayer2.cpp f86ccb3 
  src/scriptengine/AmarokWindowScript.h 5407579 
  src/dbus/mpris1/RootHandler.cpp e60eb1b 
  src/MainWindow.cpp 66f4f76 
  ChangeLog 284f188 
  src/App.cpp fdb4255 
  src/scriptengine/AmarokWindowScript.cpp 897e2da 

Diff: http://git.reviewboard.kde.org/r/109695/diff/


Testing
-------

Tested the new signal in a script


Thanks,

Anmol Ahuja

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20130328/0d54ea59/attachment.html>