Confusing class names
Reported by
dyaros...@yandex-team.ru,
Jan 20 2017
|
||||
Issue descriptionThere are two classes named NotificationDelegate. - https://cs.chromium.org/chromium/src/ui/message_center/notification_delegate.h?q=NotificationDelegate&sq=package:chromium&dr=CSs&l=26 - https://cs.chromium.org/chromium/src/chrome/browser/notifications/notification_delegate.h?q=NotificationDelegate&sq=package:chromium&l=16&dr=CSs One of them should probably be renamed.
,
Jan 25 2017
,
Feb 12 2018
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Feb 28 2018
Quoted classes no longer exist |
||||
►
Sign in to add a comment |
||||
Comment 1 by rbasuvula@chromium.org
, Jan 24 2017Labels: TE-NeedsTriageHelp