Issue metadata
Sign in to add a comment
|
Add browser_tests for push messaging that exercise the permission prompt dismissed case |
||||||||||||||||||||||
Issue descriptionWe should have a browser test to exercise the case when a permission prompt is dismissed. peter: Is it ok to assign this to you? I can help out if needed. I don't think it should be too hard, just a matter of adding a case to PushMessagingBrowserTest. Thanks!
,
Sep 8 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/efdf980d6049586a6860e36edd44729ccf76996a commit efdf980d6049586a6860e36edd44729ccf76996a Author: johnme <johnme@chromium.org> Date: Thu Sep 08 18:26:10 2016 Add tests for dismissing/denying push permission prompt on Android BUG=644050 Review-Url: https://codereview.chromium.org/2323753002 Cr-Commit-Position: refs/heads/master@{#417345} [modify] https://crrev.com/efdf980d6049586a6860e36edd44729ccf76996a/chrome/android/javatests/src/org/chromium/chrome/browser/push_messaging/PushMessagingTest.java
,
Sep 8 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8569349a2517d5429a716a0fc78c609f34d2c815 commit 8569349a2517d5429a716a0fc78c609f34d2c815 Author: johnme <johnme@chromium.org> Date: Thu Sep 08 19:52:33 2016 Add tests for notifications infobar on Android BUG=644050 Review-Url: https://codereview.chromium.org/2316303004 Cr-Commit-Position: refs/heads/master@{#417369} [modify] https://crrev.com/8569349a2517d5429a716a0fc78c609f34d2c815/chrome/android/javatests/src/org/chromium/chrome/browser/notifications/NotificationPlatformBridgeTest.java [modify] https://crrev.com/8569349a2517d5429a716a0fc78c609f34d2c815/chrome/test/data/notifications/android_test.html
,
Nov 22 2016
,
Nov 29 2016
,
Nov 10 2017
,
Feb 18 2018
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by joh...@chromium.org
, Sep 8 2016