UI: supports reboot from context menu
authorJinhyung Choi <jinh0.choi@samsung.com>
Wed, 4 Nov 2015 02:33:51 +0000 (11:33 +0900)
committerSeokYeon Hwang <syeon.hwang@samsung.com>
Thu, 26 Nov 2015 04:46:08 +0000 (13:46 +0900)
When a malfunction of emulator occurs, such as no response from the guest OS,
what a developer can do is selecting "force close"
and restarting from emulator manager.

In order to avoid the inconvenience of doing it, reboot menu is added.
It is working directly to hardware after requesting sync() to guest OS

         ----------------
         | Context menu |
         ----------------
         |   reboot     |
         ----------------                         ----------
                |                                 | emuld  |
     ------------------------                     ----------
     | 1. send sync message |   ------------->    | sync() |
     | 2. start a timer     |                     ----------
     ------------------------
               | The timer expires after 1s
   -------------------------------
   | qemu_system_reset_request() |
   -------------------------------

Conflicts:
tizen/src/extension/ui/resource/skins/tv-1080x1920/layout.xml
tizen/src/extension/ui/resource/skins/tv-1920x1080/layout.xml
tizen/src/extension/ui/resource/skins/tv-2560x1080/layout.xml

Change-Id: I3c3c31c2cc19065f53b620d83c79e68e0183db53
Signed-off-by: Jinhyung Choi <jinh0.choi@samsung.com>
(cherry picked from commit 14fc61baff1e8b48cbcb810a8392a1ac601f2511)

22 files changed:
tizen/src/ecs/ecs.h
tizen/src/ecs/ecs_msg.c
tizen/src/emulator.c
tizen/src/ui/menu/contextmenu.cpp
tizen/src/ui/menu/contextmenu.h
tizen/src/ui/menu/menuitem.h
tizen/src/ui/resource/icons/system_reset.png [new file with mode: 0644]
tizen/src/ui/resource/resource.qrc
tizen/src/ui/resource/skins/mobile-320x480-3btn/layout.xml
tizen/src/ui/resource/skins/mobile/mobile-480x800-3btn/layout.xml
tizen/src/ui/resource/skins/mobile/mobile-540x960-3btn/layout.xml
tizen/src/ui/resource/skins/mobile/mobile-720x1280-3btn/layout.xml
tizen/src/ui/resource/skins/mobile/mobile-general-3btn/layout.xml
tizen/src/ui/resource/skins/tv/tv-1920x1080/layout.xml
tizen/src/ui/resource/skins/wearable/wearable-320x320-1btn/layout.xml
tizen/src/ui/resource/skins/wearable/wearable-360x360-1btn/layout.xml
tizen/src/ui/resource/skins/wearable/wearable-360x480-1btn/layout.xml
tizen/src/ui/resource/skins/wearable/wearable-general-1btn/layout.xml
tizen/src/ui/resource/skins/wearable/wearableO-360x360-2btn/layout.xml
tizen/src/ui/resource/skins/wearableO-360x360-1btn/layout.xml
tizen/src/ui/xmllayoutkeyword.h
tizen/src/ui/xmllayoutparser.cpp

index b201a15..c1657f9 100644 (file)
@@ -62,6 +62,7 @@
 #define MSG_TYPE_GUESTIP        "guest_ip"
 #define MSG_TYPE_HDS            "hds"
 #define MSG_TYPE_PACKAGE        "package"
+#define MSG_TYPE_SYSTEM         "system"
 
 #define MSG_GROUP_STATUS        15
 
@@ -184,7 +185,12 @@ bool ntf_to_monitor(const char* data, const int len);
 bool send_msg_to_guest(const char* cmd, int group, int action,
                        char* data, int data_len);
 
-void send_shutdown_request(void);
+enum ecs_system_action {
+    ECS_SYSTEM_ACTION_FORCE_CLOSE,
+    ECS_SYSTEM_ACTION_REBOOT
+};
+
+void send_shutdown_request(int action);
 
 void make_send_device_ntf (char* cmd, int group, int action, char* data);
 
index f54b450..b8fb428 100644 (file)
@@ -193,9 +193,9 @@ bool send_msg_to_guest(const char* cmd, int group, int action, char* data, int d
     return true;
 }
 
