Issue 621873 link

Starred by 2 users

Issue metadata

Status:	Fixed
Owner:	█ harkness@chromium.org Last visit > 30 days ago
Closed:	Sep 2016
EstimatedDays:	----
NextAction:	----
OS:	All
Pri:	3
Type:	Bug

NotificationDatabase should support InvalidArgument error state

Project Member Reported by harkness@chromium.org, Jun 21 2016

Issue description

LevelDB added an InvalidArgument error state, and the NotificationDatabase error codes should be updated to reflect the new error type.

Project Member

Comment 1 by bugdroid1@chromium.org, Jun 22 2016

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

commit e564632b1c1214efcab6a4be0ae5ed4af758fac2
Author: harkness <harkness@chromium.org>
Date: Wed Jun 22 08:56:22 2016

Add InvalidArgument error type for NotificationDatabase.

BUG= 621873 

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

[modify] https://crrev.com/e564632b1c1214efcab6a4be0ae5ed4af758fac2/content/browser/notifications/notification_database.cc
[modify] https://crrev.com/e564632b1c1214efcab6a4be0ae5ed4af758fac2/content/browser/notifications/notification_database.h
[modify] https://crrev.com/e564632b1c1214efcab6a4be0ae5ed4af758fac2/tools/metrics/histograms/histograms.xml

Comment 2 by harkness@chromium.org, Sep 27 2016

Status: Fixed (was: Assigned)

►

Issue 621873 link

Issue metadata

NotificationDatabase should support InvalidArgument error state

Issue description

Comment 1 by bugdroid1@chromium.org, Jun 22 2016

Comment 1 Deleted

Comment 2 by harkness@chromium.org, Sep 27 2016

Comment 2 Deleted

Sign in to add a comment