New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 626426 link

Starred by 2 users
Status: Verified
Owner:
█ krishna...@chromium.org
Last visit > 30 days ago
Closed: Nov 2016
Cc:
dchan@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature



Sign in to add a comment

Write policy_NotificationsAllowedForUrls test using EP Framework

Project Member Reported by scunning...@chromium.org, Jul 7 2016 
Write automated test for policy_NotificationsAllowedForUrls using EP Framework. Note that test also should verify DefaultNotificationsSetting=1 (ask) and 2 (block).

Comment 1 by scunning...@chromium.org, Jul 7 2016

Components: -Test>Missing Tests>Missing

Comment 2 by krishna...@chromium.org, Oct 6 2016

Status: Started (was: Available)
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 13 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/faa5ece29f71fc7b05db3a6a6b89818f4f859f7c

commit faa5ece29f71fc7b05db3a6a6b89818f4f859f7c
Author: Krishna Gavini <krishnargv@chromium.org>
Date: Wed Oct 12 22:47:20 2016

Add policy_NotificationsAllowedForUrls test to telemetry suite.

BUG= chromium:626426 
TEST=test_that <IP> policy_NotificationsAllowedForUrls --args='case=siteallowed_show'

Change-Id: I48ac4fa5b66f3c155e63d79fd7dbbca5fe9814ec
Reviewed-on: https://chromium-review.googlesource.com/397939
Commit-Ready: Krishna Gavini <krishnargv@chromium.org>
Tested-by: Krishna Gavini <krishnargv@chromium.org>
Reviewed-by: Scott Cunningham <scunningham@chromium.org>

[modify] https://crrev.com/faa5ece29f71fc7b05db3a6a6b89818f4f859f7c/chromeos-base/autotest-chrome/autotest-chrome-9999.ebuild

Comment 4 by krishna...@chromium.org, Nov 1 2016

Status: Verified (was: Started)
This running in the lab and has been passing for a while.
►

Sign in to add a comment