Commit a251ed9f authored by binji@chromium.org's avatar binji@chromium.org

[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
parent bb04c4ed
...@@ -91,7 +91,10 @@ cr.define('options.personal_options', function() { ...@@ -91,7 +91,10 @@ cr.define('options.personal_options', function() {
/** @inheritDoc */ /** @inheritDoc */
activateItemAtIndex: function(index) { 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);
}, },
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment