Extract the code responsible for navigating within the custom tab activity (handling back button, close button, open in browser menu item). This class perhaps should also have a navigate() method for navigation to a new url, which will internally use CustomTabActivityTabController (https://crbug.com/1382496).
See CustomTabNavigationController.java here for a rough idea how this class may look like: https://chromium-review.googlesource.com/c/chromium/src/+/1382496
Replace usage of CustomTabActivity in DynamicModuleCoordinator with usage of CustomTabNavigationController.
Ensure the new class is unit-testable, write a few tests.
This is a part of CustomTabActivity refactoring plan described in
https://docs.google.com/document/d/1E9u-9eSFjLwVnUUIekhurmKk-8vsSDxek7cZiFd6ToY
Extract the code responsible for navigating within the custom tab activity (handling back button, close button, open in browser menu item). This class perhaps should also have a navigate() method for navigation to a new url, which will internally use CustomTabActivityTabController (https://crbug.com/1382496).
See CustomTabNavigationController.java in https://crrev.com/1371805 for a rough idea how this class may look like.
Replace usage of CustomTabActivity in DynamicModuleCoordinator with usage of CustomTabNavigationController.
Ensure the new class is unit-testable, write a few tests.
This is a part of CustomTabActivity refactoring plan described in
https://docs.google.com/document/d/1E9u-9eSFjLwVnUUIekhurmKk-8vsSDxek7cZiFd6ToY
Comment 1 by pshmakov@google.com
, Dec 19