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

Issue 729247 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jun 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

PlatformNotificationServiceTest.DisplayPageDisplayedEvent failed

Project Member Reported by michaelbai@chromium.org, Jun 2 2017

Issue description

It should relate to change https://codereview.chromium.org/2888303004/

but the revert of patch failed. Disable the test and assigned to the patch owner
 
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 3 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0b479f2338e67a4d5ad7922ab9760b7b7137916b

commit 0b479f2338e67a4d5ad7922ab9760b7b7137916b
Author: michaelbai <michaelbai@chromium.org>
Date: Sat Jun 03 01:00:09 2017

Disable PlatformNotificationServiceTest.DisplayPageDisplayedEvent in Android

TBR=miguelg
BUG= 729247 

Review-Url: https://codereview.chromium.org/2916363004
Cr-Commit-Position: refs/heads/master@{#476845}

[modify] https://crrev.com/0b479f2338e67a4d5ad7922ab9760b7b7137916b/chrome/browser/notifications/platform_notification_service_unittest.cc

 Issue 729328  has been merged into this issue.
Status: Fixed (was: Untriaged)
The test was removed as part of a larger refator where we are getting rid of delegates altogheter

https://codereview.chromium.org/2906883003/

Sign in to add a comment