diff options
author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-21 19:01:19 +0000 |
---|---|---|
committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-21 19:01:19 +0000 |
commit | f4f50efb4d3ee42470c8057345e902ac42e76801 (patch) | |
tree | 5b4d0c57cd94d8169db9dc441f424d6443e4f490 /chrome/browser/translate/languages_menu_model.cc | |
parent | 0bd22cab9c919ff8d9b35153f6799dc031fe2770 (diff) | |
download | chromium_src-f4f50efb4d3ee42470c8057345e902ac42e76801.zip chromium_src-f4f50efb4d3ee42470c8057345e902ac42e76801.tar.gz chromium_src-f4f50efb4d3ee42470c8057345e902ac42e76801.tar.bz2 |
Cleanup: de-inline a bunch of classes, rename and move "PluginInstaller" to "PluginInstallerInfoBarDelegate" for clarity, lots of other misc. stuff
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6249010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72166 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/translate/languages_menu_model.cc')
-rw-r--r-- | chrome/browser/translate/languages_menu_model.cc | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/chrome/browser/translate/languages_menu_model.cc b/chrome/browser/translate/languages_menu_model.cc index 5f631d1..bd3ec10 100644 --- a/chrome/browser/translate/languages_menu_model.cc +++ b/chrome/browser/translate/languages_menu_model.cc @@ -1,4 +1,4 @@ -// Copyright (c) 2010 The Chromium Authors. All rights reserved. +// Copyright (c) 2011 The Chromium Authors. All rights reserved. // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. @@ -21,16 +21,16 @@ LanguagesMenuModel::~LanguagesMenuModel() { } bool LanguagesMenuModel::IsCommandIdChecked(int command_id) const { - if (language_type_ == ORIGINAL) - return command_id == translate_infobar_delegate_->original_language_index(); - return command_id == translate_infobar_delegate_->target_language_index(); + return command_id == ((language_type_ == ORIGINAL) ? + translate_infobar_delegate_->original_language_index() : + translate_infobar_delegate_->target_language_index()); } bool LanguagesMenuModel::IsCommandIdEnabled(int command_id) const { - // Prevent from having the same language selectable in original and target. - if (language_type_ == ORIGINAL) - return command_id != translate_infobar_delegate_->target_language_index(); - return command_id != translate_infobar_delegate_->original_language_index(); + // Prevent the same language from being selectable in original and target. + return command_id != ((language_type_ == ORIGINAL) ? + translate_infobar_delegate_->target_language_index() : + translate_infobar_delegate_->original_language_index()); } bool LanguagesMenuModel::GetAcceleratorForCommandId( |