-void send_shutdown_request(void)
+void send_shutdown_request(int action)
 {
-    int ret = send_msg_to_guest("system", 0, 0, NULL, 0);
+    int ret = send_msg_to_guest(MSG_TYPE_SYSTEM, 0, action, NULL, 0);
     if (!ret) {
         LOG_SEVERE("fail to send evdi shutdown system call to emuld.\n");
     }
index 8d85458..e8398e8 100644 (file)
@@ -130,7 +130,7 @@ static Notifier emulator_exit = { .notify = emulator_notify_exit };
 
 static void* run_timed_shutdown_thread(void* args)
 {
-    send_shutdown_request();
+    send_shutdown_request(ECS_SYSTEM_ACTION_FORCE_CLOSE);
 
     const int sleep_interval_time = 1000; /* milli-seconds */
     const int timeout_for_shutdown = (uintptr_t)args;
index 90f77de..72ea6d5 100644 (file)
@@ -40,6 +40,9 @@
 extern "C" {
 // FIXME: To avoid very complex header inclusion chains
 void qemu_system_graceful_shutdown_request(unsigned int sec);
+void qemu_system_reset_request(void);
+#define MENU_ACTION_REBOOT      1
+void send_shutdown_request(int action);
 
 #include "util/osutil.h"
 #include "util/device_hotplug.h"
@@ -80,6 +83,9 @@ ContextMenu::ContextMenu(QWidget *parent) : QMenu(parent)
     /* for close */
     longPressTimer = new QTimer(this);
 
+    /* for system reset */
+    rebootTimer = new QTimer(this);
+
     createItems(this, this->parent->getUiInfo()->getMenuList());
 
     installEventFilter(this);
@@ -143,6 +149,10 @@ void ContextMenu::createItems(QMenu *menu, QList<MenuItem *> &list)
             /* About menu */
             createAboutItem(menu, item);
             break;
+        case MenuItemType::systemResetItem:
+            /* System Reset menu */
+            createSystemResetItem(menu, item);
+            break;
         case MenuItemType::forceCloseItem:
             /* Force Close menu */
             createForceCloseItem(menu, item);
@@ -525,6 +535,23 @@ void ContextMenu::createAboutItem(QMenu *menu, MenuItem *item)
     item->setAction(actionAbout);
 }
 
+void ContextMenu::createSystemResetItem(QMenu *menu, MenuItem *item)
+{
+    if (menu == NULL || item == NULL) {
+        return;
+    }
+
+    QString menuName = item->getName();
+    actionSystemReset = addGeneralAction(
+        menu, QIcon(QPixmap(":/icons/system_reset.png")),
+        menuName.isEmpty() ? MENU_FORCECLOSE_ITEM_TEXT : menuName,
+        item->getShortcuts().isEmpty()? NULL :
+            new QShortcut(item->getShortcuts().begin().value(), parent),
+        SLOT(slotSystemReset()));
+
+    item->setAction(actionSystemReset);
+}
+
 void ContextMenu::createForceCloseItem(QMenu *menu, MenuItem *item)
 {
     if (menu == NULL || item == NULL) {
@@ -1017,6 +1044,27 @@ void ContextMenu::slotAbout()
 #endif
 }
 
+void ContextMenu::slotDeviceReset()
+{
+    qDebug("System reset request.");
+    qemu_system_reset_request();
+    qDebug("Done for system reset request.");
+}
+
+void ContextMenu::slotSystemReset()
+{
+    qDebug("Reboot.");
+
+    send_shutdown_request(MENU_ACTION_REBOOT);
+
+    qDebug("Sent sync message to emuld.");
+
+    rebootTimer->setInterval(HARDWARE_REBOOT_INTERVAL);
+    rebootTimer->setSingleShot(true);
+    connect(rebootTimer, SIGNAL(timeout()), this, SLOT(slotDeviceReset()));
+    rebootTimer->start();
+}
+
 void ContextMenu::slotForceClose()
 {
     qDebug("force close");
@@ -1090,4 +1138,6 @@ ContextMenu::~ContextMenu()
     delete shellOpener;
 
     longPressTimer->stop();
+
+    rebootTimer->stop();
 }
index 63dc565..942a077 100644 (file)
@@ -49,6 +49,7 @@ extern "C" {
 }
 
 #define CLOSE_POWER_KEY_INTERVAL 1200 /* milli-seconds */
+#define HARDWARE_REBOOT_INTERVAL 1000 /* milli-seconds */
 
 class ContextMenu : public QMenu
 {
@@ -96,6 +97,9 @@ public slots:
     void slotHostKeyboard(bool on);
     void slotAbout();
 
+    void slotSystemReset();
+    void slotDeviceReset();
+
     void slotForceClose();
     void slotClose();
     void slotPwkeyRelease();
@@ -116,6 +120,7 @@ protected:
     void createControlPanelItem(QMenu *menu, MenuItem *item);
     void createScreenShotItem(QMenu *menu, MenuItem *item);
     void createAboutItem(QMenu *menu, MenuItem *item);
+    void createSystemResetItem(QMenu *menu, MenuItem *item);
     void createForceCloseItem(QMenu *menu, MenuItem *item);
     void createCloseItem(QMenu *menu, MenuItem *item);
     bool eventFilter(QObject *obj, QEvent *event);
@@ -146,6 +151,7 @@ private:
     QAction *actionControlPanel;
     QAction *actionScreenShot;
     QAction *actionAbout;
+    QAction *actionSystemReset;
     QAction *actionForceClose;
     QAction *actionClose;
 
@@ -155,5 +161,6 @@ private:
 
     ShellOpener *shellOpener;
     QTimer *longPressTimer;
+    QTimer *rebootTimer;
 };
 #endif // CONTEXTMENU_H
index d271700..0658bea 100644 (file)
@@ -50,6 +50,7 @@ namespace MenuItemType
         controlPanelItem,
         screenShotItem,
         aboutItem,
+        systemResetItem,
         forceCloseItem,
         closeItem,
     };
diff --git a/tizen/src/ui/resource/icons/system_reset.png b/tizen/src/ui/resource/icons/system_reset.png
new file mode 100644 (file)
index 0000000..bb4e7c2
Binary files /dev/null and b/tizen/src/ui/resource/icons/system_reset.png differ
index 5f009d3..7dce589 100644 (file)
@@ -38,6 +38,7 @@
         <file>icons/about.png</file>
         <file>icons/advanced.png</file>
         <file>icons/close.png</file>
+        <file>icons/system_reset.png</file>
         <file>icons/force_close.png</file>
         <file>icons/rotate.png</file>
         <file>icons/scale.png</file>
index 4170bfc..dcc7096 100644 (file)
             <shortcut>Ctrl+F6</shortcut>
         </controlPanelItem>
         <separator/>
+        <rebootItem name="Reboot">
+            <shortcut>Ctrl+Shift+F4</shortcut>
+        </rebootItem>
         <closeItem name="Close">
             <shortcut>Ctrl+F4</shortcut>
         </closeItem>
index 295b8f1..483bdbd 100644 (file)
             <shortcut>Ctrl+F6</shortcut>
         </controlPanelItem>
         <separator/>
+        <rebootItem name="Reboot">
+            <shortcut>Ctrl+Shift+F4</shortcut>
+        </rebootItem>
         <closeItem name="Close">
             <shortcut>Ctrl+F4</shortcut>
         </closeItem>
index 0929201..4c01197 100644 (file)
             <shortcut>Ctrl+F6</shortcut>
         </controlPanelItem>
         <separator/>
+        <rebootItem name="Reboot">
+            <shortcut>Ctrl+Shift+F4</shortcut>
+        </rebootItem>
         <closeItem name="Close">
             <shortcut>Ctrl+F4</shortcut>
         </closeItem>
index 39f1eb8..d52af18 100644 (file)
             <shortcut>Ctrl+F6</shortcut>
         </controlPanelItem>
         <separator/>
+        <rebootItem name="Reboot">
+            <shortcut>Ctrl+Shift+F4</shortcut>
+        </rebootItem>
         <closeItem name="Close">
             <shortcut>Ctrl+F4</shortcut>
         </closeItem>
index 798cbeb..e9eef24 100644 (file)
@@ -72,6 +72,9 @@
             <shortcut>Ctrl+F6</shortcut>
         </controlPanelItem>
         <separator/>
+        <rebootItem name="Reboot">
+            <shortcut>Ctrl+Shift+F4</shortcut>
+        </rebootItem>
         <closeItem name="Close">
             <shortcut>Ctrl+F4</shortcut>
         </closeItem>
index d6e748d..f338dc3 100644 (file)
@@ -55,6 +55,9 @@
             <shortcut>Ctrl+F6</shortcut>
         </controlPanelItem>
         <separator/>
+        <rebootItem name="Reboot">
+            <shortcut>Ctrl+Shift+F4</shortcut>
+        </rebootItem>
         <closeItem name="Close">
             <shortcut>Ctrl+F4</shortcut>
         </closeItem>
index fef73a6..b3811f6 100644 (file)
             <shortcut>Ctrl+F6</shortcut>
         </controlPanelItem>
         <separator/>
+        <rebootItem name="Reboot">
+            <shortcut>Ctrl+Shift+F4</shortcut>
+        </rebootItem>
         <closeItem name="Close">
             <shortcut>Ctrl+F4</shortcut>
         </closeItem>
index c6723b9..73e25c3 100644 (file)
             <shortcut>Ctrl+F6</shortcut>
         </controlPanelItem>
         <separator/>
+        <rebootItem name="Reboot">
+            <shortcut>Ctrl+Shift+F4</shortcut>
+        </rebootItem>
         <closeItem name="Close">
             <shortcut>Ctrl+F4</shortcut>
         </closeItem>
index 6f1d496..5181275 100644 (file)
             <shortcut>Ctrl+F6</shortcut>
         </controlPanelItem>
         <separator/>
+        <rebootItem name="Reboot">
+            <shortcut>Ctrl+Shift+F4</shortcut>
+        </rebootItem>
         <closeItem name="Close">
             <shortcut>Ctrl+F4</shortcut>
         </closeItem>
index e14fa34..15803d1 100644 (file)
@@ -62,6 +62,9 @@
             <shortcut>Ctrl+F6</shortcut>
         </controlPanelItem>
         <separator/>
+        <rebootItem name="Reboot">
+            <shortcut>Ctrl+Shift+F4</shortcut>
+        </rebootItem>
         <closeItem name="Close">
             <shortcut>Ctrl+F4</shortcut>
         </closeItem>
index cb9cbe2..2f60964 100644 (file)
             <shortcut>Ctrl+F6</shortcut>
         </controlPanelItem>
         <separator/>
+        <rebootItem name="Reboot">
+            <shortcut>Ctrl+Shift+F4</shortcut>
+        </rebootItem>
         <closeItem name="Close">
             <shortcut>Ctrl+F4</shortcut>
         </closeItem>
index 9021a69..67094fd 100644 (file)
             <shortcut>Ctrl+F6</shortcut>
         </controlPanelItem>
         <separator/>
+        <rebootItem name="Reboot">
+            <shortcut>Ctrl+Shift+F4</shortcut>
+        </rebootItem>
         <closeItem name="Close">
             <shortcut>Ctrl+F4</shortcut>
         </closeItem>
index 996ed5e..f8d2e10 100644 (file)
@@ -79,6 +79,7 @@
 #define ECP_MENU_KEYWORD "controlPanelItem"
 #define SCREENSHOT_MENU_KEYWORD "screenShotItem"
 #define ABOUT_MENU_KEYWORD "aboutItem"
+#define SYSTEM_RESET_MENU_KEYWORD "rebootItem"
 #define FORCECLOSE_MENU_KEYWORD "forceCloseItem"
 #define CLOSE_MENU_KEYWORD "closeItem"
 
index 2b016ec..483afbb 100644 (file)
@@ -502,6 +502,8 @@ int XmlLayoutParser::parseMenuList(
                 item = parseGeneralMenuItem(xml, MenuItemType::screenShotItem);
             } else if (xml.name() == ABOUT_MENU_KEYWORD) {
                 item = parseGeneralMenuItem(xml, MenuItemType::aboutItem);
+            } else if (xml.name() == SYSTEM_RESET_MENU_KEYWORD) {
+                item = parseGeneralMenuItem(xml, MenuItemType::systemResetItem);
             } else if (xml.name() == FORCECLOSE_MENU_KEYWORD) {
                 item = parseGeneralMenuItem(xml, MenuItemType::forceCloseItem);
             } else if (xml.name() == CLOSE_MENU_KEYWORD) {