aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/ui/src/main/java/com/wireguard (follow)
Commit message (Collapse)AuthorAgeFilesLines
* ui: queue up tunnel mutating on activity scope instead of fragment scopeJason A. Donenfeld2020-09-266-69/+80
| | | | | | | | | | | | Fragment scopes get cancelled when the fragment goes away, but we don't actually want to cancel an in-flight transition in that case. Also, before when the fragment would cancel, there'd be an exception, and the exception handler would call Fragment::getString, which in turn called requireContext, which caused an exception. Work around this by using the `activity ?: Application.get()` idiom to always have a context for strings and toasts. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: print proper exception trace from log viewerJason A. Donenfeld2020-09-261-1/+3
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: request intent permissions from hidden activityJason A. Donenfeld2020-09-261-2/+19
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: handle going up directories betterJason A. Donenfeld2020-09-241-25/+40
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: show volume descriptions for file pickerJason A. Donenfeld2020-09-241-4/+4
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: account for null data in callbackJason A. Donenfeld2020-09-241-0/+1
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: cleanup code after churnJason A. Donenfeld2020-09-232-16/+2
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: use system picker for API 29+Jason A. Donenfeld2020-09-231-10/+24
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: use our own file pickerJason A. Donenfeld2020-09-231-19/+166
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: escape deletion view with back buttonJason A. Donenfeld2020-09-231-0/+7
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: select first item after toggling deletion modeJason A. Donenfeld2020-09-221-0/+3
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: hook up isFocused as observable propertyJason A. Donenfeld2020-09-221-0/+4
| | | | | | | | This is kind of ridiculous, since the items own state should clearly be queryable, but it doesn't appear to be the case here, so just shuffle it around into kotlin and back. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: add text when there are no tunnelsJason A. Donenfeld2020-09-221-0/+2
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: select the right thing on loadJason A. Donenfeld2020-09-221-2/+9
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: set layout manager from XMLHarsh Shandilya2020-09-221-2/+0
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* tv: do themingHarsh Shandilya2020-09-221-0/+2
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* tv: account for broken TVs with no file pickerJason A. Donenfeld2020-09-221-1/+7
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: do not redisplay stats when deletingJason A. Donenfeld2020-09-221-3/+3
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: add ugly deletion modeJason A. Donenfeld2020-09-221-8/+26
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: wire in statsJason A. Donenfeld2020-09-223-12/+60
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: wire up tunnel start/stopJason A. Donenfeld2020-09-221-0/+50
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: begin to wire up databindingsJason A. Donenfeld2020-09-221-13/+9
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: abstract out tunnel importingJason A. Donenfeld2020-09-225-267/+175
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* tv: initial draft of Android TV supportHarsh Shandilya2020-09-223-11/+175
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* TunnelManager: catch exception in intent receiverJason A. Donenfeld2020-09-211-1/+7
| | | | | | | | | | | | | | | | | | | java.lang.IllegalStateException: at android.app.ContextImpl.startServiceCommon (ContextImpl.java:1720) at android.app.ContextImpl.startService (ContextImpl.java:1675) at android.content.ContextWrapper.startService (ContextWrapper.java:669) at com.wireguard.android.backend.GoBackend.startVpnService (GoBackend.java:4) at com.wireguard.android.backend.GoBackend.setStateInternal (GoBackend.java:4) at com.wireguard.android.backend.GoBackend.setState (GoBackend.java:2) at com.wireguard.android.model.TunnelManager$setTunnelState$2$1.invokeSuspend (TunnelManager.java:6) at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith (BaseContinuationImpl.java:2) at kotlinx.coroutines.DispatchedTask.run (DispatchedTask.java:2) at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely (CoroutineScheduler.java) at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask (CoroutineScheduler.java:7) at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker (CoroutineScheduler.java:7) at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run (CoroutineScheduler.java:7) Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* TunnelEditorFragment: do not assume a contextJason A. Donenfeld2020-09-201-2/+2
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* TunnelListFragment: do not assume binding always existsJason A. Donenfeld2020-09-201-4/+4
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* TunnelDetailFragment: use kotlin coroutine for timer and rework nullabilityJason A. Donenfeld2020-09-201-43/+38
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* BaseFragment: avoid using requireContext() in permission result callbackJason A. Donenfeld2020-09-201-2/+2
| | | | | | | | | | | | | | | | | | | | | | | java.lang.IllegalStateException: at androidx.fragment.app.Fragment.requireContext (Fragment.java:17) at com.wireguard.android.fragment.BaseFragment$setTunnelStateWithPermissionsResult$1.invokeSuspend (BaseFragment.java:4) at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith (BaseContinuationImpl.java:2) at kotlinx.coroutines.UndispatchedCoroutine.afterResume (UndispatchedCoroutine.java:19) at kotlinx.coroutines.AbstractCoroutine.resumeWith (AbstractCoroutine.java:13) at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith (BaseContinuationImpl.java:2) at kotlinx.coroutines.UndispatchedCoroutine.afterResume (UndispatchedCoroutine.java:19) at kotlinx.coroutines.AbstractCoroutine.resumeWith (AbstractCoroutine.java:13) at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith (BaseContinuationImpl.java:2) at kotlinx.coroutines.DispatchedTask.run (DispatchedTask.java:2) at android.os.Handler.handleCallback (Handler.java:790) at android.os.Handler.dispatchMessage (Handler.java:99) at android.os.Looper.loop (Looper.java:164) at android.app.ActivityThread.main (ActivityThread.java:7025) at java.lang.reflect.Method.invoke (Method.java) at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run (RuntimeInit.java:441) at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:1408) Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* AddTunnelsSheet: disable qrcode scanning if no cameraJason A. Donenfeld2020-09-201-1/+8
| | | | | | Part of the enhancements for Android TV. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* SettingsActivity: account for module present but no rootJason A. Donenfeld2020-09-181-0/+9
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: move to Jetpack DataStore instead of SharedPrefsJason A. Donenfeld2020-09-188-101/+302
| | | | | | | Hopefully PreferencesPreferenceDataStore gets to go away sometime down the line. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: reformat codeJason A. Donenfeld2020-09-183-5/+2
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: retire EdgeToEdgeHarsh Shandilya2020-09-176-101/+1
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* ui: replace deprecated onActivityCreated with onViewCreatedHarsh Shandilya2020-09-171-2/+2
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* ui: refactor AddTunnelsSheet's selection communicationHarsh Shandilya2020-09-173-42/+43
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* ui: refactor AppListDialogFragment's selection communicationHarsh Shandilya2020-09-172-33/+31
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* activityx: use contracts more and refineJason A. Donenfeld2020-09-172-26/+22
| | | | | | | This is the beginning; there are still many of the old API's callsites to convert. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* DownloadsFileSaver: encapsulate permission checksJason A. Donenfeld2020-09-174-90/+67
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* LogViewerActivity: simplify scopingJason A. Donenfeld2020-09-161-23/+19
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ZipExporterPreference: don't ask for storage permissions on newer androidJason A. Donenfeld2020-09-162-4/+11
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: resolve getColor deprecation in LogViewerActivityHarsh Shandilya2020-09-161-14/+6
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* LogViewerActivity: destroy process when coroutine scope is cancelledJason A. Donenfeld2020-09-161-37/+42
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* ui: await activity creation to change selected tunnelHarsh Shandilya2020-09-161-2/+1
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* ui: replace GlobalScope with a hand-rolled CoroutineScopeHarsh Shandilya2020-09-166-16/+25
| | | | | | | | | | | | | | | | | GlobalScope has numerous problems[1] that make it unfit for use in most applications and making it behave correctly requires an excessive amount of verbosity that's alleviated simply by using any other scope. Since we run multiple operations in the context of the application's lifecycle, introduce a new scope that is created when our application is, and cancelled upon its termination. While at it, make the scope default to Dispatchers.IO to reduce pressure on the UI event loop. Tasks requiring access to the UI thread appropriately switch context making the change completely safe. 1: https://medium.com/@elizarov/the-reason-to-avoid-globalscope-835337445abc Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* ConfigNamingDialogFragment: fix focus request for config naming dialogHarsh Shandilya2020-09-161-3/+17
| | | | Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
* TunnelEditorFragment: move backwards using fragment manager instead of hackJason A. Donenfeld2020-09-162-7/+5
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* TunnelEditorFragment: avoid extra trip through event loopJason A. Donenfeld2020-09-161-7/+6
| | | | | | | | | onSelectedTunnelChanged is already queueing us to Dispatchers.Main (rather than Dispatchers.Main.immediate, which would crash, but why?), so avoid the extra trip through the event loop by toggling the selected tunnel right away. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* Extensions: use more idiomatic kotlinJason A. Donenfeld2020-09-161-2/+2
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* global: lint codebase with recent changesJason A. Donenfeld2020-09-165-4/+8
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>