Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit da82e0cf authored by Jorge Gil's avatar Jorge Gil
Browse files

Fix Shell's injection of MainCoroutineDispatcher 

Since the return type of provideMainDispatcher() and
provideBackgroundDispatcher() where changed to return a
MainCoroutineDispatcher, the injection points of these also need be
updated or dagger can't provide them.
Since the CoroutineScopes weren't used yet there's no build errors
until another class actually tries to inject either the dispatchers
or the scopes.

Bug: None
Test: inject either scope and check build succeeds
Flag: EXEMPT bugfix
Change-Id: I7685259683134b6498ac7095dba06797a3e9c44a
parent 4688fd89

libs/WindowManager/Shell/src/com/android/wm/shell/dagger/WMShellCoroutinesModule.kt

+3 −3
Original line number Diff line number Diff line
@@ -61,20 +61,20 @@ class WMShellCoroutinesModule { 
  @WMSingleton

  @ShellMainThread

  fun provideApplicationScope(

      @ShellMainThread applicationDispatcher: CoroutineDispatcher,

      @ShellMainThread applicationDispatcher: MainCoroutineDispatcher,

  ): CoroutineScope = CoroutineScope(applicationDispatcher)



  @Provides

  @WMSingleton

  @ShellBackgroundThread

  fun provideBackgroundCoroutineScope(

      @ShellBackgroundThread backgroundDispatcher: CoroutineDispatcher,

      @ShellBackgroundThread backgroundDispatcher: MainCoroutineDispatcher,

  ): CoroutineScope = CoroutineScope(backgroundDispatcher)



  @Provides

  @WMSingleton

  @ShellBackgroundThread

  fun provideBackgroundCoroutineContext(

      @ShellBackgroundThread backgroundDispatcher: CoroutineDispatcher

      @ShellBackgroundThread backgroundDispatcher: MainCoroutineDispatcher

  ): CoroutineContext = backgroundDispatcher + SupervisorJob()

}