From ced37f52231cc3a3d4d97d46d57ec3da148b47bc Mon Sep 17 00:00:00 2001
From: ac__fancheyu <liuyangyang58@huawei.com>
Date: Wed, 28 Dec 2022 15:04:52 +0800
Subject: [PATCH] WindowManagerModifyErrorCode

Signed-off-by: ac__fancheyu <liuyangyang58@huawei.com>
Change-Id: I99dd6cd45d31296ea11d95e347c7a27b4ead3617
---
 screenhandler/src/dscreen_handler.cpp                    | 9 +++++----
 .../dscreenmgr/src/screen_manager_adapter.cpp            | 8 ++++----
 2 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/screenhandler/src/dscreen_handler.cpp b/screenhandler/src/dscreen_handler.cpp
index 5b4a669a..5741409d 100644
--- a/screenhandler/src/dscreen_handler.cpp
+++ b/screenhandler/src/dscreen_handler.cpp
@@ -48,8 +48,8 @@ int32_t DScreenHandler::Initialize()
     if (screenListener_ == nullptr) {
         screenListener_ = new (std::nothrow) ScreenListener();
     }
-    bool ret = Rosen::ScreenManager::GetInstance().RegisterScreenListener(screenListener_);
-    if (!ret) {
+    Rosen::DMError ret = Rosen::ScreenManager::GetInstance().RegisterScreenListener(screenListener_);
+    if (ret != Rosen::DMError::DM_OK) {
         DHLOGE("register screen listener failed.");
         return DSCREEN_INIT_ERR;
     }
@@ -110,7 +110,8 @@ std::vector<DHItem> DScreenHandler::Query()
 {
     DHLOGI("DScreenHandler query hardware info");
     std::vector<DHItem> dhItemVec;
-    std::vector<sptr<Rosen::Screen>> screens = Rosen::ScreenManager::GetInstance().GetAllScreens();
+    std::vector<sptr<Rosen::Screen>> screens;
+    Rosen::ScreenManager::GetInstance().GetAllScreens(screens);
     DHLOGI("screens size is: %d.", screens.size());
     for (const auto &screen : screens) {
         if (screen == nullptr) {
@@ -182,7 +183,7 @@ std::string DScreenHandler::QueryCodecInfo()
     }
     std::vector<std::shared_ptr<Media::VideoCaps>> caps = codecList->GetVideoEncoderCaps();
     json codecTypeArray = json::array();
-    
+
     for (const auto &cap : caps) {
         if (cap == nullptr) {
             continue;
diff --git a/services/screenservice/sourceservice/dscreenmgr/src/screen_manager_adapter.cpp b/services/screenservice/sourceservice/dscreenmgr/src/screen_manager_adapter.cpp
index 05227512..cb003f18 100644
--- a/services/screenservice/sourceservice/dscreenmgr/src/screen_manager_adapter.cpp
+++ b/services/screenservice/sourceservice/dscreenmgr/src/screen_manager_adapter.cpp
@@ -85,8 +85,8 @@ int32_t ScreenMgrAdapter::RegisterScreenGroupListener(sptr<Rosen::ScreenManager:
         DHLOGI("already registered listener.");
         return DH_SUCCESS;
     }
-    bool ret = Rosen::ScreenManager::GetInstance().RegisterScreenGroupListener(listener);
-    if (!ret) {
+    Rosen::DMError ret = Rosen::ScreenManager::GetInstance().RegisterScreenGroupListener(listener);
+    if (ret != Rosen::DMError::DM_OK) {
         DHLOGE("RegisterScreenGroupListener Failed.");
         return ERR_DH_SCREEN_SA_REGISTER_SCREENLISTENER_FAIL;
     }
@@ -105,8 +105,8 @@ int32_t ScreenMgrAdapter::UnregisterScreenGroupListener(sptr<Rosen::ScreenManage
         DHLOGI("listener already unregistered.");
         return DH_SUCCESS;
     }
-    bool ret = Rosen::ScreenManager::GetInstance().UnregisterScreenGroupListener(listener);
-    if (!ret) {
+    Rosen::DMError ret = Rosen::ScreenManager::GetInstance().UnregisterScreenGroupListener(listener);
+    if (ret != Rosen::DMError::DM_OK) {
         DHLOGE("UnregisterScreenGroupListener Failed.");
         return ERR_DH_SCREEN_SA_UNREGISTER_SCREENLISTENER_FAIL;
     }
-- 
Gitee