Skip to content

Commit 0a0bd95

Browse files
committed
Merge branch 'hotfix/4.0.10'
2 parents 7d85c20 + f4d3320 commit 0a0bd95

File tree

5 files changed

+8
-9
lines changed

5 files changed

+8
-9
lines changed

app/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ apply plugin: 'kotlin-kapt'
66
apply from: '../versioning.gradle'
77

88
ext {
9-
VERSION_NAME = "4.0.9"
9+
VERSION_NAME = "4.0.10"
1010
}
1111

1212
android {

app/src/main/java/com/duckduckgo/app/browser/BrowserActivity.kt

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@ import android.view.KeyEvent.KEYCODE_ENTER
3232
import android.view.Menu
3333
import android.view.MenuItem
3434
import android.view.View
35-
import android.view.View.FOCUSABLE
3635
import android.view.inputmethod.EditorInfo.IME_ACTION_DONE
3736
import android.webkit.CookieManager
3837
import android.webkit.WebSettings.MIXED_CONTENT_COMPATIBILITY_MODE
@@ -107,14 +106,14 @@ class BrowserActivity : DuckDuckGoActivity() {
107106
popupMenu = BrowserPopupMenu(layoutInflater)
108107
}
109108

110-
// inspired by https://stackoverflow.com/a/8011027/73479
111109
private fun createWebView() {
112-
webView = NestedWebView(this.applicationContext)
110+
webView = NestedWebView(this)
113111
webView.gone()
114112
webView.isFocusableInTouchMode = true
115113
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
116-
webView.focusable = FOCUSABLE
114+
webView.focusable = View.FOCUSABLE
117115
}
116+
118117
webViewContainer.addView(webView)
119118
}
120119

app/src/main/java/com/duckduckgo/app/global/DuckDuckGoApplication.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ class DuckDuckGoApplication : HasActivityInjector, HasServiceInjector, Applicati
116116
.doAfterTerminate({
117117
appConfigurationSyncer.scheduleRegularSync(this)
118118
})
119-
.subscribe({}, { Timber.w(it, "Failed to download initial app configuration") })
119+
.subscribe({}, { Timber.w("Failed to download initial app configuration ${it.localizedMessage}") })
120120
}
121121

122122
override fun activityInjector(): AndroidInjector<Activity> = activityInjector

app/src/main/java/com/duckduckgo/app/job/AppConfigurationDownloader.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ class AppConfigurationDownloader @Inject constructor(
3838
val disconnectDownload = trackerDataDownloader.downloadList(Client.ClientName.DISCONNECT)
3939
val httpsUpgradeDownload = httpsUpgradeListDownloader.downloadList()
4040

41-
return Completable.merge(listOf(
41+
return Completable.mergeDelayError(listOf(
4242
easyListDownload.subscribeOn(Schedulers.io()),
4343
easyPrivacyDownload.subscribeOn(Schedulers.io()),
4444
trackersWhitelist.subscribeOn(Schedulers.io()),

app/src/main/java/com/duckduckgo/app/job/AppConfigurationJobService.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,13 +40,13 @@ class AppConfigurationJobService : JobService() {
4040
override fun onStartJob(params: JobParameters?): Boolean {
4141
Timber.i("onStartJob")
4242

43-
appConfigurationDownloader.downloadTask()
43+
downloadTask= appConfigurationDownloader.downloadTask()
4444
.subscribeOn(Schedulers.io())
4545
.subscribe({
4646
Timber.i("Successfully downloaded all data")
4747
jobFinishedSuccessfully(params)
4848
}, {
49-
Timber.w(it, "Failed to download app configuration")
49+
Timber.w("Failed to download app configuration ${it.localizedMessage}")
5050
jobFinishedFailed(params)
5151
})
5252

0 commit comments

Comments
 (0)