From 6fe18f6a466dbae611bbe1e4fa33b352cc4ef6df Mon Sep 17 00:00:00 2001
From: Angelo Mantellini <manangel@cisco.com>
Date: Wed, 12 Jun 2019 18:02:19 +0200
Subject: Correct android issues for libdash and viper

Change-Id: Ia5b95bda6336b6544c881c416f852069662f5e3f
Signed-off-by: Angelo Mantellini <manangel@cisco.com>
---
 Input/ICNConnectionConsumerApi.cpp              |   4 +-
 Managers/MultimediaManager.cpp                  |   5 +--
 UI/DASHPlayer.cpp                               |   2 +-
 android/AndroidManifest.xml                     |   6 +--
 android/gradle.properties                       |   4 +-
 android/res/drawable-hdpi/icon.png              | Bin 2449 -> 127864 bytes
 android/res/drawable-ldpi/icon.png              | Bin 1285 -> 127864 bytes
 android/res/drawable-mdpi/icon.png              | Bin 1777 -> 114093 bytes
 android/src/io/fd/viper/ViperActivity.java      |  51 ++++++++++++++++++++++++
 android/src/org/player/viper/ViperActivity.java |  51 ------------------------
 libdash/CMakeLists.txt                          |  44 ++++++++------------
 main.cpp                                        |  19 +++++++--
 qml/Viper/Options.qml                           |   1 -
 viper.pro                                       |  17 +++++---
 14 files changed, 106 insertions(+), 98 deletions(-)
 mode change 100755 => 100644 android/res/drawable-hdpi/icon.png
 mode change 100755 => 100644 android/res/drawable-ldpi/icon.png
 mode change 100755 => 100644 android/res/drawable-mdpi/icon.png
 create mode 100755 android/src/io/fd/viper/ViperActivity.java
 delete mode 100755 android/src/org/player/viper/ViperActivity.java

diff --git a/Input/ICNConnectionConsumerApi.cpp b/Input/ICNConnectionConsumerApi.cpp
index c93e9ed4..59330852 100644
--- a/Input/ICNConnectionConsumerApi.cpp
+++ b/Input/ICNConnectionConsumerApi.cpp
@@ -160,8 +160,8 @@ int	ICNConnectionConsumerApi::Read(uint8_t *data, size_t len)
     } else
     {
         memcpy(data, (char*)response.getPayload().data() + this->dataPos, response.getPayload().size() - this->dataPos);
-	 int length = response.getPayload().size() - this->dataPos;
-	if (length == 0)
+        int length = response.getPayload().size() - this->dataPos;
+        if (length == 0)
         {
           this->res = false;
         }
diff --git a/Managers/MultimediaManager.cpp b/Managers/MultimediaManager.cpp
index 81c5514e..1bbb2c65 100644
--- a/Managers/MultimediaManager.cpp
+++ b/Managers/MultimediaManager.cpp
@@ -112,7 +112,6 @@ bool    MultimediaManager::initICN(const std::string& url)
     {
         pos = pos + 1;
     }
-
     std::string downloadFile(this->downloadPath + url.substr(pos).c_str());
     FILE *fp;
     fp = fopen(downloadFile.c_str(), "w");
@@ -131,6 +130,7 @@ bool    MultimediaManager::initICN(const std::string& url)
     }
     fclose(fp);
     IMPD* mpd = this->manager->Open(const_cast<char*>(downloadFile.c_str()), url);
+    qDebug("downloadFile %s", downloadFile.c_str());
     remove(downloadFile.c_str());
     free(data);
     if(mpd == NULL)
@@ -143,11 +143,10 @@ bool    MultimediaManager::initICN(const std::string& url)
     this->mpdWrapper->updateMPD(mpd);
     for (size_t i = 0; i < this->managerObservers.size(); i++)
         this->managerObservers.at(i)->setMPDWrapper(this->mpdWrapper);
-    if( !strcmp(this->mpdWrapper->getType().c_str(), "static") )
+    if( this->mpdWrapper->getType().compare("static") )
     {
         delete icnConn;
     }
