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

Unified Diff: components/payments/android/web_app_manifest_section_table.h

Issue 2838433002: [Payments] Cache payment manifests. (Closed)
Patch Set: address comments Created 3 years, 8 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: components/payments/android/web_app_manifest_section_table.h
diff --git a/components/payments/android/web_app_manifest_section_table.h b/components/payments/android/web_app_manifest_section_table.h
index 7c46bcc872e57df186fb6672cce78bac258bba10..12322e715c4dbef93dc19f46abfd603eb1562420 100644
--- a/components/payments/android/web_app_manifest_section_table.h
+++ b/components/payments/android/web_app_manifest_section_table.h
@@ -40,13 +40,15 @@ class WebAppManifestSectionTable : public WebDatabaseTable {
bool IsSyncable() override;
bool MigrateToVersion(int version, bool* update_compatible_version) override;
- // Adds the web app |*manifest|. Note that the previous web app manifest will
+ // Adds the web app |manifest|. Note that the previous web app manifest will
// be deleted.
- bool AddWebAppManifest(mojom::WebAppManifestSection* manifest);
+ bool AddWebAppManifest(
+ const std::vector<mojom::WebAppManifestSectionPtr>& manifest);
- // Gets manifest of the |web_app|. Returns nullptr if no manifest exists for
- // the |web_app|.
- mojom::WebAppManifestSectionPtr GetWebAppManifest(const std::string& web_app);
+ // Gets manifest of the |web_app|. Returns empty vector if no manifest exists
+ // for the |web_app|.
+ std::vector<mojom::WebAppManifestSectionPtr> GetWebAppManifest(
+ const std::string& web_app);
private:
DISALLOW_COPY_AND_ASSIGN(WebAppManifestSectionTable);

Powered by Google App Engine
This is Rietveld 408576698