diff options
| author | binji@chromium.org <binji@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-18 20:46:01 +0000 |
|---|---|---|
| committer | binji@chromium.org <binji@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-18 20:46:01 +0000 |
| commit | a251ed9f25d99c102364228806e9c998d62fc9ee (patch) | |
| tree | 9fefb659971313fd05e6fbdd38b8f374efd59479 /chrome/browser/resources/options | |
| parent | bb04c4ed964981493b8727cf67257d2ad6e8128b (diff) | |
| download | chromium_src-a251ed9f25d99c102364228806e9c998d62fc9ee.zip chromium_src-a251ed9f25d99c102364228806e9c998d62fc9ee.tar.gz chromium_src-a251ed9f25d99c102364228806e9c998d62fc9ee.tar.bz2 | |
[Multi Profile] In multi-profile list, don't allow double-clicking to edit a
profile that isn't current.
BUG=92536
TEST=Go to chrome://settings/personal. Create a profile. Try to double-click on a non-current profile. Nothing should happen.
Review URL: http://codereview.chromium.org/8549013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110745 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/resources/options')
| -rw-r--r-- | chrome/browser/resources/options/personal_options_profile_list.js | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/chrome/browser/resources/options/personal_options_profile_list.js b/chrome/browser/resources/options/personal_options_profile_list.js index 755150c..64436a3 100644 --- a/chrome/browser/resources/options/personal_options_profile_list.js +++ b/chrome/browser/resources/options/personal_options_profile_list.js @@ -91,7 +91,10 @@ cr.define('options.personal_options', function() { /** @inheritDoc */ activateItemAtIndex: function(index) { - ManageProfileOverlay.showManageDialog(this.dataModel.item(index)); + // Don't allow the user to edit a profile that is not current. + var profileInfo = this.dataModel.item(index); + if (profileInfo.isCurrentProfile) + ManageProfileOverlay.showManageDialog(profileInfo); }, }; |