From: Piotr Kosko <p.kosko@samsung.com>
Date: Wed, 27 May 2015 13:15:34 +0000 (+0200)
Subject: [Secureelement] Added logs in methods onEntered and onReturn
X-Git-Tag: submit/tizen_mobile/20150612.133019^2~2^2~88
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=92e6ee7caf43cf6ed7957ae6e561c7ef92301f09;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git

[Secureelement] Added logs in methods onEntered and onReturn

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

diff --git a/src/secureelement/secureelement_channel.cc b/src/secureelement/secureelement_channel.cc
index 3a662f57..d06cf75b 100644
--- a/src/secureelement/secureelement_channel.cc
+++ b/src/secureelement/secureelement_channel.cc
@@ -20,6 +20,7 @@ void SEChannel::close() {
 
 
 ByteArray SEChannel::transmit(const picojson::array& v_command) {
+    LoggerD("Entered");
     ByteArray response;
     if ( m_channel_ptr) {
         uint8_t* command = new uint8_t[v_command.size()];
@@ -35,6 +36,7 @@ ByteArray SEChannel::transmit(const picojson::array& v_command) {
 
 
 ByteArray SEChannel::getSelectResponse() {
+    LoggerD("Entered");
     ByteArray response;
     if ( m_channel_ptr) {
         response = m_channel_ptr->getSelectResponse();
diff --git a/src/secureelement/secureelement_instance.cc b/src/secureelement/secureelement_instance.cc
index 86827926..ed324c58 100644
--- a/src/secureelement/secureelement_instance.cc
+++ b/src/secureelement/secureelement_instance.cc
@@ -22,6 +22,8 @@ using namespace smartcard_service_api;
 
 SecureElementInstance::SecureElementInstance()
     : service_(*this) {
+    LoggerD("Entered");
+
     using std::placeholders::_1;
     using std::placeholders::_2;
 
@@ -116,7 +118,7 @@ void SecureElementInstance::CloseSessions(
 }
 
 void SecureElementInstance::CloseChannel( const picojson::value& args, picojson::object& out) {
-    LoggerD("Enter");
+    LoggerD("Entered");
     ClientChannel* channel_ptr = (ClientChannel*) static_cast<long>(args.get("handle").get<double>());
     SEChannel seChannel(channel_ptr);
     seChannel.close();
@@ -124,7 +126,7 @@ void SecureElementInstance::CloseChannel( const picojson::value& args, picojson:
 }
 
 void SecureElementInstance::GetSelectResponse( const picojson::value& args, picojson::object& out) {
-    LoggerD("Enter");
+    LoggerD("Entered");
     ClientChannel* channel_ptr = (ClientChannel*) static_cast<long>(args.get("handle").get<double>());
     SEChannel seChannel(channel_ptr);
 
diff --git a/src/secureelement/secureelement_seservice.cc b/src/secureelement/secureelement_seservice.cc
index adbef37e..16b3399e 100644
--- a/src/secureelement/secureelement_seservice.cc
+++ b/src/secureelement/secureelement_seservice.cc
@@ -103,6 +103,7 @@ void SEService::GetReaders(double callback_id) {
 
   if (is_error_) {
     // there has been an error, report it asynchronously
+    LoggerE("Failed: is_error_");
     std::shared_ptr<picojson::value> response{new picojson::value{picojson::object{}}};
     ReportError(
         PlatformResult(ErrorCode::SERVICE_NOT_AVAILABLE_ERR,
@@ -159,7 +160,7 @@ void SEService::ServiceConnected() {
 
   // notify the listeners
   if (!is_listener_set_) {
-    LoggerW("SE listener is not set.");
+    LoggerE("SE listener is not set.");
     return;
   }
 
diff --git a/src/secureelement/secureelement_session.cc b/src/secureelement/secureelement_session.cc
index aca6b563..aca5e602 100644
--- a/src/secureelement/secureelement_session.cc
+++ b/src/secureelement/secureelement_session.cc
@@ -64,6 +64,7 @@ picojson::value SESession::isClosed() {
 
 
 void SESession::close() {
+    LoggerD("Entered");
     if ( m_session_ptr) {
         m_session_ptr->closeSync();
     }
@@ -71,6 +72,7 @@ void SESession::close() {
 
 
 ByteArray SESession::getATR() {
+    LoggerD("Entered");
     ByteArray response;
     if ( m_session_ptr) {
         response = m_session_ptr->getATRSync();
@@ -80,6 +82,7 @@ ByteArray SESession::getATR() {
 
 
 void SESession::closeChannels() {
+    LoggerD("Entered");
     if ( m_session_ptr) {
         m_session_ptr->closeChannels();
     }