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

Unified Diff: chrome/test/data/banners/cancel_test_page.html

Issue 2677853002: [Webapps]: Clear AppBannerManager::page_requested_prompt_ at start of banner flow (Closed)
Patch Set: Merge branch 'master' into install_banner2 Created 3 years, 10 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/test/data/banners/cancel_test_page.html
diff --git a/chrome/test/data/banners/cancel_test_page.html b/chrome/test/data/banners/cancel_test_page.html
index 5f9d9af49bbd28aa12af8389b97c17dcba5464d5..fb1c77a1cc00e5c5c4512788346602b73a0417a2 100644
--- a/chrome/test/data/banners/cancel_test_page.html
+++ b/chrome/test/data/banners/cancel_test_page.html
@@ -1,9 +1,13 @@
<html>
<head>
<title>Web app banner test page</title>
- <link rel="manifest" href="manifest.json" />
<script src="main.js"></script>
<script>
+ // If a "manifest=/path/to/manifest.json" query argument is provided to
+ // the URL accessing this page, that path is injected as the manifest tag.
+ // Otherwise, "manifest.json" is used as the manifest tag.
+ addManifestLinkTag();
+
window.addEventListener('beforeinstallprompt', function(e) {
console.log('Preventing banner from appearing');
e.preventDefault();
« no previous file with comments | « chrome/browser/banners/app_banner_manager_browsertest.cc ('k') | chrome/test/data/banners/manifest_different_start_url.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698