-
     LeaveCriticalSection(&this->monitorMutex);
     return true;
 }
diff --git a/UI/DASHPlayer.cpp b/UI/DASHPlayer.cpp
index c53cbed7..63debc12 100644
--- a/UI/DASHPlayer.cpp
+++ b/UI/DASHPlayer.cpp
@@ -216,7 +216,7 @@ bool DASHPlayer::downloadMPD(const QString &adaptationLogic, bool icn)
 
         this->segmentDuration = this->mpdWrapper->onFirstDownloadMPD(this->gui);
         this->multimediaManager->setSegmentDuration(this->segmentDuration);
-        this->parametersAdaptation->segmentDuration = this->segmentDuration / 1000.0; //to have it in seconds 
+        this->parametersAdaptation->segmentDuration = this->segmentDuration / 1000.0; //to have it in seconds
         this->onSettingsChanged(0,0,0,0,0);
         int j =0;
         std::string temp = adaptationLogic.toStdString();
diff --git a/android/AndroidManifest.xml b/android/AndroidManifest.xml
index aaac206d..f8e00c3a 100644
--- a/android/AndroidManifest.xml
+++ b/android/AndroidManifest.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0"?>
-<manifest package="org.player.viper" xmlns:android="http://schemas.android.com/apk/res/android" android:versionName="1.0" android:versionCode="7" android:installLocation="auto">
+<manifest package="io.fd.viper" xmlns:android="http://schemas.android.com/apk/res/android" android:versionName="1.0" android:versionCode="9" android:installLocation="auto">
     <application android:hardwareAccelerated="true" android:name="org.qtproject.qt5.android.bindings.QtApplication" android:label="Viper" android:icon="@drawable/icon">
-        <activity android:configChanges="orientation|uiMode|screenLayout|screenSize|smallestScreenSize|locale|fontScale|keyboard|keyboardHidden|navigation" android:name="org.player.viper.ViperActivity" android:label="Viper" android:screenOrientation="unspecified" android:launchMode="singleTop">
+        <activity android:configChanges="orientation|uiMode|screenLayout|screenSize|smallestScreenSize|locale|fontScale|keyboard|keyboardHidden|navigation" android:name="io.fd.viper.ViperActivity" android:label="Viper" android:screenOrientation="unspecified" android:launchMode="singleTop">
             <intent-filter>
                 <action android:name="android.intent.action.MAIN"/>
                 <category android:name="android.intent.category.LAUNCHER"/>
@@ -69,7 +69,7 @@
 
     </application>
 
-    <uses-sdk android:minSdkVersion="23" android:targetSdkVersion="23"/>
+    <uses-sdk android:minSdkVersion="23" android:targetSdkVersion="26"/>
     <supports-screens android:largeScreens="true" android:normalScreens="true" android:anyDensity="true" android:smallScreens="true"/>
 
     <!-- The following comment will be replaced upon deployment with default permissions based on the dependencies of the application.
diff --git a/android/gradle.properties b/android/gradle.properties
index 5582cc87..8527bcaf 100644
--- a/android/gradle.properties
+++ b/android/gradle.properties
@@ -3,6 +3,6 @@
 # This file must *NOT* be checked into Version Control Systems,
 # as it contains information specific to your local configuration.
 
-androidBuildToolsVersion=23.0.2
-androidCompileSdkVersion=23
+androidBuildToolsVersion=28.0.3
+androidCompileSdkVersion=28
 buildDir=.build
