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

Unified Diff: chrome/browser/android/webapk/webapk_installer.h

Issue 2782313002: Add static method WebApkIconHasher::DownloadAndComputeMurmur2Hash (Closed)
Patch Set: Delete non-static download method` Created 3 years, 9 months 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.h
diff --git a/chrome/browser/android/webapk/webapk_installer.h b/chrome/browser/android/webapk/webapk_installer.h
index 999751a1d7f87da62ead6246a23774086c3534dd..2e356a4dccdb9d7d0c17b989c6be364fb077ad03 100644
--- a/chrome/browser/android/webapk/webapk_installer.h
+++ b/chrome/browser/android/webapk/webapk_installer.h
@@ -33,8 +33,6 @@ namespace webapk {
class WebApk;
}
-class WebApkIconHasher;
-
// Talks to Chrome WebAPK server and Google Play to generate a WebAPK on the
// server, download it, and install it. The native WebApkInstaller owns the
// Java WebApkInstaller counterpart.
@@ -160,9 +158,6 @@ class WebApkInstaller : public net::URLFetcherDelegate {
// net::URLFetcherDelegate:
void OnURLFetchComplete(const net::URLFetcher* source) override;
- // Downloads app icon in order to compute Murmur2 hash.
- void DownloadAppIconAndComputeMurmur2Hash();
-
// Called with the computed Murmur2 hash for the app icon.
void OnGotIconMurmur2Hash(const std::string& icon_murmur2_hash);
@@ -220,9 +215,6 @@ class WebApkInstaller : public net::URLFetcherDelegate {
// Sends HTTP request to WebAPK server.
std::unique_ptr<net::URLFetcher> url_fetcher_;
- // Downloads app icon and computes Murmur2 hash.
- std::unique_ptr<WebApkIconHasher> icon_hasher_;
-
// Downloads WebAPK.
std::unique_ptr<FileDownloader> downloader_;
« no previous file with comments | « chrome/browser/android/webapk/webapk_icon_hasher_unittest.cc ('k') | chrome/browser/android/webapk/webapk_installer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698