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

Issue 701348 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

chrome crashed after I clicked clear all notifications in the notifications center

Project Member Reported by yhanada@chromium.org, Mar 14 2017

Issue description

It happens on 59.0.3035.0 and may also happen on M58.

feedback: https://feedback.corp.google.com/#/Report/54855999613
 
Description: Show this description

Comment 2 by kanliu@chromium.org, Mar 14 2017

happened again, here's the crash id

Crash ID Chrome (Server ID: a3f72d6820000000)

Crash report uploaded on Tuesday, March 14, 2017 at 11:12:54 AM

Provide additional details
Owner: yhanada@chromium.org
Status: Started (was: Available)
I found the root cause. will send a fix soon.
It looks already fixed in Issue 687232?
Per chat with yhanada@ offline, Issue 687232? is a different issue.
Project Member

Comment 6 by bugdroid1@chromium.org, Mar 15 2017

Labels: Merge-Request-58
Project Member

Comment 8 by sheriffbot@chromium.org, Mar 17 2017

Labels: -Merge-Request-58 Hotlist-Merge-Approved Merge-Approved-58
Your change meets the bar and is auto-approved for M58. Please go ahead and merge the CL to branch 3029 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 9 by sheriffbot@chromium.org, Mar 20 2017

Cc: ketakid@google.com
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: -ketakid@google.com
Clearing added cc list by sheriffbot. sorry for the spam. thanks.
Project Member

Comment 11 by bugdroid1@chromium.org, Mar 21 2017

Labels: -merge-approved-58 merge-merged-3029
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ca74b3b18d45dbc81f3869d04ec51bdd95f51d57

commit ca74b3b18d45dbc81f3869d04ec51bdd95f51d57
Author: yhanada <yhanada@chromium.org>
Date: Tue Mar 21 05:33:44 2017

Fix crash when removing all notifications by "Clear all" button.

BUG= 701348 

Review-Url: https://codereview.chromium.org/2750853005
Cr-Commit-Position: refs/heads/master@{#457005}
(cherry picked from commit 9f177cc7d313503e07a143337105868ef182022b)

Review-Url: https://codereview.chromium.org/2762933002 .
Cr-Commit-Position: refs/branch-heads/3029@{#318}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}

[modify] https://crrev.com/ca74b3b18d45dbc81f3869d04ec51bdd95f51d57/ui/arc/notification/arc_custom_notification_view.cc
[modify] https://crrev.com/ca74b3b18d45dbc81f3869d04ec51bdd95f51d57/ui/message_center/message_center_impl.cc

Status: Fixed (was: Started)
Status: Verified (was: Fixed)

Sign in to add a comment