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

Unified Diff: views/controls/menu/menu_controller.cc

Issue 6360015: Removes debugging code as cause of crash was found. I'm TBRing as this (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: views/controls/menu/menu_controller.cc
diff --git a/views/controls/menu/menu_controller.cc b/views/controls/menu/menu_controller.cc
index dbc31a4d7a26b92f77565564cfdb218c00d79eb9..af4c2762735c66a313dfed2fa7b756debd0ba3b6 100644
--- a/views/controls/menu/menu_controller.cc
+++ b/views/controls/menu/menu_controller.cc
@@ -745,7 +745,7 @@ void MenuController::SetSelection(MenuItemView* menu_item,
if (menu_item && menu_item->GetDelegate())
menu_item->GetDelegate()->SelectionChanged(menu_item);
- CHECK(menu_item || (selection_types & SELECTION_EXIT) != 0);
+ DCHECK(menu_item || (selection_types & SELECTION_EXIT) != 0);
pending_state_.item = menu_item;
pending_state_.submenu_open = (selection_types & SELECTION_OPEN_SUBMENU) != 0;
@@ -1668,7 +1668,7 @@ void MenuController::RepostEvent(SubmenuView* source,
// We some times get an event after closing all the menus. Ignore it.
// Make sure the menu is in fact not visible. If the menu is visible, then
// we're in a bad state where we think the menu isn't visibile but it is.
- CHECK(!source->GetWidget()->IsVisible());
+ DCHECK(!source->GetWidget()->IsVisible());
return;
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698