diff options
author | Allan Wang <me@allanwang.ca> | 2018-12-28 22:15:08 -0500 |
---|---|---|
committer | Allan Wang <me@allanwang.ca> | 2018-12-28 22:15:08 -0500 |
commit | 7f545a19dcb52073bc22d2bb59e6cb4beecac4a6 (patch) | |
tree | 19f1458e408da149291ea7260b2196576ece0de9 /app/src/main/kotlin/com/pitchedapps/frost/web/FrostUrlOverlayValidator.kt | |
parent | 9c3d7c8b6cca17dc10fc310d41e547d1fe1725ea (diff) | |
download | frost-7f545a19dcb52073bc22d2bb59e6cb4beecac4a6.tar.gz frost-7f545a19dcb52073bc22d2bb59e6cb4beecac4a6.tar.bz2 frost-7f545a19dcb52073bc22d2bb59e6cb4beecac4a6.zip |
Switch default user agent to android, resolves #1245, #1229
Diffstat (limited to 'app/src/main/kotlin/com/pitchedapps/frost/web/FrostUrlOverlayValidator.kt')
-rw-r--r-- | app/src/main/kotlin/com/pitchedapps/frost/web/FrostUrlOverlayValidator.kt | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/app/src/main/kotlin/com/pitchedapps/frost/web/FrostUrlOverlayValidator.kt b/app/src/main/kotlin/com/pitchedapps/frost/web/FrostUrlOverlayValidator.kt index e2d294f7..490c02e5 100644 --- a/app/src/main/kotlin/com/pitchedapps/frost/web/FrostUrlOverlayValidator.kt +++ b/app/src/main/kotlin/com/pitchedapps/frost/web/FrostUrlOverlayValidator.kt @@ -84,7 +84,7 @@ fun FrostWebView.requestWebOverlay(url: String): Boolean { context.launchWebOverlayBasic(url) return true } - if (context is WebOverlayBasicActivity && !shouldUseBasic) { + if (userAgentString == USER_AGENT_BASIC && !shouldUseBasic) { L.i { "Switch from basic agent" } context.launchWebOverlay(url) return true @@ -105,10 +105,9 @@ val messageWhitelist: Set<String> = .mapTo(mutableSetOf(), FbItem::url) val String.shouldUseBasicAgent: Boolean - get() { - if (contains("story.php")) // do not use basic for comment section - return false - if (contains("/events/")) // do not use for events (namely the map) - return false - return true // use for everything else + get() = when { + contains("story.php") -> false // do not use basic for comment section + contains("/events/") -> false // do not use for events (namely the map) + contains("/messages") -> true // must use for messages + else -> false // default to normal user agent } |