Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(47)

Issue 545159: Insert diagnostics in an attempt to track down a crash in popup menus... (Closed)

Created:
10 years, 11 months ago by Pam (message me for reviews)
Modified:
9 years, 5 months ago
Reviewers:
Mark Mentovai
CC:
chromium-reviews_googlegroups.com, brettw+cc_chromium.org, ben+cc_chromium.org, John Grabowski, jam, pam+watch_chromium.org, darin (slow to review)
Visibility:
Public.

Description

Insert diagnostics in an attempt to track down a crash in popup menus in RenderWidgetHostViewMac::ShowPopupWithItems(). BUG=31716 TEST=should have no user-visible effect; see crash reports Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=36790

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -0 lines) Patch
M chrome/browser/renderer_host/render_widget_host_view_mac.h View 1 chunk +4 lines, -0 lines 0 comments Download
M chrome/browser/renderer_host/render_widget_host_view_mac.mm View 2 chunks +11 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Pam (message me for reviews)
10 years, 11 months ago (2010-01-21 20:53:46 UTC) #1
Mark Mentovai
10 years, 11 months ago (2010-01-21 21:05:05 UTC) #2
LGTM.  Let's see how this shakes out.  Thanks.

Powered by Google App Engine
This is Rietveld 408576698