Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(51)

Unified Diff: chrome/browser/android/webapk/webapk_installer_unittest.cc

Issue 2515293004: Chrome talks to Play to install WebAPKs. (Closed)
Patch Set: Don't use play install in webapk_installer_unittest. Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/android/webapk/webapk_installer_unittest.cc
diff --git a/chrome/browser/android/webapk/webapk_installer_unittest.cc b/chrome/browser/android/webapk/webapk_installer_unittest.cc
index 10021ff93107a0b7e7a694291742e2e107c0910b..d94322632930ed9a1dd0e7f18ed8b5b3bffa617f 100644
--- a/chrome/browser/android/webapk/webapk_installer_unittest.cc
+++ b/chrome/browser/android/webapk/webapk_installer_unittest.cc
@@ -52,7 +52,8 @@ const char* kDownloadedWebApkPackageName = "party.unicode";
// WebApkInstaller subclass where
// WebApkInstaller::StartInstallingDownloadedWebApk() and
-// WebApkInstaller::StartUpdateUsingDownloadedWebApk() are stubbed out.
+// WebApkInstaller::StartUpdateUsingDownloadedWebApk() and
+// WebApkInstaller::HasGooglePlayWebApkInstallDelegate() are stubbed out.
class TestWebApkInstaller : public WebApkInstaller {
public:
TestWebApkInstaller(const ShortcutInfo& shortcut_info,
@@ -73,6 +74,10 @@ class TestWebApkInstaller : public WebApkInstaller {
return true;
}
+ bool HasGooglePlayWebApkInstallDelegate() override {
+ return false;
+ }
+
void PostTaskToRunSuccessCallback() {
base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE,

Powered by Google App Engine
This is Rietveld 408576698