Merge pull request #477 from rdkcentral/topic/RDKEMW-5405

RDKEMW-5405

Author: madanagopalt

apis/PackageManager/IAppPackageManager.h

@@ -48,31 +48,31 @@ namespace Exchange {
             DISK_PERSISTENCE_FAILURE
         };
 
-        //struct PackageInfo {
-        //    string downloadId;
-        //    string fileLocator;
-        //    Reason reason;
-        //};
+        struct PackageInfo {
+           string downloadId;   /*@brief Download ID*/
+           string fileLocator;  /*@brief File Locator*/
+           Reason reason;       /*@brief Reason for the download status*/
+        };
 
         //typedef std::vector<PackageInfo> PackageInfoList;
-        //using IPackageIterator = RPC::IIteratorType<PackageInfo, ID_PACKAGE_INFO_ITERATOR>;
+        using IPackageInfoIterator = RPC::IIteratorType<PackageInfo, ID_PACKAGE_INFO_ITERATOR>;
 
-        /* @event */
         struct EXTERNAL INotification : virtual public Core::IUnknown {
             enum { ID = ID_PACKAGE_DOWNLOADER_NOTIFICATION  };
             ~INotification() override = default;
 
             // @brief Signal changes on the status
             // @text onAppDownloadStatus
-            virtual void OnAppDownloadStatus(const string& jsonresponse) {
-                // Thunder does not support neither standard collection nor RPC::IIteratorType in notification
+            virtual void OnAppDownloadStatus(IPackageInfoIterator* const packageInfo) {
             }
         };
 
         ~IPackageDownloader() override = default;
 
         // Register for any changes
+        // @json:omit
         virtual Core::hresult Register(IPackageDownloader::INotification *sink) = 0;
+        // @json:omit
         virtual Core::hresult Unregister(IPackageDownloader::INotification *sink) = 0;
 
         // @json:omit