diff --git a/android/res/drawable-hdpi/icon.png b/android/res/drawable-hdpi/icon.png
old mode 100755
new mode 100644
index ecd7fe96..d84d1df6
Binary files a/android/res/drawable-hdpi/icon.png and b/android/res/drawable-hdpi/icon.png differ
diff --git a/android/res/drawable-ldpi/icon.png b/android/res/drawable-ldpi/icon.png
old mode 100755
new mode 100644
index 8a4388e5..d84d1df6
Binary files a/android/res/drawable-ldpi/icon.png and b/android/res/drawable-ldpi/icon.png differ
diff --git a/android/res/drawable-mdpi/icon.png b/android/res/drawable-mdpi/icon.png
old mode 100755
new mode 100644
index a1519295..812c295f
Binary files a/android/res/drawable-mdpi/icon.png and b/android/res/drawable-mdpi/icon.png differ
diff --git a/android/src/io/fd/viper/ViperActivity.java b/android/src/io/fd/viper/ViperActivity.java
new file mode 100755
index 00000000..0696d0d0
--- /dev/null
+++ b/android/src/io/fd/viper/ViperActivity.java
@@ -0,0 +1,51 @@
+/*
+ * Copyright (c) 2017 Cisco and/or its affiliates.
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at:
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package io.fd.viper;
+import org.qtproject.qt5.android.bindings.QtActivity;
+import android.content.Context;
+import android.content.Intent;
+import android.app.PendingIntent;
+import android.util.Log;
+import android.os.Bundle;
+import android.view.WindowManager;
+import android.content.pm.ActivityInfo;
+
+public class ViperActivity extends QtActivity
+{
+    private final static String TAG = "ViperPlayer";
+    private static String m_request_url;
+    private static ViperActivity m_instance;
+    public ViperActivity() {
+        m_instance = this;
+    }
+    public static String getUrl() {
+        return m_instance.m_request_url;
+    }
+
+    @Override
+    public void onCreate(Bundle savedInstanceState) {
+        super.onCreate(savedInstanceState);
+        getWindow().addFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN);
+        getWindow().addFlags(WindowManager.LayoutParams.FLAG_KEEP_SCREEN_ON);
+        Intent intent = getIntent();
+        String action = intent.getAction();
+
+        if (intent.ACTION_VIEW.equals(action)) {
+            m_request_url = intent.getDataString();
+        }
+        setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_LANDSCAPE);
+    }
+}
diff --git a/android/src/org/player/viper/ViperActivity.java b/android/src/org/player/viper/ViperActivity.java
deleted file mode 100755
index e046d125..00000000
--- a/android/src/org/player/viper/ViperActivity.java
+++ /dev/null
@@ -1,51 +0,0 @@
-/*
- * Copyright (c) 2017 Cisco and/or its affiliates.
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at:
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package org.player.viper;
-import org.qtproject.qt5.android.bindings.QtActivity;
-import android.content.Context;
-import android.content.Intent;
-import android.app.PendingIntent;
-import android.util.Log;
-import android.os.Bundle;
-import android.view.WindowManager;
-import android.content.pm.ActivityInfo;
-
-public class ViperActivity extends QtActivity
-{
-    private final static String TAG = "ViperPlayer";
-    private static String m_request_url;
-    private static ViperActivity m_instance;
-    public ViperActivity() {
-        m_instance = this;
-    }
-    public static String getUrl() {
-        return m_instance.m_request_url;
-    }
-
-    @Override
-    public void onCreate(Bundle savedInstanceState) {
-        super.onCreate(savedInstanceState);
-        getWindow().addFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN);
-        getWindow().addFlags(WindowManager.LayoutParams.FLAG_KEEP_SCREEN_ON);
-        Intent intent = getIntent();
-        String action = intent.getAction();
-
-        if (intent.ACTION_VIEW.equals(action)) {
-            m_request_url = intent.getDataString();
-        }
-        setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_LANDSCAPE);
-    }
-}
diff --git a/libdash/CMakeLists.txt b/libdash/CMakeLists.txt
index 1ecf7706..0e47c993 100644
--- a/libdash/CMakeLists.txt
+++ b/libdash/CMakeLists.txt
@@ -1,33 +1,24 @@
 cmake_minimum_required(VERSION 3.5)
 project(libdash)
-
-
+set(CMAKE_CXX_STANDARD 14)
 if(ANDROID_API)
+    set(CMAKE_CXX_FLAGS "${CMAKE_C_FLAGS} ${ANDROID_C_FLAGS} -std=c++0x -g -DLOG_BUILD -stdlib=libc++")
     set(CMAKE_FIND_ROOT_PATH "$ENV{CCNX_HOME};${CMAKE_FIND_ROOT_PATH}")
-	#include_directories(${ANDROID_NDK}/sources/cxx-stl/gnu-libstdc++/4.9/include)
-	#include_directories(${ANDROID_NDK}/sources/cxx-stl/gnu-libstdc++/4.9/libs/armeabi/include)
-	message("ciao!!!")
-	option(ANDROID "Build CP api based on ANDROID log lib" ON)
-	find_package(LibXml2 REQUIRED)
-	find_package(ZLIB REQUIRED)
-	find_package(CURL REQUIRED)
-	#set(LIBXML2_INCLUDE_DIR "$ENV{DISTILLERY_ROOT_DIR}/external/libxml2_android/jni/libxml2/include")
-	#set(LIBXML2_LIBRARIES "$ENV{DISTILLERY_ROOT_DIR}/usr/lib/libxml2.a")
-	#set(CURL_INCLUDE_DIRS "$ENV{DISTILLERY_ROOT_DIR}/external/libcurl_android/jni/libcurl/include")
-	#set(CURL_LIBRARIES "$ENV{DISTILLERY_ROOT_DIR}/usr/lib/libcurl.a" "$ENV{DISTILLERY_ROOT_DIR}/usr/lib/libcurl-library.a")
-	#set(ANDROID_LIBRARIES "$ENV{NDK}/sources/cxx-stl/gnu-libstdc++/4.9/libs/$ENV{ABI}/libgnustl_shared.so")
-	set(CMAKE_CXX_FLAGS "${CMAKE_C_FLAGS} ${ANDROID_C_FLAGS} -std=c++0x -g -DLOG_BUILD -DANDROID_STL=c++_shared")
+    find_package(LibXml2 REQUIRED)
+    find_package(ZLIB REQUIRED)
+    find_package(CURL REQUIRED)
+    find_package(OpenSSL REQUIRED)
 elseif (COMPILE_FOR_IOS)
-	set(CMAKE_CXX_FLAGS "-std=c++0x -g -DLOG_BUILD")
-	find_package(LibXml2 REQUIRED)
-	find_host_package(OpenSSL REQUIRED)
-	find_package(ZLIB REQUIRED)
-	find_host_package(CURL REQUIRED)
+    set(CMAKE_CXX_FLAGS "-std=c++0x -g -DLOG_BUILD")
+    find_package(LibXml2 REQUIRED)
+    find_host_package(OpenSSL REQUIRED)
+    find_package(ZLIB REQUIRED)
+    find_host_package(CURL REQUIRED)
 else ()
-	set(CMAKE_CXX_FLAGS "-std=c++0x -g -DLOG_BUILD")
-	find_package(LibXml2 REQUIRED)
-	find_package(ZLIB REQUIRED)
-	find_package(CURL REQUIRED)
+    set(CMAKE_CXX_FLAGS "-std=c++0x -g -DLOG_BUILD")
+    find_package(LibXml2 REQUIRED)
+    find_package(ZLIB REQUIRED)
+    find_package(CURL REQUIRED)
 endif (ANDROID_API)
 set(DEBUG_BUILD FALSE)
 set(HEADER_FILES
@@ -190,14 +181,13 @@ include_directories(${WLIB_INCLUDE_DIRS})
 include_directories(${CURL_INCLUDE_DIRS})
 include_directories(include)
 
-if (COMPILE_FOR_IOS OR COMPILE_FOR_IOS_SIMULATOR)
+if (COMPILE_FOR_IOS OR COMPILE_FOR_IOS_SIMULATOR OR ANDROID_API)
   add_library(dash STATIC ${SOURCE_FILES})
 else()
   add_library(dash SHARED ${SOURCE_FILES})
 endif()
 set_target_properties(dash PROPERTIES LINKER_LANGUAGE CXX)
-target_link_libraries(dash ${CURL_LIBRARIES} ${ZLIB_LIBRARIES} ${CURL_LIBRARIES} ${LIBXML2_LIBRARIES} ${CURL_LIBRARIES} ${ANDROID_LIBRARIES} ${CURL_LIBRARIES})
-message("libxml ${LIBXML2_LIBRARIES} ${LIBXML2_INCLUDE_DIR}")
+target_link_libraries(dash ${CURL_LIBRARIES} ${ZLIB_LIBRARIES} ${CURL_LIBRARIES} ${LIBXML2_LIBRARIES} ${CURL_LIBRARIES} ${ANDROID_LIBRARIES} ${CURL_LIBRARIES}   ${OPENSSL_LIBRARIES})
 install(TARGETS dash DESTINATION ${CMAKE_INSTALL_PREFIX}/lib PERMISSIONS OWNER_EXECUTE OWNER_WRITE OWNER_READ GROUP_EXECUTE GROUP_READ WORLD_EXECUTE WORLD_READ COMPONENT library)
 install(FILES ${HEADER_FILES} DESTINATION ${CMAKE_INSTALL_PREFIX}/include/libdash COMPONENT headers)
 
diff --git a/main.cpp b/main.cpp
index a1f2969e..76fbe0cf 100644
--- a/main.cpp
+++ b/main.cpp
@@ -27,6 +27,7 @@
 #include <QtAV/QtAV.h>
 #ifdef Q_OS_ANDROID
 #include <QAndroidJniObject>
+#include <QtAndroid>
 #endif
 #include "Common/QtQuick2ApplicationViewer.h"
 #include "../Common/Common.h"
@@ -159,7 +160,8 @@ int main(int argc, char *argv[])
         viewer.setY(op.value().toInt());
     if (options.value(QStringLiteral("fullscreen")).toBool())
         viewer.showFullScreen();
-    viewer.setTitle(QStringLiteral("Viper"));
+    viewer.setTitle(QStringLiteral("Viper")); 
+
 #if 1
     QString json = app.arguments().join(QStringLiteral("\",\""));
     json.prepend(QLatin1String("[\"")).append(QLatin1String("\"]"));
@@ -171,8 +173,19 @@ int main(int argc, char *argv[])
     qApp->installEventFilter(ae);
     QString file;
 #ifdef Q_OS_ANDROID
-    engine->rootContext()->setContextProperty("platform",  1);
-    file = QAndroidJniObject::callStaticObjectMethod("org.viper.com.ViperActivity"
+    const QVector<QString> permissions({"android.permission.INTERNET",
+                                        "android.permission.WRITE_EXTERNAL_STORAGE",
+                                        "android.permission.READ_EXTERNAL_STORAGE"});
+    for(const QString &permission : permissions){
+            auto result = QtAndroid::checkPermission(permission);
+            if(result == QtAndroid::PermissionResult::Denied){
+                auto resultHash = QtAndroid::requestPermissionsSync(QStringList({permission}));
+                if(resultHash[permission] == QtAndroid::PermissionResult::Denied)
+                    return 0;
+            }
+        }
+
+    file = QAndroidJniObject::callStaticObjectMethod("io.fd.viper.ViperActivity"
                                                      , "getUrl"
                                                      , "()Ljava/lang/String;")
             .toString();
diff --git a/qml/Viper/Options.qml b/qml/Viper/Options.qml
index 3f8d7ab6..64d6f45b 100755
--- a/qml/Viper/Options.qml
+++ b/qml/Viper/Options.qml
@@ -449,7 +449,6 @@ Rectangle {
             anchors.top: parent.top
             anchors.right: textInputv6FirstWord.left
             anchors.rightMargin: Utils.scaled(5)
-            anchors.topMargin: (textInputv6Prefix.height - height)/2
             font.bold: true
             font.pixelSize: Utils.scaled(10);
         }
diff --git a/viper.pro b/viper.pro
index 4290c7e1..0d9e0e70 100644
--- a/viper.pro
+++ b/viper.pro
@@ -21,8 +21,7 @@ android {
 CONFIG -= release
 CONFIG += debug
 CONFIG += c++14
-QMAKE_CXXFLAGS += -D__STDC_CONSTANT_MACROS -D__STDC_LIMIT_MACROS -DANDROID_STL=c++_static
-QMAKE_CXXFLAGS += -std=c++14 -g -fpermissive -DASIO_STANDALONE=1
+QMAKE_CXXFLAGS += -std=c++14 -g -fpermissive -DASIO_STANDALONE=1 -stdlib=libc++ -DQTC_ENABLE_CLANG_LIBTOOLING=ON
 # Add more folders to ship with the application, here
 folder_01.source = qml/Viper
 folder_01.target = qml
@@ -306,7 +305,7 @@ macx:!ios {
 SOURCES *= main.cpp
 android {
     DISTFILES += \
-    android/src/org/player/viper/ViperActivity.java \
+    android/src/io/fd/viper/ViperActivity.java \
     android/gradle/wrapper/gradle-wrapper.jar \
     android/AndroidManifest.xml \
     android/res/values/libs.xml \
@@ -319,8 +318,16 @@ android {
     INCLUDEPATH += $$(DISTILLARY_INSTALLATION_PATH)/include
     INCLUDEPATH += $$(DISTILLARY_INSTALLATION_PATH)/include/libdash
     equals(TRANSPORT_LIBRARY, "HICNET") {
-        LIBS += -L"$$(DISTILLARY_INSTALLATION_PATH)/lib" -lhicnet -ljsoncpp -ldash -lcurl  -lxml2 -lccnx_common -lparc -lhicntransport -lhicn -levent -lssl -lcrypto -lavcodec -lavutil -lavformat
+        LIBS += -L"$$(DISTILLARY_INSTALLATION_PATH)/lib" -ldash -lhicntransport  -lhicn -lparc -lavcodec -lavutil -lavformat -lswresample -lcurl -lxml2 -lssl -lcrypto -lQtAV
         DEFINES += "HICNET=ON"
+        ANDROID_EXTRA_LIBS += $$(DISTILLARY_INSTALLATION_PATH)/lib/libswresample.so
+        ANDROID_EXTRA_LIBS += $$(DISTILLARY_INSTALLATION_PATH)/lib/libavresample.so
+        ANDROID_EXTRA_LIBS += $$(DISTILLARY_INSTALLATION_PATH)/lib/libavdevice.so
+        ANDROID_EXTRA_LIBS += $$(DISTILLARY_INSTALLATION_PATH)/lib/libavfilter.so
+        ANDROID_EXTRA_LIBS += $$(DISTILLARY_INSTALLATION_PATH)/lib/libavcodec.so
+        ANDROID_EXTRA_LIBS += $$(DISTILLARY_INSTALLATION_PATH)/lib/libavformat.so
+        ANDROID_EXTRA_LIBS += $$(DISTILLARY_INSTALLATION_PATH)/lib/libswscale.so
+        ANDROID_EXTRA_LIBS += $$(DISTILLARY_INSTALLATION_PATH)/lib/libavutil.so
     } else {
         LIBS += -L"$$(DISTILLARY_INSTALLATION_PATH)/lib" -licnet -ldash -lcurl  -lxml2 -lccnx_api_portal -lccnx_transport_rta -lccnx_api_control -lccnx_api_notify -lccnx_common -lparc -llongbow -llongbow-ansiterm -llongbow-textplain -levent -lssl -lcrypto -lavcodec -lavutil -lavformat   -lboost_system
         DEFINES += "ICNET=ON"
@@ -336,4 +343,4 @@ DISTFILES += \
     android/res/drawable-mdpi/icon.png \
     android/gradle.properties \
     android/src/org/qtav/qmlplayer/ViperActivity.java \
-    android/src/org/player/viper/ViperActivity.java
+    android/src/io/fd/viper/ViperActivity.java
-- 
cgit 1.2.3-korg