Merge lp:~paulliu/unity8/reboot into lp:unity8

[Ying-Chun Liu]

Add reboot/shutdown

* Are there any related MPs required for this MP to build/function as expected?
No.

 * Did you perform an exploratory manual test run of your code change and any related functionality?
Yes.

 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A

 * If you changed the UI, has there been a design review?
No. We need to discuss this in Malta

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:850
http://jenkins.qa.ubuntu.com/job/unity8-ci/3314/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/1539/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/318/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/408/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/408/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/408/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/2522/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/3314/rebuild

review: Needs Fixing (continuous-integration)

[lp:~paulliu/unity8/reboot]

851. By Ying-Chun Liu on 2014-07-04

merged trunk
* debian/control:
  list qtdeclarative5-ubuntu-ui-toolkit-plugin-gles as an alternative
  choice, since provides are not versionned, should restore installability
  on amd64 and i386
[ Michał Sawicz ]
* Adapt to suru theme.
[ Michael Terry ]
* Fix path in launcher mock after moving our mock icons, to avoid a
  lot of "icon not found" warnings during qmluitests.
* Fix the testMultiGreeter qmluitest. Incoming method variables are
  apparently read-only in Qt5.3. (LP: #1332488)
[ CI bot ]
* Resync trunk
[ Michał Sawicz ]
* Adapt scope mock to new api and quiet unused variable warnings.
* Fix dynamic overlay height. (LP: #1334879)
* Don't center items in CardVerticalJournal, kind of beats the
  purpose... Also don't bind unnecessarily.
[ Ying-Chun Liu ]
* Fix LP:1330957 Fix some failed test cases. (LP: #1330957)
[ Albert Astals ]
* Don't seem to need this waitForRendering And makes test fail in 5.3
[ Alberto Aguirre ]
* Update Powerd plugin and Shell.qml to accommodate changes in the
  display power state notification.
[ Ying-Chun Liu ]
* Add logout support. Reviewed by: Daniel d'Andrada (LP: #1302213)
[ Michał Sawicz ]
* Make so that fixedArtShapeSize actually fixes artShapeSize.
[ Albert Astals ]
* Add VerticalJournal integration to Dash/scopes/QML (LP: #1326467)
* Make so that fixedArtShapeSize actually fixes artShapeSize.
[ Mirco Müller ]
* Added the frontend-part of sound-hint support for notifications with
  updated QML-tests.
* New rebuild forced
[ Albert Astals ]
* Departments support (LP: #1320847)
[ Pawel Stolowski ]
* Extend the hack for click scope categories with the upcoming 'store'
  category: single-tap on results from the 'store' category should
  activate them, instead of requesting a preview. (LP: #1326292)
[ Albert Astals ]
* Drop the " Preview" suffix from Preview title As requested in
  https://bugs.launchpad.net/unity8/+bug/1316671 (LP: #1316671)
[ Michael Terry ]
* Revert split greeter for now. We will bring it back as an option
  for Desktop, but use a big hammer revert right now to get Touch back
  in shape.
[ CI bot ]
* Fix build problems. Reviewed by: Michael Terry (LP: #1328850)
[ Michał Sawicz ]
* Make lockscreen buttons translatable.
[ Albert Astals ]
* Correctly mark these functions as overrides
* Remove connections to non existant signal
* Better test name
* Improvements for headerless categories LVPWH: No section name -> no
  header LVPWH: New hasSectionHeader context property for delegates
  GSV: Add topMargin if no hasSectionHeader (LP: #1326415)
* Make tryVerticalJournal, tryHorizontalJournal and tryOrganicGrid
  work again
[ Michael Zanetti ]
* Don't crash when we get an invalid app from ApplicationManager (LP:
  #1309162)
[ Andrea Cimitan ]
* Workaround for lp1324159 (LP: #1322233, #1324159)
[ CI bot ]
* Resync trunk
[ Florian Boucault ]
* Application startup: changed splash rectangle to be black instead of
  white and added a neat little animation. (LP: #1124265)
[ Michael Terry ]
* Bump version so ubuntu-touch-session can reference this one
[ CI bot ]
* Resync trunk
[ Michał Sawicz ]
* Move env setup past session init in greeter wrapper. (LP: #1325882)
* no change rebuild
* drop dbus-x11 dependency of unity8-greeter, it makes us end up with
  multiple session dbus daemons which breaks many AP tests in the lab
[ Michael Terry ]
* Bump version for Breaks due to unity8-greeter
* In split mode, determine whether the application identifiers in
  AccountsService are click packages or not, so we know the correct
  url prefix to use.
* Start logrotate in the greeter's session.
[ Andrea Cimitan ]
* Passes to make tryCommand -qmljsdebugger=port:3768 to enable
  debug/profiling test apps
* Fixes carousel shadow
[ Michał Sawicz ]
* Use dpkg-architecture, not gcc, to determine the machine triplet.
[ Ying-Chun Liu ]
* Fix ZoomableImage test failure. (LP: #1317254)
[ Albert Astals ]
* We don't need iconutils in this mock
* Don't reserve space for mascot if no mascot is specified (LP:
  #1319343)
* CardHeader is no more, remove stale line in CMakeLists.txt
* GenericScopeViewTest: Wait a bit more Otherwise sometimes we end
  getting up the wrong delegate (maybe one that will be garbage
  collected?) (LP: #1322279)
* Fix crash in organicgridtest
[ Daniel d'Andrada ]
* Remove Shell's underlay background image as it cannot be seen
  anymore Now that the Dash has its own, opaque, background, the
  underlay's background image can no longer be seen. So it's just a
  waste of resources to have it.
[ Michael Terry ]
* Use the same animation when dismissing a greeter slide from the
  launcher as from a normal greeter drag. (LP: #1316513)
[ Michael Zanetti ]
* enhance lockscreen add a retry indication label (e.g. 3 attempts
  left). add an additional label (e.g. phone number for multi sim).
  add a infoPopup (e.g. to display a warning for last retry). add min
  and max limit values. add tests for the above (LP: #1302050)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:851
http://jenkins.qa.ubuntu.com/job/unity8-ci/3332/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/1585/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/336/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/426/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/426/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/426/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/2583/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/3332/rebuild

review: Needs Fixing (continuous-integration)

[lp:~paulliu/unity8/reboot]

852. By Ying-Chun Liu on 2014-07-08

fix dialog

853. By Ying-Chun Liu on 2014-07-08

Merge trunk
[ Nick Dedekind ]
* Added support for TransferMenu
[ Michał Sawicz ]
* Initial code for a payment button widget, to handle purchasing apps
  from the click scope.
[ Nick Dedekind ]
* Moved [Message]MenuItemFacotory from Unity.Indicators plugin to qml
  folder.
[ Albert Astals ]
* EasingCurve: Initialize members
* Fix valgrind warning by not emitting reset on model destructor That
  won't be supported until Qt 5.4 More info at https://bugreports.qt-
  project.org/browse/QTBUG-39780 Warning was ==16693== Invalid read of
  size 8 ==16693== at 0x72B19A0: QQmlContext::isValid() const
  (qqmlcontext.cpp:231) ==16693== by 0x736C82B:
  QQmlDelegateModelPrivate::emitChanges() (qqmldelegatemodel.cpp:1412)
  ==16693== by 0x7372AE6: QQmlDelegateModel::_q_modelReset()
  (qqmldelegatemodel.cpp:1463) ==16693== by 0x7397224:
  QQmlDelegateModel::qt_static_metacall(QObject*, QMetaObject::Call,
  int, void**) (moc_qqmldelegatemodel_p.cpp:196) ==16693== by
  0x739769E: QQmlDelegateModel::qt_metacall(QMetaObject::Call, int,
  void**) (moc_qqmldelegatemodel_p.cpp:292) ==16693== by 0x66379CC:
  QMetaObject::activate(QObject*, int, int, void**) (in
  /usr/lib/x86_64-linux-gnu/libQt5Core.so.5.3.0) ==16693== by
  0x65AEEFD: QAbstractItemModel::endResetModel() (in /usr/lib/x86_64-
  linux-gnu/libQt5Core.so.5.3.0) ==16693== by 0x23461EFD:
  FakeIndicatorsModel::unload() (fakeindicatorsmodel.cpp:53) ==16693==
  by 0x23461E13: FakeIndicatorsModel::~FakeIndicatorsModel()
  (fakeindicatorsmodel.cpp:34) ==16693== by 0x2345C073:
  QQmlPrivate::QQmlElement<FakeIndicatorsModel>::~QQmlElement() (in
  /home/tsdgeos_work/phablet/unity8/investigate_test_shell_crash/build
  dir/tests/mocks/Unity/Indicators/libIndicatorsFakeQml.so) ==16693==
  by 0x2345C0A3:
  QQmlPrivate::QQmlElement<FakeIndicatorsModel>::~QQmlElement()
  (qqmlprivate.h:106) ==16693== by 0x663636B:
  QObjectPrivate::deleteChildren() (in /usr/lib/x86_64-linux-
  gnu/libQt5Core.so.5.3.0) ==16693== Address 0x1862d448 is 8 bytes
  inside a block of size 16 free'd ==16693== at 0x4C2C2BC: operator
  delete(void*) (vg_replace_malloc.c:503) ==16693== by 0x72B21B8:
  QQmlContextData::destroy() (qqmlcontext.cpp:647) ==16693== by
  0x7293458: QQmlPrivate::qdeclarativeelement_destructor(QObject*)
  (qqmlengine.cpp:612) ==16693== by 0x6C0CADD:
  QQmlPrivate::QQmlElement<QQuickItem>::~QQmlElement()
  (qqmlprivate.h:105) ==16693== by 0x663636B:
  QObjectPrivate::deleteChildren() (in /usr/lib/x86_64-linux-
  gnu/libQt5Core.so.5.3.0) ==16693== by 0x663F0EB: QObject::~QObject()
  (in /usr/lib/x86_64-linux-gnu/libQt5Core.so.5.3.0) ==16693== by
  0x6BF64B5: QQuickItem::~QQuickItem() (qquickitem.cpp:2064) ==16693==
  by 0x6C0CAE5: QQmlPrivate::QQmlElement<QQuickItem>::~QQmlElement()
  (qqmlprivate.h:106) ==16693== by 0x663636B:
  QObjectPrivate::deleteChildren() (in /usr/lib/x86_64-linux-
  gnu/libQt5Core.so.5.3.0) ==16693== by 0x663F0EB: QObject::~QObject()
  (in /usr/lib/x86_64-linux-gnu/libQt5Core.so.5.3.0) ==16693== by
  0x6BF64B5: QQuickItem::~QQuickItem() (qquickitem.cpp:2064) ==16693==
  by 0x6C0CAE5: QQmlPrivate::QQmlElement<QQuickItem>::~QQmlElement()
  (qqmlprivate.h:106) (LP: #1332598)
[ Michael Zanetti ]
* drop launcher item spacing (LP: #1332022)
* change wording in launcher quicklist (LP: #1332035)
[ Alejandro J. Cura ]
* Initial code for a payment button widget, to handle purchasing apps
  from the click scope.
[ Rodney Dawes ]
* Initial code for a payment button widget, to handle purchasing apps
  from the click scope.
[ Renato Araujo Oliveira Filho ]
* Create IndicatorsLight.qml component used to control indicator led.
  A blue led will pulse if the message indicator is blue and screen is
  off.

854. By Ying-Chun Liu on 2014-07-09

Fix FTBFS

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:854
http://jenkins.qa.ubuntu.com/job/unity8-ci/3377/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/1746
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/381
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/471
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/471
        deb: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/471/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/471
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/2009
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/2802
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/2802/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/9533

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/3377/rebuild

review: Approve (continuous-integration)

[Michael Zanetti (mzanetti)]

I'm not sure if its a good idea to have the exact same interface exported into the shell and on D-Bus. Besides the fact that it looks quite chaotic, it also raises some security concerns here...

Michal, your opinion?

review: Needs Information

[Michał Sawicz (saviq)]

> I'm not sure if its a good idea to have the exact same interface exported into
> the shell and on D-Bus. Besides the fact that it looks quite chaotic, it also
> raises some security concerns here...

While I agree this is not ideal, the whole API here is rather temporary, until we spend the much needed time for analysis on session handling with accordance to fdo, systemd, logind etc. etc.

And applications can't call this API anyway, so it *should* be fine for now.

[lp:~paulliu/unity8/reboot]

855. By Ying-Chun Liu on 2014-07-10

Add fade-out effect.

856. By Ying-Chun Liu on 2014-07-10

Merge trunk

857. By Ying-Chun Liu on 2014-07-10

Fix bugs.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:857
http://jenkins.qa.ubuntu.com/job/unity8-ci/3390/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/1805/console
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/394
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/484
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/484
        deb: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/484/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/484
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/2069/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/2887
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/2887/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/9627

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/3390/rebuild

review: Needs Fixing (continuous-integration)

[Ying-Chun Liu (paulliu)]

The DBus is not used by the phone. It is for the desktop.
The indicators-session will use that DBus interface to ask the unity8 to shutdown.

[lp:~paulliu/unity8/reboot]

858. By Ying-Chun Liu on 2014-07-11

Merge trunk
[ Michal Hruby ]
* Fix FTBFS when using latest unity-api.
[ Michał Sawicz ]
* Refactor carousel item activation.
* Refactor ScopeItem into GenericScopeView.
* Add initial support for scope customizations.
* Make rating stars in PreviewReviewDisplay.qml non-interactive. (LP:
  #1337508)
[ Nick Dedekind ]
* Added active call hint A hint is displayed in the indicator panel
  when an call is active on the Telephony Serivce
[ Albert Astals ]
* We need to boostrap height also when we have 1 item ^_^ (LP:
  #1337408)
* Add initial support for scope customizations.
* CardCreator: Give a correct implicitHeight if we only have art The
  hasSubtitle change is really unrelated and not needed here, just
  sneaking it in to not create yet another review. (LP: #1330899)
* Fake Scopes Plugin: Register PreviewModelInterface
[ Michael Zanetti ]
* make the launcher's drag'n'drop indicator more prominent (LP:
  #1332042)
* make launcher items live having them non-live is not really required
  and reveals an issue in combination with UbuntuShape (LP: #1302761)
* Fade out launcher in place instead of moving it to the left on long
  left edge swipes. (LP: #1332096)
* update launcher icon glow as requested by design (LP: #1336725)
* update header in dash to use the new header from the SDK (LP:
  #1335491)
* fix testPreview with larger GRID_UNIT_PX values
* clip the corner of pinned icons in the launcher as per new design
* update launcher background according to latest design (LP: #1336314)
* Update Launcher's home button design according to new spec. (LP:
  #1329331)
[ CI bot ]
* make launcher items live having them non-live is not really required
  and reveals an issue in combination with UbuntuShape (LP: #1302761)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:858
http://jenkins.qa.ubuntu.com/job/unity8-ci/3409/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/1873/console
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/413
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/503
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/503
        deb: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/503/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/503
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/2130/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/2970
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/2970/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/9706

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/3409/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:858
http://jenkins.qa.ubuntu.com/job/unity8-ci/3419/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/1899
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/423
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/513
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/513
        deb: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/513/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/513
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/2152
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/3000
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/3000/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/9731

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/3419/rebuild

review: Needs Fixing (continuous-integration)

[Michael Zanetti (mzanetti)]

> The DBus is not used by the phone. It is for the desktop.
> The indicators-session will use that DBus interface to ask the unity8 to
> shutdown.

Not sure I understand what you're trying to say with this. But anyways, given its temporary I guess we can land as is... It worked fine in my tests...

 * Did you perform an exploratory manual test run of the code change and any related functionality?

ya

 * Did CI run pass? If not, please explain why.

no. flaky, unrelated tests

review: Approve

[Michael Zanetti (mzanetti)]

Oh... just saw the description says to NOT approve this branch yet... What's the status with that?

review: Needs Information

[Ying-Chun Liu (paulliu)]

> Oh... just saw the description says to NOT approve this branch yet... What's
> the status with that?

Sorry. I fix the description.

Thanks.

[Michael Zanetti (mzanetti)]

ack

review: Approve

[lp:~paulliu/unity8/reboot]

859. By Ying-Chun Liu on 2014-07-15

Merge trunk.
[ Michał Sawicz ]
* Activate all results in click scope by default. (LP: #1341262)
[ Mirco Müller ]
* Added support for utilization of the ComboButton SDK-element for
  snap-decision notifications with many actions.

860. By Ying-Chun Liu on 2014-07-16

Merge trunk.

[Ying-Chun Liu (paulliu)]

I just merge trunk and request a jenkins rebuild to see if it fix.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:858
http://jenkins.qa.ubuntu.com/job/unity8-ci/3455/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/2088/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/456/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/549
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/549
        deb: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/549/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/549
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/2318/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/3224
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/3224/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/9956

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/3455/rebuild

review: Needs Fixing (continuous-integration)

=== modified file 'plugins/Unity/Session/dbusunitysessionservice.cpp'
--- plugins/Unity/Session/dbusunitysessionservice.cpp	2014-05-27 11:44:25 +0000
+++ plugins/Unity/Session/dbusunitysessionservice.cpp	2014-07-16 14:44:35 +0000
@@ -19,6 +19,7 @@
 
 // Qt
 #include <QDBusConnection>
+#include <QDBusInterface>
 
 DBusUnitySessionService::DBusUnitySessionService() : QObject()
 {
@@ -44,3 +45,35 @@
 {
   Q_EMIT logoutRequested(false);
 }
+
+void DBusUnitySessionService::Reboot()
+{
+  QDBusConnection connection = QDBusConnection::systemBus();
+  QDBusInterface iface1 ("org.freedesktop.login1",
+                         "/org/freedesktop/login1",
+                         "org.freedesktop.login1.Manager",
+                         connection);
+
+  iface1.call("Reboot", false);
+}
+
+void DBusUnitySessionService::RequestReboot()
+{
+  Q_EMIT rebootRequested(false);
+}
+
+void DBusUnitySessionService::Shutdown()
+{
+  QDBusConnection connection = QDBusConnection::systemBus();
+  QDBusInterface iface1 ("org.freedesktop.login1",
+                         "/org/freedesktop/login1",
+                         "org.freedesktop.login1.Manager",
+                         connection);
+
+  iface1.call("PowerOff", false);
+}
+
+void DBusUnitySessionService::RequestShutdown()
+{
+  Q_EMIT shutdownRequested(false);
+}

=== modified file 'plugins/Unity/Session/dbusunitysessionservice.h'
--- plugins/Unity/Session/dbusunitysessionservice.h	2014-05-26 15:11:09 +0000
+++ plugins/Unity/Session/dbusunitysessionservice.h	2014-07-16 14:44:35 +0000
@@ -47,6 +47,27 @@
     void logoutRequested(bool have_inhibitors);
 
     /**
+     * rebootRequested signal
+     *
+     * This signal is emitted when some applications request the system to
+     * reboot.
+     * @param have_inhibitors if there are any special running applications
+     *        which inhibit the reboot.
+     */
+    void rebootRequested(bool have_inhibitors);
+
+    /**
+     * shutdownRequested signal
+     *
+     * This signal is emitted when some applications request the system to
+     * shutdown.
+     * @param have_inhibitors if there are any special running applications
+     *        which inhibit the shutdown.
+     */
+    void shutdownRequested(bool have_inhibitors);
+
+
+    /**
      * logoutReady signal
      *
      * This signal is emitted when all the apps are closed. And the system
@@ -61,22 +82,58 @@
      * This method directly logout the system without user's confirmation.
      * Ordinary applications should avoid calling this method. Please call
      * RequestLogout() to ask the user to decide logout or not.
-     * This method will stop all the running applications and then signal
-     * logoutReady when all the apps stopped.
      */
     Q_SCRIPTABLE void Logout();
 
     /**
+     * Reboot the system.
+     *
+     * This method directly reboot the system without user's confirmation.
+     * Ordinary applications should avoid calling this method. Please call
+     * RequestReboot() to ask the user to decide reboot or not.
+     */
+    Q_SCRIPTABLE void Reboot();
+
+    /**
+     * Shutdown the system.
+     *
+     * This method directly shutdown the system without user's confirmation.
+     * Ordinary applications should avoid calling this method. Please call
+     * RequestShutdown() to ask the user to decide shutdown or not.
+     */
+    Q_SCRIPTABLE void Shutdown();
+
+    /**
      * Issue a logout request.
      *
      * This method emit the logoutRequested signal to the shell with a boolean
      * which indicates if there's any inhibitors. The shell should receive
      * this signal and display a dialog to ask the user to confirm the logout
-     * action. If confirmed, shell can call Logout() method to kill the apps
-     * and then logout
+     * action. If confirmed, shell can call Logout() method to logout.
      */
     Q_SCRIPTABLE void RequestLogout();
 
+    /**
+     * Issue a reboot request.
+     *
+     * This method emit the rebootRequested signal to the shell with a boolean
+     * which indicates if there's any inhibitors. The shell should receive
+     * this signal and display a dialog to ask the user to confirm the reboot
+     * action. If confirmed, shell can call Reboot() method to reboot
+     */
+    Q_SCRIPTABLE void RequestReboot();
+
+    /**
+     * Issue a shutdown request.
+     *
+     * This method emit the shutdownRequested signal to the shell with a
+     * boolean which indicates if there's any inhibitors.
+     * The shell should receive
+     * this signal and display a dialog to ask the user to confirm the reboot
+     * action. If confirmed, shell can call Shutdown() method to reboot
+     */
+    Q_SCRIPTABLE void RequestShutdown();
+
 };
 
 #endif // DBUSUNITYSESSIONSERVICE_H

=== modified file 'qml/Shell.qml'
--- qml/Shell.qml	2014-07-09 19:44:14 +0000
+++ qml/Shell.qml	2014-07-16 14:44:35 +0000
@@ -19,6 +19,7 @@
 import GSettings 1.0
 import Unity.Application 0.1
 import Ubuntu.Components 0.1
+import Ubuntu.Components.Popups 0.1
 import Ubuntu.Gestures 0.1
 import Unity.Launcher 0.1
 import LightDM 0.1 as LightDM
@@ -268,7 +269,128 @@
             }
         }
 
+        property bool dialogShown: false
+
+        Component {
+            id: logoutDialog
+            Dialog {
+                id: dialogueLogout
+                title: "Logout"
+                text: "Are you sure that you want to logout?"
+                Button {
+                    text: "Cancel"
+                    onClicked: {
+                        PopupUtils.close(dialogueLogout);
+                        stages.dialogShown = false;
+                    }
+                }
+                Button {
+                    text: "Yes"
+                    onClicked: {
+                        DBusUnitySessionService.Logout();
+                        PopupUtils.close(dialogueLogout);
+                        stages.dialogShown = false;
+                    }
+                }
+            }
+        }
+
+        Component {
+            id: shutdownDialog
+            Dialog {
+                id: dialogueShutdown
+                title: "Shutdown"
+                text: "Are you sure that you want to shutdown?"
+                Button {
+                    text: "Cancel"
+                    onClicked: {
+                        PopupUtils.close(dialogueShutdown);
+                        stages.dialogShown = false;
+                    }
+                }
+                Button {
+                    text: "Yes"
+                    onClicked: {
+                        dBusUnitySessionServiceConnection.closeAllApps();
+                        DBusUnitySessionService.Shutdown();
+                        PopupUtils.close(dialogueShutdown);
+                        stages.dialogShown = false;
+                    }
+                }
+            }
+        }
+
+        Component {
+            id: rebootDialog
+            Dialog {
+                id: dialogueReboot
+                title: "Reboot"
+                text: "Are you sure that you want to reboot?"
+                Button {
+                    text: "Cancel"
+                    onClicked: {
+                        PopupUtils.close(dialogueReboot)
+                        stages.dialogShown = false;
+                    }
+                }
+                Button {
+                    text: "Yes"
+                    onClicked: {
+                        dBusUnitySessionServiceConnection.closeAllApps();
+                        DBusUnitySessionService.Reboot();
+                        PopupUtils.close(dialogueReboot);
+                        stages.dialogShown = false;
+                    }
+                }
+            }
+        }
+
+        Component {
+            id: powerDialog
+            Dialog {
+                id: dialoguePower
+                title: "Power"
+                text: i18n.tr("Are you sure you would like to turn power off?")
+                Button {
+                    text: i18n.tr("Power off")
+                    onClicked: {
+                        dBusUnitySessionServiceConnection.closeAllApps();
+                        PopupUtils.close(dialoguePower);
+                        stages.dialogShown = false;
+                        shutdownFadeOutRectangle.enabled = true;
+                        shutdownFadeOutRectangle.visible = true;
+                        shutdownFadeOut.start();
+                    }
+                }
+                Button {
+                    text: i18n.tr("Restart")
+                    onClicked: {
+                        dBusUnitySessionServiceConnection.closeAllApps();
+                        DBusUnitySessionService.Reboot();
+                        PopupUtils.close(dialoguePower);
+                        stages.dialogShown = false;
+                    }
+                }
+                Button {
+                    text: i18n.tr("Cancel")
+                    onClicked: {
+                        PopupUtils.close(dialoguePower);
+                        stages.dialogShown = false;
+                    }
+                }
+            }
+        }
+
+        function showPowerDialog() {
+            if (!stages.dialogShown) {
+                stages.dialogShown = true;
+                PopupUtils.open(powerDialog);
+            }
+        }
+
         Connections {
+            id: dBusUnitySessionServiceConnection
+            objectName: "dBusUnitySessionServiceConnection"
             target: DBusUnitySessionService
 
             function closeAllApps() {
@@ -282,8 +404,27 @@
             }
 
             onLogoutRequested: {
-                // TODO: Display a dialog to ask the user to confirm.
-                DBusUnitySessionService.Logout();
+                // Display a dialog to ask the user to confirm.
+                if (!stages.dialogShown) {
+                    stages.dialogShown = true;
+                    PopupUtils.open(logoutDialog);
+                }
+            }
+
+            onShutdownRequested: {
+                // Display a dialog to ask the user to confirm.
+                if (!stages.dialogShown) {
+                    stages.dialogShown = true;
+                    PopupUtils.open(shutdownDialog);
+                }
+            }
+
+            onRebootRequested: {
+                // Display a dialog to ask the user to confirm.
+                if (!stages.dialogShown) {
+                    stages.dialogShown = true;
+                    PopupUtils.open(rebootDialog);
+                }
             }
 
             onLogoutReady: {
@@ -679,4 +820,50 @@
         target: SessionBroadcast
         onShowHome: showHome()
     }
+
+    Keys.onPressed: {
+        if (event.key == Qt.Key_PowerOff || event.key == Qt.Key_PowerDown) {
+            if (!powerKeyTimer.running) {
+                powerKeyTimer.start();
+            }
+            event.accepted = true;
+        }
+    }
+
+    Keys.onReleased: {
+        if (event.key == Qt.Key_PowerOff || event.key == Qt.Key_PowerDown) {
+            powerKeyTimer.stop();
+            event.accepted = true;
+        }
+    }
+
+    Timer {
+        id: powerKeyTimer
+        interval: 2000
+        repeat: false
+        triggeredOnStart: false
+
+        onTriggered: {
+            stages.showPowerDialog();
+        }
+    }
+
+    Rectangle {
+        id: shutdownFadeOutRectangle
+        enabled: false
+        visible: false
+        color: "black"
+        anchors.fill: parent
+        opacity: 0.0
+        NumberAnimation on opacity {
+            id: shutdownFadeOut
+            from: 0.0
+            to: 1.0
+            onStopped: {
+                if (shutdownFadeOutRectangle.enabled && shutdownFadeOutRectangle.visible) {
+                    DBusUnitySessionService.Shutdown();
+                }
+            }
+        }
+    }
 }