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

Unified Diff: ash/common/system/ime/tray_ime_chromeos.cc

Issue 2805733002: [Ash] Use DCHECK in ImeListView::HandleButtonPressed() (Closed)
Patch Set: Created 3 years, 8 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
Index: ash/common/system/ime/tray_ime_chromeos.cc
diff --git a/ash/common/system/ime/tray_ime_chromeos.cc b/ash/common/system/ime/tray_ime_chromeos.cc
index e395bd20e66088e7f48ebd2657e8e28eba74ede5..7317d7c62c7f010f7038e8e2f04b18099aa79a82 100644
--- a/ash/common/system/ime/tray_ime_chromeos.cc
+++ b/ash/common/system/ime/tray_ime_chromeos.cc
@@ -103,9 +103,10 @@ class IMEDetailedView : public ImeListView {
void HandleButtonPressed(views::Button* sender,
const ui::Event& event) override {
- ImeListView::HandleButtonPressed(sender, event);
if (sender == settings_button_)
ShowSettings();
+ else
+ ImeListView::HandleButtonPressed(sender, event);
}
void CreateExtraTitleRowButtons() override {

Powered by Google App Engine
This is Rietveld 408576698