[Download] Added logs in methods onEntered and onReturn

Change-Id: I74c3dbfe007c976eadec5adb693c11deb9525479
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>

diff --git a/src/download/download_instance.cc b/src/download/download_instance.cc
index 14d7d74..69cc524 100644 (file)
--- a/src/download/download_instance.cc
+++ b/src/download/download_instance.cc
@@ -33,6 +33,7 @@ using common::ServiceNotAvailableException;
 using common::TypeMismatchException;
 
 DownloadInstance::DownloadInstance() {
+  LoggerD("Entered");
   using std::placeholders::_1;
   using std::placeholders::_2;
   #define REGISTER_SYNC(c, x) \
@@ -53,6 +54,7 @@ DownloadInstance::DownloadInstance() {
 }
 
 DownloadInstance::~DownloadInstance() {
+  LoggerD("Entered");
   for (DownloadCallbackVector::iterator it = downCbVector.begin();
     it != downCbVector.end(); it++) {
     delete (*it);
@@ -86,6 +88,7 @@ bool DownloadInstance::CheckInstance(DownloadInstance* instance) {
 
 void DownloadInstance::OnStateChanged(int download_id,
   download_state_e state, void* user_data) {
+  LoggerD("Entered");
   DownloadCallback* downCbPtr = static_cast<DownloadCallback*>(user_data);
 
   downCbPtr->state = state;
@@ -114,6 +117,7 @@ void DownloadInstance::OnStateChanged(int download_id,
 }
 
 gboolean DownloadInstance::OnProgressChanged(void* user_data) {
+  LoggerD("Entered");
   DownloadCallback* downCbPtr = static_cast<DownloadCallback*>(user_data);
   std::lock_guard<std::mutex> lock(instances_mutex_);
   if (!CheckInstance(downCbPtr->instance)) {
@@ -140,6 +144,7 @@ gboolean DownloadInstance::OnProgressChanged(void* user_data) {
 }
 
 void DownloadInstance::OnStart(int download_id, void* user_data) {
+  LoggerD("Entered");
   unsigned long long totalSize;
   int ret;
 
@@ -159,6 +164,7 @@ void DownloadInstance::OnStart(int download_id, void* user_data) {
 }
 
 gboolean DownloadInstance::OnFinished(void* user_data) {
+  LoggerD("Entered");
   char* fullPath = NULL;
 
   DownloadCallback* downCbPtr = static_cast<DownloadCallback*>(user_data);
@@ -188,6 +194,7 @@ gboolean DownloadInstance::OnFinished(void* user_data) {
 }
 
 gboolean DownloadInstance::OnPaused(void* user_data) {
+  LoggerD("Entered");
   DownloadCallback* downCbPtr = static_cast<DownloadCallback*>(user_data);
   std::lock_guard<std::mutex> lock(instances_mutex_);
   if (!CheckInstance(downCbPtr->instance)) {
@@ -208,6 +215,7 @@ gboolean DownloadInstance::OnPaused(void* user_data) {
 }
 
 gboolean DownloadInstance::OnCanceled(void* user_data) {
+  LoggerD("Entered");
   DownloadCallback* downCbPtr = static_cast<DownloadCallback*>(user_data);
   std::lock_guard<std::mutex> lock(instances_mutex_);
   if (!CheckInstance(downCbPtr->instance)) {
@@ -232,6 +240,7 @@ gboolean DownloadInstance::OnCanceled(void* user_data) {
 }
 
 gboolean DownloadInstance::OnFailed(void* user_data) {
+  LoggerD("Entered");
   download_error_e error;
   picojson::object out;
 
@@ -355,6 +364,7 @@ gboolean DownloadInstance::OnFailed(void* user_data) {
 
 void DownloadInstance::progress_changed_cb
   (int download_id, long long unsigned received, void* user_data) {
+  LoggerD("Entered");
   DownloadCallback* downCbPtr = static_cast<DownloadCallback*>(user_data);
   downCbPtr->received = received;
   downCbPtr->downloadId = download_id;
@@ -364,6 +374,7 @@ void DownloadInstance::progress_changed_cb
 
 void DownloadInstance::DownloadManagerStart
   (const picojson::value& args, picojson::object& out) {
+  LoggerD("Entered");
   CHECK_EXIST(args, "callbackId", out)
 
   int ret, downlodId;
@@ -428,6 +439,7 @@ void DownloadInstance::DownloadManagerStart
     network_available = cell_available || wifi_available;
     diPtr->network_type = DOWNLOAD_NETWORK_ALL;
   } else {
+    LoggerE("The input parameter contains an invalid network type");
     ReportError(
         InvalidValuesException(
             "The input parameter contains an invalid network type."),
@@ -437,6 +449,7 @@ void DownloadInstance::DownloadManagerStart
 
   if (!network_support) {
     SLoggerE("Requested network type (%s) is not supported.", networkType.c_str());
+    LoggerE("Requested network type is not supported");
     ReportError(
         common::PlatformResult(common::ErrorCode::NOT_SUPPORTED_ERR,
                                "The networkType of the given DownloadRequest "
@@ -447,6 +460,7 @@ void DownloadInstance::DownloadManagerStart
 
   if (!network_available) {
     SLoggerE("Requested network type (%s) is not available.", networkType.c_str());
+    LoggerE("Requested network type is not available");
     ReportError(
         common::PlatformResult(common::ErrorCode::NETWORK_ERR,
                                "The networkType of the given DownloadRequest "
@@ -527,12 +541,14 @@ void DownloadInstance::DownloadManagerStart
 
 void DownloadInstance::DownloadManagerCancel
   (const picojson::value& args, picojson::object& out) {
+  LoggerD("Entered");
   CHECK_EXIST(args, "downloadId", out)
   int downloadId, ret;
 
   int callbackId = static_cast<int>(args.get("downloadId").get<double>());
 
   if (!GetDownloadID(callbackId, downloadId)) {
+    LoggerE("The identifier does not match any download operation in progress");
     ReportError(NotFoundException
       ("The identifier does not match any download operation in progress"),
       out);
@@ -560,12 +576,14 @@ void DownloadInstance::DownloadManagerCancel
 
 void DownloadInstance::DownloadManagerPause
   (const picojson::value& args, picojson::object& out) {
+  LoggerD("Entered");
   CHECK_EXIST(args, "downloadId", out)
   int downloadId, ret;
 
   int callbackId = static_cast<int>(args.get("downloadId").get<double>());
 
   if (!GetDownloadID(callbackId, downloadId)) {
+    LoggerE("The identifier does not match any download operation in progress");
     ReportError(NotFoundException(
       "The identifier does not match any download operation in progress"),
       out);
@@ -593,6 +611,7 @@ void DownloadInstance::DownloadManagerPause
 
 void DownloadInstance::DownloadManagerResume
   (const picojson::value& args, picojson::object& out) {
+  LoggerD("Entered");
   CHECK_EXIST(args, "downloadId", out)
   int downloadId, ret;
 
@@ -636,6 +655,7 @@ void DownloadInstance::DownloadManagerResume
 }
 void DownloadInstance::DownloadManagerGetstate
   (const picojson::value& args, picojson::object& out) {
+  LoggerD("Entered");
   CHECK_EXIST(args, "downloadId", out)
   int downloadId, ret;
   std::string stateValue;
@@ -644,6 +664,7 @@ void DownloadInstance::DownloadManagerGetstate
   int callbackId = static_cast<int>(args.get("downloadId").get<double>());
 
   if (!GetDownloadID(callbackId, downloadId)) {
+    LoggerE("The identifier does not match any download operation in progress");
     ReportError(NotFoundException
       ("The identifier does not match any download operation in progress"),
       out);
@@ -693,6 +714,7 @@ void DownloadInstance::DownloadManagerGetstate
 
 void DownloadInstance::DownloadManagerGetmimetype
   (const picojson::value& args, picojson::object& out) {
+  LoggerD("Entered");
   CHECK_EXIST(args, "downloadId", out)
 
   int downloadId, ret;
@@ -729,7 +751,7 @@ void DownloadInstance::DownloadManagerGetmimetype
 
 bool DownloadInstance::GetDownloadID
   (const int callback_id, int& download_id) {
-
+  LoggerD("Entered");
   if (diMap.find(callback_id) != diMap.end()) {
     download_id = diMap.find(callback_id)->second->download_id;
   } else {
@@ -740,11 +762,13 @@ bool DownloadInstance::GetDownloadID
 
 void DownloadInstance::DownloadManagerGetdownloadrequest
   (const picojson::value& args, picojson::object& out) {
+  LoggerD("Entered");
   // Nothing to do
 }
 
 void DownloadInstance::DownloadManagerSetlistener
   (const picojson::value& args, picojson::object& out) {
+  LoggerD("Entered");
   // Nothing to do
 }