[fenix] Update IntentProcessors after breaking API change

pull/600/head
Christian Sadilek 4 years ago
parent 042f7dd697
commit 61e10d8561

@ -68,7 +68,9 @@ class Components(private val context: Context) {
IntentProcessors( IntentProcessors(
context, context,
core.sessionManager, core.sessionManager,
core.store,
useCases.sessionUseCases, useCases.sessionUseCases,
useCases.tabsUseCases,
useCases.searchUseCases, useCases.searchUseCases,
core.relationChecker, core.relationChecker,
core.customTabsStore, core.customTabsStore,

@ -6,6 +6,7 @@ package org.mozilla.fenix.components
import android.content.Context import android.content.Context
import mozilla.components.browser.session.SessionManager import mozilla.components.browser.session.SessionManager
import mozilla.components.browser.state.store.BrowserStore
import mozilla.components.feature.customtabs.CustomTabIntentProcessor import mozilla.components.feature.customtabs.CustomTabIntentProcessor
import mozilla.components.feature.customtabs.store.CustomTabsServiceStore import mozilla.components.feature.customtabs.store.CustomTabsServiceStore
import mozilla.components.feature.intent.processing.TabIntentProcessor import mozilla.components.feature.intent.processing.TabIntentProcessor
@ -14,6 +15,7 @@ import mozilla.components.feature.pwa.intent.TrustedWebActivityIntentProcessor
import mozilla.components.feature.pwa.intent.WebAppIntentProcessor import mozilla.components.feature.pwa.intent.WebAppIntentProcessor
import mozilla.components.feature.search.SearchUseCases import mozilla.components.feature.search.SearchUseCases
import mozilla.components.feature.session.SessionUseCases import mozilla.components.feature.session.SessionUseCases
import mozilla.components.feature.tabs.TabsUseCases
import mozilla.components.service.digitalassetlinks.RelationChecker import mozilla.components.service.digitalassetlinks.RelationChecker
import mozilla.components.support.migration.MigrationIntentProcessor import mozilla.components.support.migration.MigrationIntentProcessor
import mozilla.components.support.migration.state.MigrationStore import mozilla.components.support.migration.state.MigrationStore
@ -30,7 +32,9 @@ import org.mozilla.fenix.utils.Mockable
class IntentProcessors( class IntentProcessors(
private val context: Context, private val context: Context,
private val sessionManager: SessionManager, private val sessionManager: SessionManager,
private val store: BrowserStore,
private val sessionUseCases: SessionUseCases, private val sessionUseCases: SessionUseCases,
private val tabsUseCases: TabsUseCases,
private val searchUseCases: SearchUseCases, private val searchUseCases: SearchUseCases,
private val relationChecker: RelationChecker, private val relationChecker: RelationChecker,
private val customTabsStore: CustomTabsServiceStore, private val customTabsStore: CustomTabsServiceStore,
@ -62,13 +66,12 @@ class IntentProcessors(
val externalAppIntentProcessors by lazyMonitored { val externalAppIntentProcessors by lazyMonitored {
listOf( listOf(
TrustedWebActivityIntentProcessor( TrustedWebActivityIntentProcessor(
sessionManager = sessionManager, addNewTabUseCase = tabsUseCases.addTab,
loadUrlUseCase = sessionUseCases.loadUrl,
packageManager = context.packageManager, packageManager = context.packageManager,
relationChecker = relationChecker, relationChecker = relationChecker,
store = customTabsStore store = customTabsStore
), ),
WebAppIntentProcessor(sessionManager, sessionUseCases.loadUrl, manifestStorage), WebAppIntentProcessor(store, tabsUseCases.addTab, sessionUseCases.loadUrl, manifestStorage),
FennecWebAppIntentProcessor(context, sessionManager, sessionUseCases.loadUrl, manifestStorage) FennecWebAppIntentProcessor(context, sessionManager, sessionUseCases.loadUrl, manifestStorage)
) )
} }

@ -69,7 +69,7 @@ class FennecWebAppIntentProcessor(
webAppManifest?.toCustomTabConfig() ?: createFallbackCustomTabConfig() webAppManifest?.toCustomTabConfig() ?: createFallbackCustomTabConfig()
sessionManager.add(session) sessionManager.add(session)
loadUrlUseCase(url, session, EngineSession.LoadUrlFlags.external()) loadUrlUseCase(url, session.id, EngineSession.LoadUrlFlags.external())
intent.putSessionId(session.id) intent.putSessionId(session.id)

@ -44,7 +44,7 @@ class FennecBookmarkShortcutsIntentProcessor(
val session = Session(url, private = false, source = SessionState.Source.HOME_SCREEN) val session = Session(url, private = false, source = SessionState.Source.HOME_SCREEN)
sessionManager.add(session, selected = true) sessionManager.add(session, selected = true)
loadUrlUseCase(url, session, EngineSession.LoadUrlFlags.external()) loadUrlUseCase(url, session.id, EngineSession.LoadUrlFlags.external())
intent.action = ACTION_VIEW intent.action = ACTION_VIEW
intent.putSessionId(session.id) intent.putSessionId(session.id)

@ -66,7 +66,7 @@ class FennecBookmarkShortcutsIntentProcessorTest {
assertEquals(expectedSession.id, fennecShortcutsIntent.getSessionId()) assertEquals(expectedSession.id, fennecShortcutsIntent.getSessionId())
verifyAll { verifyAll {
sessionManager.add(expectedSession, true) sessionManager.add(expectedSession, true)
loadUrlUseCase(testUrl, expectedSession, EngineSession.LoadUrlFlags.external()) loadUrlUseCase(testUrl, expectedSession.id, EngineSession.LoadUrlFlags.external())
} }
} }
} }

Loading…
Cancel
Save