chrome crashed after I clicked clear all notifications in the notifications center |
|||||||||
Issue descriptionIt happens on 59.0.3035.0 and may also happen on M58. feedback: https://feedback.corp.google.com/#/Report/54855999613
,
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
,
Mar 15 2017
I found the root cause. will send a fix soon.
,
Mar 15 2017
It looks already fixed in Issue 687232?
,
Mar 15 2017
Per chat with yhanada@ offline, Issue 687232? is a different issue.
,
Mar 15 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9f177cc7d313503e07a143337105868ef182022b commit 9f177cc7d313503e07a143337105868ef182022b Author: yhanada <yhanada@chromium.org> Date: Wed Mar 15 06:03:11 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} [modify] https://crrev.com/9f177cc7d313503e07a143337105868ef182022b/ui/arc/notification/arc_custom_notification_view.cc [modify] https://crrev.com/9f177cc7d313503e07a143337105868ef182022b/ui/message_center/message_center_impl.cc
,
Mar 17 2017
,
Mar 17 2017
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
,
Mar 20 2017
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
,
Mar 20 2017
Clearing added cc list by sheriffbot. sorry for the spam. thanks.
,
Mar 21 2017
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
,
Mar 21 2017
,
Mar 30 2017
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by yhanada@chromium.org
, Mar 14 2017