We need an intrumentation test to cover this feature
This the next browser test that we could like to add in the ServicificationBackgroundServiceTest:)
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/698f5fb8f05a44681382029746c3ea533caa0c9d commit 698f5fb8f05a44681382029746c3ea533caa0c9d Author: Min Qin <qinmin@chromium.org> Date: Wed Dec 05 06:26:23 2018 Add instrumentation test for resuming a download with service manager only mode This CL adds an instrumentation test for resuming a download with only service manager running BUG= 910734 Change-Id: I0b8468b8db5920bdc8c3b57ad155865771f36fe0 Reviewed-on: https://chromium-review.googlesource.com/c/1357600 Commit-Queue: Min Qin <qinmin@chromium.org> Reviewed-by: Xing Liu <xingliu@chromium.org> Reviewed-by: Xi Han <hanxi@chromium.org> Reviewed-by: Tommy Nyquist <nyquist@chromium.org> Cr-Commit-Position: refs/heads/master@{#613886} [modify] https://crrev.com/698f5fb8f05a44681382029746c3ea533caa0c9d/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerService.java [modify] https://crrev.com/698f5fb8f05a44681382029746c3ea533caa0c9d/chrome/android/java_sources.gni [add] https://crrev.com/698f5fb8f05a44681382029746c3ea533caa0c9d/chrome/android/javatests/src/org/chromium/chrome/browser/ServicificationBackgroundService.java [modify] https://crrev.com/698f5fb8f05a44681382029746c3ea533caa0c9d/chrome/android/javatests/src/org/chromium/chrome/browser/ServicificationBackgroundServiceTest.java [add] https://crrev.com/698f5fb8f05a44681382029746c3ea533caa0c9d/chrome/android/javatests/src/org/chromium/chrome/browser/download/ServicificationDownloadTest.java [modify] https://crrev.com/698f5fb8f05a44681382029746c3ea533caa0c9d/chrome/browser/android/download/download_manager_service.cc [modify] https://crrev.com/698f5fb8f05a44681382029746c3ea533caa0c9d/chrome/browser/android/download/download_manager_service.h [modify] https://crrev.com/698f5fb8f05a44681382029746c3ea533caa0c9d/components/download/internal/common/in_progress_download_manager.cc [modify] https://crrev.com/698f5fb8f05a44681382029746c3ea533caa0c9d/components/download/public/common/in_progress_download_manager.h
Comment 1 by hanxi@chromium.org
, Nov 30