Enhancement/video (#669)v1.8.0

* Test stricter video extractor

* Mess around with mutations

* Revert back to strict click listener

* Update theme

* Update dependencies

* Update theme

* Create pip toggle

* Add pip disabler

* Update theme

* Update theme

1 files changed, 10 insertions, 8 deletions

diff --git a/app/src/main/kotlin/com/pitchedapps/frost/web/FrostJSI.kt b/app/src/main/kotlin/com/pitchedapps/frost/web/FrostJSI.kt
index d735fd50..4a3fa2a2 100644
--- a/app/src/main/kotlin/com/pitchedapps/frost/web/FrostJSI.kt
+++ b/app/src/main/kotlin/com/pitchedapps/frost/web/FrostJSI.kt
@@ -26,17 +26,19 @@ class FrostJSI(val web: FrostWebView) {
      * or {@code false} otherwise, meaning the event should be propagated
      */
     @JavascriptInterface
-    fun loadUrl(url: String?): Boolean
-            = if (url == null) false else web.requestWebOverlay(url)
+    fun loadUrl(url: String?): Boolean = if (url == null) false else web.requestWebOverlay(url)
 
     @JavascriptInterface
-    fun loadVideo(url: String?, isGif: Boolean) {
-        if (url != null)
-            web.post {
-                (context as? VideoViewHolder)?.showVideo(url, isGif)
-                        ?: L.d { "Could not load video; contract not implemented" }
+    fun loadVideo(url: String?, isGif: Boolean): Boolean =
+            if (url != null && Prefs.enablePip) {
+                web.post {
+                    (context as? VideoViewHolder)?.showVideo(url, isGif)
+                            ?: L.e { "Could not load video; contract not implemented" }
+                }
+                true
+            } else {
+                false
             }
-    }
 
     @JavascriptInterface
     fun reloadBaseUrl(animate: Boolean) {